Skip to content

Set animation at runtime via arguments #234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Set animation at runtime via arguments #234

wants to merge 3 commits into from

Conversation

@SealedJoy
Copy link

@SealedJoy SealedJoy commented Aug 21, 2019 •
edited
Loading

Allows user to choose animation at runtime.
E.g.
python python/visualization.py spectrum
will start in spectrum mode

@joeybab3 joeybab3 changed the title Callwithargs Set animation at runtime via arguments Aug 21, 2019
@joeybab3
Copy link
Collaborator

As mentioned in the other PR I unfortunately do not have a ws2812 strip and esp to test this with so I can't merge until I am able which might be a bit longer as I need to wait for a strip and esp to get here from China.

Thanks for your contributions though and I will come back asap with feedback once I can test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joeybab3 joeybab3 joeybab3 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SealedJoy @joeybab3