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

MusicXML pedal import failure in lower versions #20395

Open
Jojo-Schmitz opened this issue Dec 10, 2023 · 2 comments
Open

MusicXML pedal import failure in lower versions #20395

Jojo-Schmitz opened this issue Dec 10, 2023 · 2 comments
Labels
regression Regression on a prior release

Comments

@Jojo-Schmitz
Copy link
Contributor

Jojo-Schmitz commented Dec 10, 2023

Issue type

Import/export issue

Bug description

No response

Steps to reproduce

Seems something is wrong with pedal export, or its later import in older versions:
Take the attached score,
image
export to MusicXML (attached too), import in Mu3 (the official way from Mu4 back to Mu3), gives:

Fatal error: line 108 column 73 Content of attribute type does not match its type definition: String content is not listed in the enumeration facet..

      <direction placement="below">
        <direction-type>
          <pedal type="resume" line="yes" sign="yes" default-y="-65.00"/>
          </direction-type>
        </direction>

It is the type="resume" it complains about
pedal.zip

Mu 4.1.1 doesn't complain, but screws up on import quite badly:
image

Screenshots/Screen recordings

No response

MuseScore Version

4.2.0 Beta and latest master

Regression

Yes, this used to work in a previous version of MuseScore 4.x

Operating system

Windows 11

Additional context

I believe this is related to #20224 (for 4.2.0) and #20219 (for master/4.3.0)
It might be older than that though, not sure whether #20224 in in 4.2.0 Beta?

@muse-bot muse-bot added the regression Regression on a prior release label Dec 10, 2023
@Jojo-Schmitz Jojo-Schmitz changed the title MusicXML pedal import failure MusicXML pedal import failure in lower versions Dec 10, 2023
@rettinghaus
Copy link
Contributor

Support for type="resume" was introduced with #19972. Did you back port that also?
NB: This additional type came with MusicXML 4.0, it wasn't available in MusicXML 3.1.

@Jojo-Schmitz
Copy link
Contributor Author

Jojo-Schmitz commented Dec 11, 2023

It is an issue with 3.6.2, not my 3.7. And with 4.1.1 too.

But yes, MueseScore 3.6.2 is still on MusicXML 3.1, not 4, so I guess that puts this part of the issue to a rest.

But what about to very bizarre 'import' in 4.1.1?

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

No branches or pull requests

3 participants