ModPlug Central

OpenMPT Development (Archive) => Bug Reports => Bug Report Archive => Topic started by: Sam_Zen on June 21, 2010, 00:31:00

Title: 1.18.02.00 Wrong pan value after conversion
Post by: Sam_Zen on June 21, 2010, 00:31:00
I wanted to convert a 4 channel MOD to XM.
The default channel panning of a MOD is set to L - R - R - L.
With earlier versions of OMPT this got lost, so all channels were set to mono.
Now, the conversion works correct, so the Initial Pan values become 64 for L and 192 for R.

But there's a bug. Channel 1 is set to 128 (centered) instead of 64.
And with XM, the Initial Pan slider is greyed out, so it can't be corrected.
It's only channel 1. With a 12 CH MOD, channels 5 and 9 do have the correct 64.

Conversion of MOD to IT works correctly, and of course there the sliders are available for a change.
Title: 1.18.02.00 Wrong pan value after conversion
Post by: Saga Musix on June 21, 2010, 07:06:17
Thanks for pointing this out - it revealed that channel settings were neither set correctly for MOD, XM nor S3M when converting, due to a an error in the code which I didn't even expect to compile, but apparently it did.
Title: 1.18.02.00 Wrong pan value after conversion
Post by: Saga Musix on August 04, 2010, 23:36:38
Please confirm that this has been fixed in OpenMPT 1.18.03.00 (http://forum.openmpt.org/index.php?topic=4032.0). You may set the thread status to S=C then.
Title: 1.18.02.00 Wrong pan value after conversion
Post by: Sam_Zen on August 06, 2010, 12:03:15
(sorry for the delay)

Can't confirm this.
Now all channels are set to a value of 128
Title: 1.18.02.00 Wrong pan value after conversion
Post by: Saga Musix on August 06, 2010, 12:11:49
well, of course. xm doesn't have channel panning. xm only has sample panning, so all channels are just defaulted back to center.
Title: 1.18.02.00 Wrong pan value after conversion
Post by: Sam_Zen on August 07, 2010, 20:02:07
This would mean that my original prepatch bug should be read the other way :
Channel 1 is behaving correctly with 128 (centered), while channels 2 - 4 do it the wrong way, transferring 64 and 192
from the default MOD values.
Title: 1.18.02.00 Wrong pan value after conversion
Post by: Saga Musix on August 07, 2010, 20:02:46
Quote from: "Sam_Zen"This would mean that my original prepatch bug should be read the other way :
Channel 1 is behaving correctly with 128 (centered), while channels 2 - 4 do it the wrong way, transferring 64 and 192
from the default MOD values.
I know. And exactly that has been fixed?
Title: 1.18.02.00 Wrong pan value after conversion
Post by: Sam_Zen on August 10, 2010, 12:47:55
As such, yes.