Mixxx

BPM not displayed in numerical order

Bug #523774 reported by Korin on 2010-02-18

This bug affects 3 people

Affects		Status	Importance	Assigned to	Milestone
	Mixxx	Fix Released	Low	RJ Skerry-Ryan	Mixxx 1.8.0

Bug Description

BPMs are displayed (when set highest-to-lowest) from 99.99.. down to the lowest BPM, then the highest BPM all the way down to 100.0, instead of entirely from highest-to-lowest.

Additionally, I'm sure some of these BPMs should be half/double what they are detected as, but that's another issue (possibly already nearly resolved?)

Attached are some examples.

Tags:

Related branches

lp://staging/~mixxxdevelopers/mixxx/trunk

Revision history for this message

Korin (4chons) wrote on 2010-02-18:

mixxxBPM2.png Edit (123.1 KiB, image/png)

Revision history for this message

Korin (4chons) wrote on 2010-02-18:

mixxxBPM1.jpg Edit (122.3 KiB, image/jpeg)

Sorted by BPM, it starts at the 99.9 mark.

Revision history for this message

Sean M. Pappalardo (pegasus-renegadetech) wrote on 2010-02-18:

Sounds like it's looking at the BPM field as a string instead of a (float) number.

Revision history for this message

Smashuu (smashuu) wrote on 2010-02-19:

That's what's happening, I'm getting it too. The 100s come first, then the 70s, 80s, 90s.

RJ Skerry-Ryan (rryan) on 2010-02-19

Changed in mixxx:
status:	New → Confirmed
milestone:	none → 1.8.0

Revision history for this message

RJ Skerry-Ryan (rryan) wrote on 2010-02-20:

Should be fixed in trunk now -- thanks for reporting this.

Changed in mixxx:
importance:	Undecided → Low
assignee:	nobody → RJ Ryan (rryan)
status:	Confirmed → Fix Committed

ironstorm (ironstorm-gmail) on 2010-02-25

tags:

added: bpm library

Revision history for this message

Rafael Winter (rafael-winter) wrote on 2010-06-13:

mixxxBPM3.png Edit (107.6 KiB, image/png)

I am using Mixxx 1.8.0~beta1 and this issue still happens. There is something worth commenting, in my system (Windows Vista, locale PT-BR) the BPM column uses comma instead of dot, i.e 72,465 instead of 72.465 and this may mess the column sorting. I have installed the 1.8.0 beta1 over the 1.7.0 and the library was migrated automatically.

Please take a look at my screenshot: mixxxBPM3.png. If you need more information, please let me know.

Revision history for this message

RJ Skerry-Ryan (rryan) wrote on 2010-06-18:

Hi Rafael,

The bug has been fixed in our trunk, which is where we do development on the latest version of Mixxx. Mixxx 1.8.0 beta1 has this issue, but beta2 which is due in the next week will not have the issue.

Thanks,
RJ Ryan

RJ Skerry-Ryan (rryan) on 2010-10-05