grame-cncm / libmusicxml

A C/C++ library to support the MusicXML format.
Mozilla Public License 2.0
152 stars 33 forks source link

Or the octave problem, I found that I can't get around it #51

Closed zhwei8090 closed 2 years ago

zhwei8090 commented 2 years ago

When the octave label appears at the last position of the measure, the sound at the first position of the next measure will inevitably be affected. Through analysis, there should be a problem when converting xml to gmn exported xml: 42_3_finale.xml.zip In finale it is like this: image But after converting to gmn, it's not right, moved down an octave: image

{[ \staff<1> \set<autoHideTiedAccidentals="on"> \auto<autoInstrPos="on"> \instr<"钢琴"> \accol<id=0, range="1-2"> \barFormat<style= "system", range="1-2"> 
   (* meas. 1 *)  \clef<"g2"> \key<0> \meter<"4/4", autoBarlines="off", autoMeasuresNum="system"> \oct<1> \stemsDown \beamsOff b2/2 \stemsDown \beamsOff b2/2 \oct<0> \bar<measNum=2> 
   (* meas. 2 *)  \stemsAuto \beamsOff b0/1 ]
 , 
[ \staff<2> \set<autoHideTiedAccidentals="on"> \barFormat<style= "system", range="2"> 
   (* meas. 1 *)  \staffFormat<distance=13> \clef<"f4"> \key<0> \meter<"4/4", autoBarlines="off", autoMeasuresNum="system"> _/1 \bar<measNum=2> 
   (* meas. 2 *)  _/1 ]
  }
zhwei8090 commented 2 years ago

It has been solved, create a pr, please take a look, thank you

arshiacont commented 2 years ago

@zhwei8090 where did you create the PR? I can't see it here

zhwei8090 commented 2 years ago

@arshiacont It has been submitted, please check it, thank you

zhwei8090 commented 2 years ago

I have no problem with the initial test here, you can verify it again

arshiacont commented 2 years ago

Thank you @zhwei8090 ! and sorry I forgot to correctly initialize the variable... 👍