Conversation
|
@haroon10725 can you just tell me like what the error with the lib/tone.js |
|
This will need some testing. I did a few tests, such as:
So far, it passes all of these tests, but I think we should spend some more time testing this before we merge it. We should also document a list of things to test every time we update tone.js. |
|
@pikurasa Sure. |
Updated the library |
Pretty much everything in MB that makes sounds uses tone.js. I think what we should look for are any errors that may come up with different ways we use tone.js, as well as whether or not there is a difference in overall performance. |
|
Sampler should be tested too. |
|
@walterbender @pikurasa These warnings have also been removed by updating Tone.js. There were some depreciated methods. |
|
This PR links to issue #3555. |

@walterbender Can you please review this.
Updated Tone.js v14.7.40 to Tone.js v15.0.4