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

Audio Mode #174

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Audio Mode #174

wants to merge 2 commits into from

Conversation

gsus24
Copy link

@gsus24 gsus24 commented May 2, 2021

This is for audio visualization on some effects like rainbow or plasma.
By captering internal audio and drawing a gradient view with scaled dimensions of the screen.
Every section in the fan view is dimmed or light up depending on the sampled audio waveform.

For now its audio only.

Would be great to see this feature in master.

@abrenoch
Copy link
Owner

abrenoch commented May 4, 2021

This sounds awesome! I need to get a development environment set back up but I'll try to get to this soon.

@gsus24
Copy link
Author

gsus24 commented May 5, 2021

Nice to hear. I used the same classes for encoder, service and thread. In my opinion, they can be put together and pass parameter for video/audio when starting the service. I am no android developer, take a look yourself.

For next step, it need to change the gradientview which draws the fan. I used the SweepGradient(), which produce unexpected results, when the values are not monotic. You can see it good at RGB-White-Effect, that the sections are not always red,green,blue,.... For the other effects it works quite good.

Do you have an idea for custom gradientview to draw a fan?
If i can help implement, please let me know. Would be great to see this in future

@thundergreen
Copy link

Any news here ?

@StEfDu6o
Copy link

Interesting, maybe in 2033...

@abrenoch
Copy link
Owner

Hey sorry it took me a while to get back to this. However I get an java.lang.IndexOutOfBoundsException when reading the buffer data (from HyperionAudioEncoder.java:232). This seems interesting but would need some work before I could accept the PR as it currently is (such as support for the tv version).

@StEfDu6o By 2033 google will have made their own APIs so useless it won't even be a concern of anybody's

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.

4 participants