Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

XMI - tempo change #108

Closed
psi29a opened this issue Jan 2, 2015 · 2 comments
Closed

XMI - tempo change #108

psi29a opened this issue Jan 2, 2015 · 2 comments

Comments

@psi29a
Copy link
Member

psi29a commented Jan 2, 2015

@chrisisonwildcode said "Noticed that xmidi.c only uses the first tempo change and ignores all others. Issue?"

Do we have example VMI files that change tempo?

Could this have anything to do with this issue: #107
^-- Relates to converted types (mt32-to-GM/GS) "This is only suitable for music that doesn't do timbre changes XMIDIs that contain Timbre changes will not convert properly. "

@psi29a psi29a added the bug label Jan 2, 2015
@psi29a psi29a added this to the 0.4.1 - Crank it up to 11 milestone Jan 2, 2015
@chrisisonwildcode
Copy link
Contributor

The "Timbre" is throwing me off ... I think what they are referring to is that the xmi files have patch settings outside of the MIDI data (although in my tests the MIDI data also contained patch settings) ... so I speculate that its something to do with those, however because all the XMI's I tested contained patch information in the MIDI data, I ignored the patch settings outside of the MIDI data. If it later becomes an issue I can fix.

If we ever do find a xmi with tempo change, the player and associated dump (pressing 'm' while playing) will honour the tempo change, however the converter in xmidi.c which is used when you convert without playing will not.

@chrisisonwildcode
Copy link
Contributor

Based on recorded in game audio and the finding of an XMI with a tempo change within it, -updates now only uses the first tempo setting within an XMI and ignores all others. This now matches the behaviour of the xmidi.c code when it comes to tempo changes.

Going to close this as the original reason for ticket is now sorted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants