Skip to content

Conversation

@alexdmiller
Copy link
Contributor

@alexdmiller alexdmiller commented Jan 2, 2021

This is a port of Minim's beat detection implementation (source here) to Processing Sound.

If this were to be merged, there is still some work to do to clean it up and add documentation. But I wanted to figure out if merging this into processing-sound is something people even want. Also, I'm not quite sure what the licensing implications are with this PR.

Thanks!

@kevinstadler
Copy link
Collaborator

That looks like great work, much appreciated! The original code is GPL-licensed which is compatible with this library's LGPL, we should probably just include an attribution ('based on ....') to the original author.

Regarding documentation, Processing's new web documentation pipeline should go live in the next few weeks, which will make the XML files in the processing/processing-docs repository obsolete, as all documentation will then be built directly from the JavaDoc comments in the library source code! If you were happy to add some basic documentation of the public methods you could check the format of the class/method comments in other analyzer classes e.g. here. If not I hope I'd get around to it in the next couple of weeks when I package the code up to make a new library release...(which will also address some of #58)!

@alexdmiller
Copy link
Contributor Author

@kevinstadler great, glad to hear this could be useful!

I will update this PR with attribution and JavaDoc documentation sometime within the next week (probably next weekend).

@alexdmiller
Copy link
Contributor Author

I have updated this PR with JavaDoc comments as well as a Processing example. Let me know what you think!

Screen.Recording.2021-01-09.at.9.10.32.AM.mov

@kevinstadler
Copy link
Collaborator

All looks good, do you have any license information about the sound wave included in the example?

@alexdmiller
Copy link
Contributor Author

Sorry for the long delay to answer your simple question. I created the sound file and am happy to have it fall under whatever license governs the repository as a whole?

I added a comment attributing the beat detection code to minim. Let me know if you think that should look different.

@kevinstadler
Copy link
Collaborator

No worries, I think that all looks great! I'll hopefully get around to releasing a new version (and finally adding some proper attribution to contributors like yourself...) this week, thanks!

@kevinstadler kevinstadler merged commit f8767fd into processing:master Jan 19, 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