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 nodes crash when changing file on Filestream / #36

Open
tekcor opened this issue May 19, 2015 · 2 comments
Open

audio nodes crash when changing file on Filestream / #36

tekcor opened this issue May 19, 2015 · 2 comments

Comments

@tekcor
Copy link
Contributor

tekcor commented May 19, 2015

@geloescht @zauner

Hi
I was quite amazed when I realized that I could build an online DJ mixer application already now.
So I did that and it worked even with fake scratching due to the delay node.
In the very end I brought in the database for the tracks switching.
But then I realized that the Filestream node crashes if you switch the file.
It initializes correctly on startup with the file name but can change in runtime.
chrome chrashes the tab after slowly dying into freeze. duration showing NaN as visible on the screenshot.

screenshot 2015-05-19 20 51 09

is this the intended way of using it or did i miss a point?
Or is it realy bugged
Maybe call this.evaluate on fileswitching for the audio buffer or something like that?

@geloescht
Copy link
Contributor

Wow, I am quite impressed by what you are doing.
Crashes are never intended, I guess ;) Can you please post a minimal example as a v4p (or ideally upload the example to an URL)? It could very well be related to memory bugs in Chrome, as Web Audio is such a new feature.
One thing I noticed in your screenshot is you are using FileAudioBuffer on presumably long audio files. You should never use this node for audio that is longer than a few seconds, as it will decode the entire tile into memory. Currently, FileAudioBuffer is only useful for loading impulse responses for Convolver.

@tekcor
Copy link
Contributor Author

tekcor commented May 19, 2015

Hi thanks
I think I am the one who is impressed by the AudioAPI you implemented
I sneaked your email address from matthias and send you a mail with the file sources.
Probably the easyest way!

Thanks

By the way.
I just realized that it must be realy a terrible bug with chrome.
When you want to close the window when having patch editor open it askes if you realy want to close the windows. in that moment the filestream starts to play until you hit chancel. you can repeat this as much as you want!

also got rid of the fileaudiobuffers. In the first moment I thought that would be needed... it was not the problem however

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

2 participants