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

**harm spine parser does not handle rests #1658

Open
alexandermorgan opened this issue Oct 24, 2023 · 0 comments
Open

**harm spine parser does not handle rests #1658

alexandermorgan opened this issue Oct 24, 2023 · 0 comments

Comments

@alexandermorgan
Copy link
Contributor

music21 version

8.3.0

Problem summary

The humdrum **harm parser does not parse rest tokens in the **harm spine despite them being part of the **harm syntax as detailed here: https://www.humdrum.org/rep/harm/index.html

Steps to reproduce
Importing a kern file with a rest in the **harm spine raises an error.

# This raises an error:

**harm	**kern
1r	1r
*-	*-

# Omitting the duration of the rest in the **harm spine also raises an error:

**harm	**kern
r	1r
*-	*-

# Using a null token period correctly does not raise an error. This one is fine:

**harm	**kern
.	1r
*-	*-


Expected vs. actual behavior

Rest tokens should be allowed in **harm spines since they're part of the syntax. If there's no music21 representation of this it would be fine to treat them the same as a "." null token.
More information

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

No branches or pull requests

1 participant