Author Topic: 1.17.02.53 MOD conversion vs FX commands  (Read 3448 times)

Offline Harbinger

  • Extreme artist
  • *****
  • Posts: 1,141
  • Gender: Male
  • Operating System: Windows XP
1.17.02.53 MOD conversion vs FX commands
« on: September 22, 2009, 19:16:00 »
Full Version:
OpenMPT v1.17.2.53

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 sure.


Description of the bug: Found this while converting an IT file to MOD to test volume commands. Was warned about converting "down" to MOD format in dialog, but MPT changed IT's Vxx command to XM's Gxx command (both are Global Volume settings), but Gxx is not an official MOD command. It did set the Global Volume however. When saving as MPT MOD (did not test Compatible Export), the Gxx commands disappeared.
IMO either file conversion should not apply unequivalent commands, or the conversion should not be allowed (and perhaps explained why not in a dialog window).

How often does it happen?: Always.

How to reproduce (step by step description): Open an IT track (it may work with other formats). Make sure there are Global Volume commands; for IT, this would be Vxx. Go to Song Properties and change the format to MOD. A dialog will open reminding you that you lose some information -- fair enough. After the conversion, the Vxx commands will be changed to Gxx commands, which shouldn't be allowed in MOD format. It works, but is not saved.

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 6,641
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
1.17.02.53 MOD conversion vs FX commands
« Reply #1 on: September 22, 2009, 19:59:32 »
The converter does not remove most of the unusable formats because this gives the author a chance to fix them (although admittedly, he has to know what works and what doesn't with the new format). The mod conversion will be much better in the next version, but if anyone else finds this behavior annoying / irritating / whatever, please tell me and I will consider removing invalid commands.
» 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.

Offline LPChip

  • Administrator
  • *****
  • Posts: 5,044
    • http://lpchip.nl
  • Operating System: Windows 10 Pro x64
1.17.02.53 MOD conversion vs FX commands
« Reply #2 on: September 22, 2009, 20:02:17 »
Although I will not use this functionality, I plead for removing it. It works more confusing rather than good, and if you want to keep it, at least change the warning box to explain that it is not right.

I would've blindly trusted OpenMPT to make a good conversion, then close OpenMPT to find out that my song is now screwed up.
"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

Offline g

  • Workaholic artist
  • ***
  • Posts: 280
1.17.02.53 MOD conversion vs FX commands
« Reply #3 on: September 23, 2009, 16:48:58 »
I'm going to be a PITA and suggest to make it optional. I.e. "remove invalid commands? Y/N".

Offline bvanoudtshoorn

  • Extreme artist
  • *****
  • Posts: 1,053
  • Gender: Male
    • Barryvan
1.17.02.53 MOD conversion vs FX commands
« Reply #4 on: September 23, 2009, 23:01:33 »
Maybe it's worth highlighting dodgy commands with a red background or something to make them obvious to the user.

Offline Harbinger

  • Extreme artist
  • *****
  • Posts: 1,141
  • Gender: Male
  • Operating System: Windows XP
1.17.02.53 MOD conversion vs FX commands
« Reply #5 on: October 01, 2009, 18:25:36 »
I would opt for a warning dialog showing what commands will be altered/deleted, with the chance to back out of the conversion one last chance.

Offline Saga Musix

  • OpenMPT Developers
  • *****
  • Posts: 6,641
  • aka Jojo
    • Download music, samples, VST plugins: Saga Musix Website
  • Operating System: Windows 10 x64
1.17.02.53 MOD conversion vs FX commands
« Reply #6 on: January 25, 2010, 22:25:26 »
Please confirm that this has been fixed in OpenMPT 1.18 release candidate. You can set the thread status to S=C yourself then.
» 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.