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 exporting: system-distance value is too large for systems that end with a multi-measure rest. #17796

Closed
JosephBuchholz opened this issue May 30, 2023 · 0 comments · Fixed by #19512
Labels
Projects

Comments

@JosephBuchholz
Copy link

JosephBuchholz commented May 30, 2023

Issue type

Other type of issue

Bug description

When exporting MusicXML, the system-distance value in the element of the measure is way too large when the system ends with a multi-measure rest. You can see this in the attached .musicxml file at line 303:

    <measure number="15" width="177.03">
      <print new-system="yes">
        <system-layout>
          <system-margins>
            <left-margin>0.00</left-margin>
            <right-margin>0.00</right-margin>
            </system-margins>
          <system-distance>416.24</system-distance> <----- HERE
          </system-layout>
        </print> 

Which is way to big, it should only be around 100, not 400.

Steps to reproduce

You can easily reproduce the bug by adding a multi-measure rest at the end of the system and then export to uncompressed MusicXML.

Screenshots/Screen recordings

screenshot

MuseScore Version

OS: Windows 10 Version 2009, Arch.: x86_64, MuseScore version (64-bit): 4.0.2-230651553, revision: github-musescore-musescore-dbe7c6d

Regression

I don't know

Operating system

Windows 11

Additional context

MuseScore project and the MusicXML file:
files.zip

@muse-bot muse-bot added the needs review The issue needs review to set priority, fix version or change status etc. label May 30, 2023
@bkunda bkunda added this to To do in MusicXML via automation May 31, 2023
@bkunda bkunda removed their assignment May 31, 2023
@cbjeukendrup cbjeukendrup removed the needs review The issue needs review to set priority, fix version or change status etc. label Sep 30, 2023
@cbjeukendrup cbjeukendrup moved this from To do to In progress in MusicXML Sep 30, 2023
MusicXML automation moved this from In progress to Done Sep 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
MusicXML
  
Done
Development

Successfully merging a pull request may close this issue.

5 participants