Skip to content
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.

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

Colorwaves! Like many others, Spheres. Only multiple. #16

Closed
wants to merge 4 commits into from

Conversation

Tirine
Copy link
Contributor

@Tirine Tirine commented Dec 26, 2020

Also, like many others, using simulator for Pull request #5, hope it works fine.
Did send it to Matt, though I did so last evening and noticed some mistakes today.. :)

Except for multiple spheres, each sphere is different from the others in slight ways, which I belive can easily be changed.

Not so used with Python, so seems like I wrote some functions myself that are probably better to use numpy or other libraries for, which I've seen afterwards many others have been using.

@oliverdunk
Copy link
Member

Hey @Tirine! Matt asked if I could help with maintaining this repository, so I've been thinking about how best to do that. I've decided to create an examples folder where we can add contributions like this one - you can find some more information in the README.

Would you be up for updating your PR, to move your effects in to the examples folder?

@oliverdunk
Copy link
Member

Closing in favour of #24 :)

@oliverdunk oliverdunk closed this Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants