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

Writing mensural impossible without --all-pages #3938

Closed
rettinghaus opened this issue Feb 5, 2025 · 5 comments · Fixed by #3947
Closed

Writing mensural impossible without --all-pages #3938

rettinghaus opened this issue Feb 5, 2025 · 5 comments · Fixed by #3947

Comments

@rettinghaus
Copy link
Contributor

Describe the problem
Trying to load a Mensural MEI into Verovio and saving it back gives an Error:

[Error] MEI output with filter is not possible for mensural music

To Reproduce
Steps to reproduce the behavior:

  1. Take a mensural file from Verovio's test suite
  2. verovio -t mei filename
  3. See error

Expected behavior
The file is written back.

Input data
See https://www.verovio.org/test-suite.xhtml?cat=mensural

Verovio information

  • develop
  • command-line

Environment information (as appropriate)

  • macOS
@lpugin
Copy link
Contributor

lpugin commented Feb 5, 2025

Looks similar to this #3804

@rettinghaus
Copy link
Contributor Author

rettinghaus commented Feb 5, 2025

Ok, adding --all-pages works. But still I don't understand why. This is very counter-intuitive.
Especially as no "filter" was selected.

@lpugin
Copy link
Contributor

lpugin commented Feb 5, 2025

Yes, I agree. Filters were added here

@lpugin lpugin changed the title Writing mensural impossible Writing mensural impossible without --all-pages Feb 5, 2025
@rettinghaus
Copy link
Contributor Author

@brdvd Could you have a look?

@rettinghaus
Copy link
Contributor Author

Probably connected to this:
Empty measures are lost without using --all-pages. Try score-013 and score-014 from the test suite.

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 a pull request may close this issue.

2 participants