Waveform scratch too sensitive

[jus]

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)]

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.

[William Good (bkgood)]

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)]

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)]

I agree -- it's too sensitive.

[shanxS (shanx-shashank)]

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

[jus (jus)]

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)]

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)]

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

[Swiftb0y (swiftb0y)]

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