.50 Instrument panning not saved correctly (reopened)

Started by Saga Musix, March 08, 2008, 19:55:20

Previous topic - Next topic

Saga Musix

Full Version:
OpenMPT v1.17.02.50

Has the bug occured in previous versions? If yes, please specify version(s): (This option is optional, and doesn't need research. But in case you know...)
not tested

Description of the bug:
Instrument panning is not saved correctly. If you choose 256 as a paning value, it will be reset to 0. So the maximum value is 252, because panning is saved in steps of 4 units.

How often does it happen?:
tested in IT format.

How to reproduce (step by step description):
try 255 and 256 ans an instrument panning position.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Winter

Hello there,

this problem is annoying me! I always get this problem! Everytime I save the panning for each channel and reload the IT-file again, all! pannings are reset!
The bug appears only in large IT-files or detailed and complex projects.
If you open a new file, set a panning, save, reload, then everything is ok!

:D Sometimes it's a game of luck. I always set the panning befor saving to do not abandon hope, because sometimes it happens that the panning is saved correctly. No idea why or why not!

This is really a strange bug! And you need good anit-bug spray!

my MP Version: 1.17.02.48
Channels: 17!
Patterns: 105 different Patterns



Oh! Now I see that the pannings of channel 15 and 16 are safed correctly, but not the channels 5-8 (pans: 32,64,192 and 224)

psishock

temp solution, to keep you from further head-aches on saving/loading:
set panning positions in pattern data (either with Pxx or Xxx commands), not in "instruments tab" or "general tab".
Those codes will not dissapear for sure.
I'm as calm as a synth without a player.  (Sam_Zen)

bvanoudtshoorn

Winter, please also upgrade to the latest version of OpenMPT - you might find that the bug is fixed (or at least behaves differently ;)) on a newer version.

You should monitor this thread for info about new versions. As I write this, OpenMPT is up to 0.52. You can download it here - download "mptrack.exe" to wherever you'd like.

Saga Musix

oh right, seems like it has been fixed and i haven't noticed it. :) bug closed.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Relabsoluness

Quote from: "Jojo"oh right, seems like it has been fixed and i haven't noticed it. :) bug closed.
Fixed? I haven't noticed either :) Winter's description of 'really strange bug' doesn't sound promising, though. If someone has a file with which the bug can be reproduced, it could be valuable for debugging purposes.

Winter

bvanoudtshoorn, I was wondering why you all are talking about .50 and now .52, because I could not find it anywhere in the download-section.
Thanks for the link!

But I am sorry!  This bug isn't fixed.
I made the test with the new exe. I panned, saved, loaded, wrong panning! :(

I can not give you my .IT-file, because the track will be released on a netlabel. You know... exclusive rights and so on. :D

LPChip

Quote from: "Winter"bvanoudtshoorn, I was wondering why you all are talking about .50 and now .52, because I could not find it anywhere in the download-section.
Thanks for the link!

But I am sorry!  This bug isn't fixed.
I made the test with the new exe. I panned, saved, loaded, wrong panning! :(

I can not give you my .IT-file, because the track will be released on a netlabel. You know... exclusive rights and so on. :D

But I'm sure you can make a new track, setup a test case that has a similar thing (just fill many patterns with garbage if you have to) and send that one?
"Heh, maybe I should've joined the compo only because it would've meant I wouldn't have had to worry about a damn EQ or compressor for a change. " - Atlantis
"yes.. I think in this case it was wishful thinking: MPT is makng my life hard so it must be wrong" - Rewbs

Saga Musix

Quote from: "Relabsoluness"
Quote from: "Jojo"oh right, seems like it has been fixed and i haven't noticed it. :) bug closed.
Fixed? I haven't noticed either :) Winter's description of 'really strange bug' doesn't sound promising, though. If someone has a file with which the bug can be reproduced, it could be valuable for debugging purposes.
i don't recall the complete situation, but i i just tried to save a module with those instrument pannings and it worked...
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Winter

Okay here is a contaminated .IT-file!  :twisted:

Good luck!

Click here!

Relabsoluness

Based on the testcase and other examinations, it seems that the reason is in the 'surround'-setting. If channels surround setting is enabled, pan value for that channel will not be saved to file - it just saves the information that the channel surround is enabled. And when loaded, pan is set to 128. This behavior has been there since mpt 1.16.

2Winter
Thanks for testcase. Can you confirm that surround-setting is the cause of the problem?

Saga Musix

this behaviour seems to be logical because surround disables stereo position, but this is not a reason not to save the stereo position imo.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

Winter

Ok, let's see...

Indeeed!

Thanks, Relabsoluness!
Now that's the trigger!

Although it's a pity, because I want to use stereo-effects and surround!
Even after a restart! ;-)

Saga Musix

winter: it is just not possible because of the way surround works. trackers use "fake surround" by reversing one of the stereo channels. this does not work correctly with panned samples.
» No support, bug reports, feature requests via private messages - they will not be answered. Use the forums and the issue tracker so that everyone can benefit from your post.

älskling

Quote from: "Jojo"winter: it is just not possible because of the way surround works. trackers use "fake surround" by reversing one of the stereo channels. this does not work correctly with panned samples.

It should work with 4 channel output instead of prologic.