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

Add support for CMME import #3797

Merged
merged 133 commits into from
Oct 17, 2024
Merged

Add support for CMME import #3797

merged 133 commits into from
Oct 17, 2024

Conversation

lpugin
Copy link
Contributor

@lpugin lpugin commented Sep 20, 2024

CMME files can be imported with -f cmme.xml but should be auto-detected.

A CMME file is imported as an MEI mensural file with one single mdiv/score. Sections in CMME are imported as distinct section elements within the score.

Everything in imported as mensural notation, including sections marked as Plainchant in CMME since there is no difference in the way the music is encoded in those sections. Where there is less voices in a section than in the rest of the score, empty and invisible staff elements <staff visibility="false"/> are added to the section.

The initial staffDef elements have no @clef.* and no @keyisg and these are given in the layer instead. Every staffDef contains a mensur indicating that all level of divisions are binary by default.

There is one issue listing remaining points to tackle.

* 2 static_cast left in BeamSegment::AdjustBeamToFrenchStyle
* 1 static_cast left in BeamSegment::CalcStemDefiningNote
# Conflicts:
#	src/alignfunctor.cpp
* Remove some duration defines
* Remove some uses of DUR_MAX
* Adjust code comments
# Conflicts:
#	src/durationinterface.cpp
# Conflicts:
#	src/alignfunctor.cpp
#	src/durationinterface.cpp
#	src/horizontalaligner.cpp
#	src/layerelement.cpp
@lpugin lpugin marked this pull request as ready for review October 17, 2024 07:08
@lpugin
Copy link
Contributor Author

lpugin commented Oct 17, 2024

I would suggest for this to be merged. Several small things that remain to be adjusted will quite likely need some adjustments to MEI. In the meantime it will be easier if we already have this in the next release. More documentation will be added to the reference book once the release is made.

@lpugin lpugin requested review from a team and ahankinson October 17, 2024 07:10
@ahankinson ahankinson merged commit 8ebb1c8 into develop Oct 17, 2024
9 checks passed
@lpugin lpugin deleted the develop-cmme branch October 18, 2024 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants