Waveform scratch too sensitive

Bug #790744 reported by jus on 2011-05-31
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Medium
shanxS

Bug Description

Tested with lates trunk

Currently, 100 pixel mouse movement translate into 1 second of music at default waveform zoom level which is way to sensitive. A competitor uses 500 pixel per second, while another uses 250 pixel. Personally i found 250 pixel works best for scratching with mouse/pen/trackpad.

jus (jus) wrote :

Would be interesting to hear if the current waveform scratch in trunk really usable for any of you guys. I found you can`t do any controlled scratches with a mouse/trackpad.

Bill Good (bkgood) wrote :

It's much better if you raise your latency to ~40ms, complete non-solution but you asked if it does work much better (effectively larger polling period for the mouse movement allows the scratch code to act 'smoother') :)

jus (jus) wrote :

Perhaps I have expressed myself vague but i found it to be unrelated to latency , the sound is pretty realistic at any setting for me.
You have to move the mouse really,really careful, e.g try to scratch on a bass kick and is should be clear what i mean.

RJ Skerry-Ryan (rryan) wrote :

I agree -- it's too sensitive.

Changed in mixxx:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → RJ Ryan (rryan)
milestone: none → 1.10.0
shanxS (shanx-shashank) wrote :

Perhaps I should have addressed this issue while I was working on mouse scratching.
Patch is against trunk #2855.

jus (jus) wrote :

Thanks shanxS,
tested your patch and found the mouse movement feels better now.

But as long we dont have zoomable waveforms, the DEFAULT_PIXELS_PER_SECOND should stay at 100, not 250 like in your patch.

shanxS (shanx-shashank) wrote :

Hi Jus
Thanks for review.
This patch decreases sensitivity without changing zoom.

If I am allowed, I'd suggest to merge zoom-able waveform feature into upcoming release. This will resolve issue of sensitivity and of course waveforms will be zoom-able.
The code for zoom-able waveform is ready already, I'll update it for latest version of trunk.

regards
shanxS

RJ Skerry-Ryan (rryan) wrote :

Added the patch to change the proportional constant of the scratch controller to 0.1 from 0.3. Thanks shanxS!

Changed in mixxx:
status: Confirmed → Fix Committed
assignee: RJ Ryan (rryan) → shanxS (shanx-shashank)
RJ Skerry-Ryan (rryan) on 2011-12-25
Changed in mixxx:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers