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 visualization #765

Merged
merged 9 commits into from
Jun 4, 2024
Merged

Audio visualization #765

merged 9 commits into from
Jun 4, 2024

Conversation

panxinmiao
Copy link
Contributor

@panxinmiao panxinmiao commented May 24, 2024

An Example of Audio Visualization.

This PR implements a simple audio analyzer based on W3C standards and demonstrates how to create an audio visualization application.
See: https://www.w3.org/TR/webaudio/#AnalyserNode

@panxinmiao panxinmiao mentioned this pull request May 24, 2024
@almarklein
Copy link
Collaborator

Nice! A smaller sound file might be nice though 😅

@panxinmiao
Copy link
Contributor Author

Nice! A smaller sound file might be nice though 😅

To prevent large binary files from being included in the PR, this example has been modified to use a network address for music playback. To avoid the long startup delay caused by downloading the entire file before playing, I have implemented a simple streaming solution that allows downloading and playing simultaneously. This functionality works well on my computer, but I'm unsure about the experience under poor network conditions. Achieving a better experience in such scenarios might require estimating the buffer size based on network speed and remaining playback time, though I don't think this is crucial for this example.

Additionally, this example also supports playing local files and loading network addresses in a non-streaming manner ( downloading the entire file before playing). You can achieve this by simply modifying the parameters in the example.

@panxinmiao panxinmiao marked this pull request as ready for review May 29, 2024 07:51
@panxinmiao panxinmiao requested a review from Korijn as a code owner May 29, 2024 07:51
@panxinmiao
Copy link
Contributor Author

This PR is ready for review now.

@Korijn Korijn enabled auto-merge (squash) June 4, 2024 10:57
@Korijn Korijn merged commit 217d90e into pygfx:main Jun 4, 2024
12 checks passed
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.

None yet

3 participants