.50 Zxx in S3M files...

Started by Saga Musix, February 08, 2008, 10:36:41

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...)
All versions

Description of the bug:
The original S3M format did not have Zxx effects, but you can actually use them in MPT. This may result in wrongly played modules which used extensions of old players like PixPlay which interpreted Zxx as panning.
Example Tune
I guess that (almost) nobody used Zxx in S3M tunes (as it really should not be possible!), so it's probably safer to remove this feature from the S3M playback engine.

Fixed in OpenMPT 1.17.02.53
» 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.

Sam_Zen

Zxx indeed is not in the list of S3M effects.
Listening to this example tune, I got the impression that the playback suffers a strong lowpassfilter.
0.618033988

Saga Musix

Of course it does since Zxx is a low-pass filter :P
Replacing those commands by S8x works remedies the situation.
» 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.

psishock

nah, in this case Zxx is a high-cut filter :lol:
I'm as calm as a synth without a player.  (Sam_Zen)

Sam_Zen

I'm afraid some readers will wonder here : - what's the difference ?
0.618033988

psishock

lol, it was a joke :lol:,  the 2 terms are completely same =)
letting only the low-frequency signals pass, or cutting of the high ones gives exactly the same resoult ^_^
I'm as calm as a synth without a player.  (Sam_Zen)

Saga Musix

it probably doesn't, depending on the quality of the filter. however, the filter MPT uses doesn't seem to be the same the original IT uses, so Modules with resonant filters sound different (mostly sharper) in IT.
» 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.

Sam_Zen

I think you're both right here. But psishock in a theoretical way, and Jojo in a practical way.
Electronic circuits, analog or digital, always will have some slope, or curve, between the pass and the cut area.
Btw: A resonant filter is not the same as a plain LPF or HPF. Then the 'Q-factor' also is a variable.
0.618033988

Saga Musix

As the Zxx macro editor is disabled anyway for Zxx files, I'm going to try to remove the Zxx standard macro (cutoff) for S3M files. I guess nobody has something against this..? :P
» 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

How about simply interpreting Zxx as S8x for S3M?

Saga Musix

I haven't looked at many S3M files or players, but there aren't many which use Zxx for anything indeed. but as this isn't a standard behavious it's maybe better to ignore it. I'm not really sure...
» 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.

Saga Musix

I thought of an option in the Module settings (there's enough empty space anyway): A dropdown list how to handle Zxx: Compatible (No interpretation), PixPlay (Emulate S8x), MPT (Cutoff/Resonance)
» 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"I thought of an option in the Module settings (there's enough empty space anyway): A dropdown list how to handle Zxx: Compatible (No interpretation), PixPlay (Emulate S8x), MPT (Cutoff/Resonance)
The list seems reasonable -- my previous idea of simply interpreting Zxx as S8x isn't good.

Saga Musix

I think this is worth discussing in the forums so we can hear other opinions as well...

If a Zxx effect is detected in a S3M module while being loaded, this message will appear:



I'll quickly c&p my mail to relabs concerning the action assigned to the "no" button:

QuoteI don't know if *removing* the effects is really that good. If the user clicks "no", they could be kept instead and the macro configuration could be deleted as I suggested it in the beginning. But I guess there are arguments for both sides:
- You can remove the Zxx effects from the patterns (which would contradict the way it's done at the moment)
- You can remove Macros by yourself (which would contradict my new suggestion)
» 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.

bvanoudtshoorn

Is it perhaps possible to relabel the buttons? That'd be a lot more intuitive, IMHO...