-
Notifications
You must be signed in to change notification settings - Fork 26
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
Equalizer/fft graph not shown on Android phones #93
Comments
Here's the section that's supposed to draw the graph! webjack/examples/fft/sketch.js Lines 40 to 46 in 04c0def
|
test soul brauser on android. |
aww darn, any way you could test with a javascript console? Thanks for
reporting!
…On Mon, Jul 19, 2021 at 3:39 AM Serega ***@***.***> wrote:
test soul brauser on android.
Chrome browser hasn't been very good lately
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#93 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAF6J4H77FVW53BB4UYSULTYPJENANCNFSM4HGN2Z7A>
.
|
https://editor.p5js.org/jywarren/sketches/rkztwSt8M
webkitAudioContext - is out of date https://developer.mozilla.org/en-US/docs/Web/API/Web_Audio_API/Migrating_from_webkitAudioContext |
Ah. Hm. I wonder if this is fixed in a more recent version of p5.js; we
seem to be at v0.8.0. I searched for the error in their repo but found
nothing but their current version is v1.4.0:
https://github.com/processing/p5.js/
We could try bumping our version...
…On Sat, Aug 14, 2021 at 8:30 AM Serega ***@***.***> wrote:
aww darn, any way you could test with a javascript console? Thanks for
reporting!
… <#m_8691155359467393051_>
On Mon, Jul 19, 2021 at 3:39 AM Serega *@*.***> wrote: test soul brauser
on android. Chrome browser hasn't been very good lately — You are receiving
this because you authored the thread. Reply to this email directly, view it
on GitHub <[#93 <#93>
(comment)]
https://editor.p5js.org/jywarren/sketches/rkztwSt8M
#ReferenceError: webkitAudioContext is not defined
at Class.init (https://webjack.io/dist/webjack.js:663:44)
at new Class (https://webjack.io/dist/webjack.js:63:19)
at (anonymous function) (https://jywarren.github.io/p5-webjack/lib.js:15:20)
at (anonymous function) (https://cdnjs.cloudflare.com/ajax/libs/p5.js/0.8.0/addons/p5.sound.min.js:24:4838)(https://github.com/publiclab/webjack/issues/93#issuecomment-882318312)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAF6J4H77FVW53BB4UYSULTYPJENANCNFSM4HGN2Z7A> .
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#93 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAF6J2WTVEZEBNLTEL27FDT4ZOXZANCNFSM4HGN2Z7A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email>
.
|
Wow, we're actually at v0.7.3 of p5js: Lines 16 to 18 in 5995944
Would you be able to try bumping the version in the CDN links and trying again? If not I can try. Noting also that the code referenced in your error log mentions this wrapper library I also wrote, but there's no reference to "webkit" in the code there so I think that's not the issue. But still, good to notice the included code: |
thanks so much for helping out here!!! |
i dont know how get js errors on android brausers whithout conect to PC I can only validate code in some JavaScript sandbox, editor.p5js.org or others |
|
oh whoa... cert invalid! This i think i can fix. I don't know if it'll fix
things on Android but it's a good first step to get out of the way.
…On Sun, Aug 22, 2021 at 11:26 AM Serega ***@***.***> wrote:
thanks so much for helping out here!!!
webjack.io-1629558757386.log
<https://github.com/publiclab/webjack/files/7027624/webjack.io-1629558757386.log>
[image: 1]
<https://user-images.githubusercontent.com/13694921/130360748-dab06750-07d6-47ff-a7cb-de18fb042ca5.gif>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#93 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAF6J7LI5G7S75THCATSMLT6EJIZANCNFSM4HGN2Z7A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email>
.
|
What version of Chrome are you using? |
I don't know if this is the problem on Android but it makes me wonder, what is your Android version and Android Chrome version too? chrome://settings/help can help you determine this. Would you be able to try out the other change as well, the update to p5js v1.4.0, to see if that helps? #202 I've tried the update here. Do you know how to start it up to test? You could use GitPod -- http://gitpod.io/#https://github.com/publiclab/webjack/pull/202 once it starts up, you'll see the example page in the preview window. |
The interface there is very slow and it is intended for a computer, the page does not move well on the phone and the navigation is bad I can not check the scripts |
#88, #87 and #92 fixed the audio security limits issue and installed an equalizer to display what sounds it can hear on the spectrum, for debugging. It's awesome!
However, on my Android phone in Chrome 73, I can't see the graph:
I can't reproduce this on desktop Chrome:
Not sure what's happening here!
The text was updated successfully, but these errors were encountered: