experimental branch - ofxSingleSoundPlayer - load via getBuffer()

[stephanschulz]

I am trying to load each player by using a file buffer file_0.getBuffer() instead of load(dir.getPath(i)), but app crashes right away.

 ofxSoundFile file_0;
 file_0.load(dir.getPath(0));
        
 if(players[startIndex + i]->load( file_0.getBuffer(), ofToString(i))){

}

[roymacdonald]

Yes. I think that this generates a data race which makes it crash.

[stephanschulz]

this used to work with the main branch.

[roymacdonald]

sure. I will double check what is going on.

[stephanschulz]

here my project that has this crash
https://www.dropbox.com/s/dg02lg2hyx7vktt/example-matrixMixer_exp_crash.zip?dl=0

[roymacdonald]

@stephanschulz I fixed it. please test it and let me know.
I was not able to download the project you linked. Seems like the file got corrupt or something

[stephanschulz]

Thanks so much for fixing ofxSingleSoundPlayer.
Here are my finding.

What is working:

• load from fileBuffer
• load from file
• setLoop(false) + playerEnded(size_t & id)
• setSpeed() during loading, playerEnded() and while playing

What has issues:

• still have to use ->setId(startIndex + i); during loading in order for playerEnded(size_t & id) to produce the correct index
• loading 300 or more files with setSpeed(ofRandom(0.5,2))+setLoop(true) produces loud pops. here the code to reproduce this issue https://gist.github.com/stephanschulz/10a8de28c6c27f43a731f2a847f4dab2 Without setSpeed this does not happen.

[roymacdonald]

Hi @stephanschulz
About the issues you mention.
each instance of ofxSimpleSoundPlayer gets a unique id automatically. (it is a static member variable that is used to set the id and incremented on each constructor call). If you want the id to reflect the index you should use setId.

When you set the speed calling setSpeed(...) with a single argument, the buffer is processed on real time on each audio call back. which consume resources but mostly, those loud pops come from a bug in the ofSoundBuffer resampleTo(,,,) method. Sometimes it skips the last samples which ends up producing these pops. I am working on implementing libsamplerate with gives better results. Regardless of this, you should pass true as the second argument of setSpeed(...) as it will preprocess and resample the whole sound file data which will give you a clean playback and reduced resources usage, as there is no processing done on the audio callback. Let me know how it goes