wspinny rapidly fast forwards track when clicked

Bug #879720 reported by Owen Williams
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
High
RJ Skerry-Ryan

Bug Description

When clicking the WSpinny widget to initiate a scratch, the playhead zooms forward along the track. This is due to a new bug:

m_dPrevTheta is calculated as a simple -180 to 180 value, but m_iFullRotations is set based on the absolute position in the track. When the widget is clicked, wspinny calculates the difference between the "old" position (-180 to 180) and the "new" position which could be many many full rotations down the way. So this causes a massive forward seek.

possible solution: initialize m_dPrevTheta to include full rotations

Revision history for this message
Owen Williams (ywwg) wrote :

there are also some instances where wspinny is not correcting for stereo audio, and these need to be checked for correctness

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Fixed -- this was because of my fix for Bug #779179. I didn't update WSpinny to set scratch_position to use relative samples instead of absolute.

Changed in mixxx:
status: New → Fix Committed
importance: Undecided → High
assignee: nobody → RJ Ryan (rryan)
milestone: none → 1.10.0
RJ Skerry-Ryan (rryan)
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/6035

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.