mpogue2 / SquareDesk

Fully-featured music player and sequence designer, designed for square dance callers
10 stars 4 forks source link

Double check that these are fixed #782

Closed mpogue2 closed 1 year ago

mpogue2 commented 1 year ago

Bugs reported against 0.9.5 by Gero, that should be checked:

image

mpogue2 commented 1 year ago

Testing "tiny break in the music":

With the current top-of-tree version (f8297b599b3eb34fc0a296c841ec9ce9993232d8), I jumped back from a 200Hz section (precisely aligned to the sample) to a 100Hz section (precisely aligned to the sample), and the splice at loop time was sample-accurate. The song was WAV encoded to avoid MP3 artifacts. Here's a picture:

image

Here is the same test, done with the WAV file converted to MP3. The join is seamless and sample accurate as well:

image
mpogue2 commented 1 year ago

Testing "letters of active tab disappear":

Here's what I see with current top-of-tree:

image

This was a long-standing Qt bug, and since we've upgraded to later versions of Qt since the 0.9.5 release, this bug has been fixed.

mpogue2 commented 1 year ago

Regarding "overall time counter/stopwatch...":

That feature was implemented a while ago, although it might not be documented well. Once the music is started, the big timer numbers (see picture below) reflect "Time in tip", and they continue to count, even though the music may be paused. If the music is stopped entirely (by pressing the STOP button, or pressing the PERIOD key), the Time-In-Tip will go back to zero automatically.

Screenshot 2023-03-23 at 12 58 42 PM
Gero5 commented 1 year ago

Regarding "overall time counter/stopwatch...":

For that "patter timer" has to be active in the preferences. From the way it is described there "patter timer after x minutes of patter, visual indicator" I was expecting just a visual indicator only not a time counter. Now I know. :-) It is quite useful for most callers, I guess. Maybe you want to activate it by default?

mpogue2 commented 1 year ago

I think it would indeed be good to make "Patter Timer ON" the default. The pull down menu is supposed to refer to what happens after the patter timer expires: either a) an audible indicator (which I use), or b) a visual indicator only (no audible sound is played).

Gero5 commented 1 year ago

I tested this with v1.0.4