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

Waveform rending bug in safari - analysis rod1 #11

Open
rconstanzo opened this issue Dec 14, 2015 · 7 comments
Open

Waveform rending bug in safari - analysis rod1 #11

rconstanzo opened this issue Dec 14, 2015 · 7 comments
Labels

Comments

@rconstanzo
Copy link
Collaborator

For some reason this analysis:
http://rodrigoconstanzo.com/analyses/?q=rod1

Only renders the first 40seconds of the image in Safari 9.02, whereas Firefox 39 renders the whole thing correctly.

I've emptied cache, and tried it on multiple computers, and still nothing.

@rconstanzo rconstanzo added the bug label Dec 14, 2015
@madwort
Copy link
Owner

madwort commented Dec 14, 2015

Verified - same issue on Safari 7.1.5 here ( works on FF ). Will investigate. Is this the longest track / largest mp3 file so far?

On 14 Dec 02015, at 10:35, Rodrigo Constanzo [email protected] wrote:

For some reason this analysis:
http://rodrigoconstanzo.com/analyses/?q=rod1

Only renders the first 40seconds of the image, whereas Firefox renders the whole thing correctly.

I've emptied cache, and tried it on multiple computers, and still nothing.


Reply to this email directly or view it on GitHub.

@rconstanzo
Copy link
Collaborator Author

Oddly, no.
This one renders fine, and it's over twice as long: http://www.rodrigoconstanzo.com/analyses/?q=feldman

If this is a rough fix it's not high priority (as in before thesis stuff) as it only seems to effect this particular analysis.

I also tested to make sure the audio plays fine so it's not the mp3 that's corrupted or something. (I've also reuploaded the mp3 a couple times, though I haven't reencoded it).

@rconstanzo
Copy link
Collaborator Author

This one DOES have the most analysis points, but I don't think that would effect the waveform rendering.

@madwort
Copy link
Owner

madwort commented Dec 14, 2015

Weird. You could try re-encoding it with slightly different mp3 parameters & see if you can bypass the bug (in case there's something weird in the mp3 that Safari's decoder is barfing on), or try a different format - I quite like .m4a but not sure if Firefox will decode it, .ogg is obviously good but Safari won't decode that...
We're using the most recent version of the waveform lib (hasn't been updated since I last did work on this) https://github.com/bbcrd/waveform-data.js , this bug looks maybe similar-ish but not really (and is a browser bug... bbc/waveform-data.js#32 )

@rconstanzo
Copy link
Collaborator Author

rconstanzo commented Dec 14, 2015

Ok, tested a bunch of different stuff.

Transcoded 128kb mp3 didn't work.
Transcoded wav file WORKED! (but took forever to render it)
Transcoded 320kb mp3 (from wav file) didn't work.

Versions that didn't work showed the same 46seconds only, which is weird. That's 7%, which doesn't appear to be a significant number, computationally.

@rconstanzo
Copy link
Collaborator Author

Also tested encoding an mp3 with iTunes instead of audacity and still nothing.

@madwort
Copy link
Owner

madwort commented Jul 20, 2019

just checking the changelog for waveform-data.js, there is a bugfix for Safari https://github.com/bbc/waveform-data.js/blob/master/CHANGELOG.md#v210-20181110 , not sure whether this will fix the issue we're seeing. Unfortunately, we're on v1.5 and the bugfix is in v2.1, and there's a little bit of re-jigging to get from v1 to v2 of that lib...

@madwort madwort changed the title Waveform rending bug in safari Waveform rending bug in safari - analysis rod1 Jul 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants