View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0001386||OpenMPT||File Format Support||public||2020-11-10 05:49||2020-11-10 18:20|
|Reporter||Lachesis||Assigned To||Saga Musix|
|Product Version||OpenMPT 1.29.05.00 / libopenmpt 0.5.3 (upgrade first)|
|Target Version||OpenMPT 1.29.06.00 / libopenmpt 0.5.4 (upgrade first)||Fixed in Version||OpenMPT 1.29.06.00 / libopenmpt 0.5.4 (upgrade first)|
|Summary||0001386: OctaMED 8-channel mode modules may rely on octave wrapping.|
From my testing the 8-channel mode part of this appears to be incorrect. Multiple 8-channel mode modules I have tested in OctaMED 4 rely on both types of octave wrapping for sample instruments (4-7 → 3; 8-A → -1-1), and in the 8-channel modules I've tested, octave wrapping works exactly the same way as it does in 4-channel mode.
The module mentioned in the comment (dissociate.mmd0) appears to use notes in octave 4, but from what I can tell these are actually in octave 3 due to instrument transpose (order 21).
|Steps To Reproduce|
Compare any of these modules in OctaMED and OpenMPT:
I've intentionally been avoiding modules that use IFFOCT and synth/hybrid instruments in my testing so I can't speak for how they behave in cases like these; anything I've claimed here applies to sample instruments only. (I found this bug while putting together a list of test modules for my MikMod transpose and octave wrapping patch.)
|Tags||No tags attached.|
|Has the bug occurred in previous versions?|
|Tested code revision (in case you know it)|
Thanks. This is now changed in r13816. I guess I reached this incomplete conclusion when transposition was not implemented yet in the new loader.
Now the modules should sound fine, although the tempo is slightly off in "you got to let the music". MED Soundstudio imports it as 116 BPM, at which the samples loop properly. Both OpenMPT and xmp play it at a 113.6 BPM, which causes some breaks in the loops. Gotta find out where that discrepancy comes from...
|2020-11-10 05:49||Lachesis||New Issue|
|2020-11-10 10:23||Saga Musix||Assigned To||=> Saga Musix|
|2020-11-10 10:23||Saga Musix||Status||new => assigned|
|2020-11-10 17:55||Saga Musix||Note Added: 0004506|
|2020-11-10 18:20||Saga Musix||Status||assigned => resolved|
|2020-11-10 18:20||Saga Musix||Resolution||open => fixed|
|2020-11-10 18:20||Saga Musix||Fixed in Version||=> OpenMPT 1.29.06.00 / libopenmpt 0.5.4 (upgrade first)|
|2020-11-10 18:20||Saga Musix||Target Version||=> OpenMPT 1.29.06.00 / libopenmpt 0.5.4 (upgrade first)|