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

Incorporate effects from Animatrix #272

Open
davepl opened this issue May 19, 2023 · 11 comments
Open

Incorporate effects from Animatrix #272

davepl opened this issue May 19, 2023 · 11 comments

Comments

@davepl
Copy link
Contributor

davepl commented May 19, 2023

Feature

There are a number of interesting matrix effects to be found here:

https://github.com/StefanPetrick/animartrix/blob/main/animation_collection.ino

I think we should port some of the best ones to NightDriver. They would need to be wrapped in LEDStripEffect, and the oscillators likely need to go in GFXBase.

@robertlipe
Copy link
Contributor

Be respectful of his license: https://github.com/StefanPetrick/animartrix#license

@davepl
Copy link
Contributor Author

davepl commented May 24, 2023

It's Creative Commons Attribution, the only license collision I can spot is non-commercial use, and we're non-commercial anyway. I doubt GPL3 is compatible with CCA, so likely not a good idea to use any of this code. But looking at the code, not stuff I'd want directly included as is because it's a big monolithic source file. Could be a good source of ideas for new effects, though.

@robertlipe
Copy link
Contributor

Agreed on most points of that interpretation. Just wanted to call it out before anyone took "port" toooo literally. Look for mathematic inspiration.

(e.g. learn from https://fabiensanglard.net/doom_fire_psx/ - don't copy and paste from it.)

@davepl
Copy link
Contributor Author

davepl commented May 24, 2023

Agree 100%... my bad for not having read the CCA license more closely before making the suggestion. Appreciate you catching it!

@Bun-Bun
Copy link

Bun-Bun commented May 31, 2023

This is something I would be interested in looking at. There are some neat effects in his library.

As far as the licensing goes, what is the best way to handle "learn from https://fabiensanglard.net/doom_fire_psx/ - don't copy and paste from it."? Do you give credit for inspiration in comments of the code?

@rbergen
Copy link
Collaborator

rbergen commented May 31, 2023

I'd say it is a matter of not copying/pasting any code, but be inspired by the mathematics used. The code will need to be structured to fit into this project's effect structure anyway.

@Bun-Bun
Copy link

Bun-Bun commented May 31, 2023

Yes, I understand that. But does one give any credit where the inspiration for the mathematics came from? Or just silently take said inspiration?

@rbergen
Copy link
Collaborator

rbergen commented May 31, 2023

Personally I would give credit, and have done so in the past. There already are examples of source code comment credits within NightDriverStrip as well (case in point: PatternPongClock.h).

@Bun-Bun
Copy link

Bun-Bun commented May 31, 2023

Perfect. That makes sense to me and will do the same.

@robertlipe
Copy link
Contributor

robertlipe commented May 31, 2023 via email

@Bun-Bun
Copy link

Bun-Bun commented May 31, 2023

Thank you for that expansion of what you said. Makes sense to me.

I think the process of bringing the ideas over to this project (as davepl pointed out, it can't be directly included anyway due to the differing structure of the projects) as well as the process of learning and experimentation of the math/ideas would take care of any copyright issues.

Ethically it makes sense to me to give credit where inspirations came from. See it all the time in the music world but then you do see instances where some are still offended and seek legal action. My coding experience is all from a purely tinkering/hobbyist or 100% private business (ie. nothing shared publicly). Seeing others insights on the licensing aspect is valuable. Personally I'd be glad if someone learned something from my code and was inspired to create something the same or similar and would appreciate credit.

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

No branches or pull requests

4 participants