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

iOS + Tone.min.js(14.5.37) で new PingPongDelay() するとexceptionが発生する #1

Closed
cat2151 opened this issue May 30, 2021 · 3 comments

Comments

@cat2151
Copy link
Owner

cat2151 commented May 30, 2021

Describe the bug

バグの説明
A description of what the bug is.
そのバグが何であるかの説明。

iOS + Tone.min.js(14.5.37) で new PingPongDelay() するとexceptionが発生します。

To Reproduce

再現するために
Please include a way to reproduce your issue. If possible, please includes a link to some example code using a platform like jsfiddle or codesandbox where the code can be edited. This makes it much easier to debug the issue and also create a validation test to verify the bug was fixed.
あなたの問題を再現する方法を含めてください。可能であれば、コードを編集できるjsfiddleやcodesandboxのようなプラットフォームを使ったサンプルコードへのリンクを含めてください。これにより、問題のデバッグが非常に容易になり、またバグが修正されたことを確認するための検証テストを作成することができます。

iPadでこのコードを実行すると再現します。
https://cat2151.github.io/try-tonejs-ios/index.html
https://codepen.io/cat2151/project/editor/DPEjMG

Expected behavior

期待される動作
A description of what you expected to happen.
期待していたことの説明。

期待していたことは、iOSでこのコードを実行すると、ピンポンディレイの音が鳴ることです。https://cat2151.github.io/try-tonejs-ios/index.html
https://codepen.io/cat2151/project/editor/DPEjMG

What I've tried

私が試したこと
How have you tried resolving/debugging this issue? This can be helpful context for getting to the heart of the issue faster and not duplicating effort.
この問題をどのように解決/デバッグしましたか?この情報は、問題の核心に早く到達し、重複した作業を行わないための参考になります。

WindowsPC + Tone.js(npm install tone) で実行したとき再現しませんでした。Tone.jsのバージョンは14.7.77です。
iPad + Tone.js(npm install tone) で実行したとき再現しませんでした。Tone.jsのバージョンは14.7.77です。
https://cat2151.github.io/try-tonejs-ios-npm-webpack/dist/index.html

iPad + Tone.min.js で実行したとき再現しました。Tone.min.jsのバージョンは14.5.37です。
https://cat2151.github.io/try-tonejs-ios/index.html
https://codepen.io/cat2151/project/editor/DPEjMG

Additional context

追加コンテキスト
Add any other context about the problem here.
ここで問題に関するその他のコンテキストを追加します。

@cat2151
Copy link
Owner Author

cat2151 commented May 30, 2021

Tone.min.js(14.5.37)を使わず、
Tone.js(14.8.17)を使うことで解決しました。
https://qiita.com/cat2151/items/d6c2bf0dd3e5f441ad69

@cat2151
Copy link
Owner Author

cat2151 commented May 30, 2021

@cat2151
Copy link
Owner Author

cat2151 commented May 30, 2021

クローズします。

@cat2151 cat2151 closed this as completed May 30, 2021
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

1 participant