waveform and transport out of sync for mp3

Bug #1407394 reported by Daniel Schürmann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
High
Unassigned

Bug Description

Waveform and transport are out of sync.

My test track has according to Audacity 0,4 s silence at the beginning.

This is displayed in Mixxx correctly, the waveform starts, when the track time is on 0,40.

But the sound output starts actually starts at 0,36 and the first beat marker of the non const beat grid is at 0,36

Tags: soundsource
Changed in mixxx:
milestone: none → 1.12.0
importance: Undecided → High
Revision history for this message
Daniel Schürmann (daschuer) wrote :

The original issue appears using a mp3 file. If I save the same file as wave using Audacity, it sound output aligns perfectly with the waveform.
This was tested with 64 bit Linux build of 1.12.0-alpha (build master r5041) on Ubuntu Trusty.
the original track is an an mp3float.

I have also recoded the file in 192 kbps variable and fixed rate.
Now the Audio starts at 0:45 according to the waveforms im Mixxx and according to Audacity, but the sound starts later at 0.47 for both tracks in Mixxx.

Is that regression?

summary: - waveform and transport out of sync
+ waveform and transport out of sync for mp3
Revision history for this message
Daniel Schürmann (daschuer) wrote :

> Is that regression?
Tested with 1.11 release and it suffers the same issue.

Revision history for this message
Daniel Schürmann (daschuer) wrote :
Changed in mixxx:
status: New → Confirmed
assignee: nobody → Daniel Schürmann (daschuer)
status: Confirmed → In Progress
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Have you discovered the root cause? What's your planned fix?

Revision history for this message
Daniel Schürmann (daschuer) wrote :

https://github.com/mixxxdj/mixxx/pull/461

On cause are simple bugs in seek calculations.
An other cause are mad_frame_decode calls that to not advance to the next frame.

Revision history for this message
Daniel Schürmann (daschuer) wrote :

The fix is now in Master. Not in 1.12 alpha :-(

Changed in mixxx:
milestone: 1.12.0 → 2.1
status: In Progress → Fix Committed
tags: added: soundsource
Revision history for this message
Be (be.ing) wrote :
Changed in mixxx:
assignee: Daniel Schürmann (daschuer) → Uwe Klotz (uklotzde)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7794

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.