ModPlug Central

OpenMPT Development (Archive) => Bug Reports => Bug Report Archive => Topic started by: Saga Musix on March 14, 2007, 17:14:05

Title: .46 bug in channel volume
Post by: Saga Musix on March 14, 2007, 17:14:05
Full Version:
OpenMPT v1.17.03.00

Description of the bug:
In v1.17.02.45 (and previous versions), modplug had another manner to handle channel volumes.
if you lower the channel volume (let's say you use the M10 effect -> Channel volume is 10h) and you skipped to a pattern BEFORE the volume change, you heared the channel in its original volume. after a short listen in version 1.17.3, I noticed anohter beviour: when i skipped back, the NEW volume was kept! very bad...

and by the way, if you double-click on the effect volume, you see the effects list with values. the Mxx effect allows values from 0 to 64 (40h), but the whole range  from 0 to 255 (FFh) is available in this window. this is similar for other effects..
Title: .46 bug in channel volume
Post by: LPChip on March 14, 2007, 21:46:44
I've had a chat with Relabsoluness, and we've decided to not persue the bugs for now.

The 3.0a build is just a build to play with and there are bugs to be expected.

A newer version will come and have most (if not all) features from the 3.0a build. If that happens, we'll see if the bug still exists and reopen the bug.

Until then, this topic will be locked.
Title: .46 bug in channel volume
Post by: Relabsoluness on March 15, 2007, 20:50:40
I'm unlocking this topic for I would like to ask about the bug - especially as it's descriped very bad :). It may well be that some volume handling have changed, but I don't quite understand the bug description; maybe an example file could help.
Title: .46 bug in channel volume
Post by: Saga Musix on March 15, 2007, 21:36:08
I am not sure what went wrong here, but it seems not to happen again :)
oh no, wait, it happened again in another module!

Download volume.it
load the file, enable playback.
skip to the third pattern.
now skip to the second pattern. it should be playing at original volume again, but it's playing at M10.
This behavious was "normal" for "repeat pattern" mode before, but i think that it is rather irritating and confusing.
Title: .46 bug in channel volume
Post by: Relabsoluness on March 16, 2007, 21:37:47
Ok, bug reproduced.
Title: .46 bug in channel volume
Post by: Saga Musix on March 17, 2007, 13:42:57
Okay, but be sure to check the other effects too ;) It also happens with Global volume, so it maybe happens with every effect.
Title: .46 bug in channel volume
Post by: Saga Musix on May 07, 2007, 18:36:52
okay, now it even happens in the .46 release. :(
Title: .46 bug in channel volume
Post by: Relabsoluness on May 08, 2007, 21:44:06
For me, at least the test file sounds as intended.
Title: .46 bug in channel volume
Post by: Saga Musix on May 09, 2007, 13:18:32
If you skipped around in MPT, the tracker "knew" which volume this pattern should be played at (previous volume commands were handled)
I uploaded another example at (link removed)
Skip around the patterns and you will notice that the channel / global volumes are not as they should be.
Title: .46 bug in channel volume
Post by: rewbs on May 09, 2007, 23:42:54
I think the description of this bug makes it sound a lot more complicated than it actually is.

QuoteIn v1.17.02.45 [...] if you lower the channel volume (let's say you use the M10 effect -> Channel volume is 10h) and you skipped to a pattern BEFORE the volume change, you heared the channel in its original volume.
As far as I can see that is not necessarily true. There has been a change in behaviour but it has nothing to do with skipping backwards vs forwards:

- Before .46, jumping to another pattern by clicking in the order list maintained the active channel and global volumes.
- In .46, doing so maintains the active global volume but resets the channel volume to the max.


As stated in the release notes:

 / <Relabsoluness> Channel states resetting when jumping between orders from orderlist.
     --> NOTE: While this fixes a bug (behavior change) introduced in the tuning branch(?), it also changes quite prominent behavior so can annoy people - might need to be modified at some point.


I'm amongst those who don't like the new behavior. My problem with the new behaviour is the inconsistency:
A. Global volume and channel volume behave differently.
B. If I navigate to a pattern during playback using +/- (not using the orderlist) and press "play pattern from start", the states are reset. However, if I press "play song from current row", the states are not reset. This is confusing imho.
Title: .46 bug in channel volume
Post by: Relabsoluness on May 10, 2007, 00:03:48
Quote from: "rewbs"I'm amongst those who don't like the new behavior. My problem with the new behaviour is the inconsistency:
A. Global volume and channel volume behave differently.
B. If I navigate to a pattern during playback using +/- (not using the orderlist) and press "play pattern from start", the states are reset. However, if I press "play song from current row", the states are not reset. This is confusing imho.
Ok, this is to be changed back (certainly wasn't most clever modification anyway).
Title: .46 bug in channel volume
Post by: Saga Musix on May 10, 2007, 17:02:30
good to know that you second my opinion :) if you want to skip fastly through your song (e.g. to check some special parts), this maybe ruins playback.
Title: .46 bug in channel volume
Post by: Relabsoluness on May 10, 2007, 23:56:01
Quote from: "Jojo"if you want to skip fastly through your song
Well actually the original reason why I even touched the skipping behavior was that I would have liked the thing to work faster :) I found it annoying that, at least with certain mods, the delay when skipping patterns is rather large so I removed a single function call whose purpose I didn't understand :oops:
Title: .46 bug in channel volume
Post by: LPChip on May 11, 2007, 07:55:31
Quote from: "Relabsoluness"
Quote from: "Jojo"if you want to skip fastly through your song
Well actually the original reason why I even touched the skipping behavior was that I would have liked the thing to work faster :) I found it annoying that, at least with certain mods, the delay when skipping patterns is rather large so I removed a single function call whose purpose I didn't understand :oops:

Perhaps its a good idea to make it optional, and accessable through a toolbar icon?
Title: .46 bug in channel volume
Post by: Saga Musix on May 11, 2007, 15:12:03
yes, that's a good idea. but making everything optional will also make the settings list grow and grow... at some point, it has to be split :D coz i dunno if this icon is located well in the toolbar...
Title: .46 bug in channel volume
Post by: Relabsoluness on May 27, 2007, 21:26:06
Can someone confirm the bug been fixed in >= .47?
Title: .46 bug in channel volume
Post by: Saga Musix on May 27, 2007, 21:27:34
it works, thanks ;)