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

Additive meter time signatures #554

Closed
rettinghaus opened this issue Apr 22, 2017 · 8 comments
Closed

Additive meter time signatures #554

rettinghaus opened this issue Apr 22, 2017 · 8 comments

Comments

@rettinghaus
Copy link
Contributor

Although this is a libMEI issue, I leave it here, as Verovio is affected by this:

meter.count is treated as int but it should be read as str as it allows "additive expression":

Value of datatype a string matching the following regular expression: "\d+(.\d+)?(\s*+\s*\d+(.\d+)?)*"

(same applies to @count)
Showing a plus sign in SVG shouldn't be that hard then, is it?

@rettinghaus
Copy link
Contributor Author

Glyph U+EA6C ("+") would be needed also.

@lpugin
Copy link
Contributor

lpugin commented May 17, 2017

@rettinghaus
Copy link
Contributor Author

Yeah, makes more sense. So E08D should be it.

@lpugin
Copy link
Contributor

lpugin commented May 18, 2017

The glyph is added. Now it is not as simple as changing to string, because this will break all the timestamp calculation. We need to think about a custom class for compound values.

@lpugin lpugin changed the title Compound time signatures Additive meter time signatures May 18, 2017
@rettinghaus
Copy link
Contributor Author

Not really. We could keep m_meterCount as an integer with the "resulting" count from something like "3+2+4+1". There should be a little function calculating the resulting count.

@lpugin
Copy link
Contributor

lpugin commented May 22, 2019

General feature request

@lpugin lpugin closed this as completed May 22, 2019
@craigsapp craigsapp mentioned this issue Jan 19, 2020
@craigsapp
Copy link
Contributor

@rettinghaus
Copy link
Contributor Author

Support added with #2200.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants