bpm_tap only works when no beatgrid is set

Bug #1441588 reported by jus
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Medium
Be

Bug Description

With latest master on OSX , i can`t get the ``bpm_tap`` control to work, if the track has a beatgrid detected.

The BPM of the track does not change when tapping. Tried to map a keyboard shortcut, assign MIDI, a pushbutton - nothing.
However, i can change the BPM using the tap button in the BPM tab of the Properties editor.

If you right-click > BPM Options > Clear BPM and Beatgrid , tapping works.

Controls are currently in LateNight (Decks) and Shade (Samplers).

Edit:
-----
Ok, now i see you can only tap beatgridded tracks if you have SYNC enabled. Bad for those playing by ear.

Is this intentional?

jus (jus)
description: updated
Revision history for this message
jus (jus) wrote :

Still seeing this in latest master, even with latest improvements from https://github.com/mixxxdj/mixxx/pull/450
Confirmed in lp:1530173.

bpm_tap CO does not work with the skins & midi unless SYNC is engaged.
Tapping from within the BPM tab in the track properties editor does work without having SYNC engaged though.

Changed in mixxx:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Be (be.ing) wrote :

Anyone want to take this up? Users ask about it on the forum and IRC fairly often.

Revision history for this message
Be (be.ing) wrote :

I think this is a duplicate of Bug 1523286. Is this still an issue in master?

Revision history for this message
jus (jus) wrote :

Still present in master as of "Mixxx" "2.1.0-alpha-pre" "(git master r6398; built on: Nov 30 2017)

You can bpm_tap on beadgrided tracks only if channelN is sync_enabled

Revision history for this message
Be (be.ing) wrote :

Thank you for testing. We should fix this for 2.1. Anyone want to volunteer to fix it?

Changed in mixxx:
milestone: none → 2.1.0
summary: - BPM tap control broken with tracks that have beatgrid detected
+ bpm_tap only works when master sync is on
Changed in mixxx:
assignee: nobody → Johan Lasperas (johan-lasperas)
Revision history for this message
Johan Lasperas (johan-lasperas) wrote : Re: bpm_tap only works when master sync is on

Made a pull request to fix this : https://github.com/mixxxdj/mixxx/pull/1404

Be (be.ing)
summary: - bpm_tap only works when master sync is on
+ bpm_tap only works when no beatgrid is set
Changed in mixxx:
assignee: Johan Lasperas (johan-lasperas) → Be (be.ing)
status: Confirmed → In Progress
Be (be.ing)
summary: - bpm_tap only works when no beatgrid is set
+ bpm_tap only works when no beatgrid is set and sync is disabled
summary: - bpm_tap only works when no beatgrid is set and sync is disabled
+ bpm_tap only works when no beatgrid is set
Revision history for this message
Be (be.ing) wrote :
Be (be.ing)
Changed in mixxx:
status: In Progress → Fix Committed
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/7951

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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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