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

Ties and Triplets do not work together in TinyNotation #1733

Open
mscuthbert opened this issue Sep 18, 2024 · 0 comments
Open

Ties and Triplets do not work together in TinyNotation #1733

mscuthbert opened this issue Sep 18, 2024 · 0 comments
Assignees

Comments

@mscuthbert
Copy link
Member

music21 version

9.2b

Problem summary

In tinyNotation, a triplet that is tied to a note outside the triplet does not parse properly.

Steps to reproduce

c = converter.parse('tinynotation: 2/4 trip{C8 D8 E8~} E4')
c.show('text', addEndTimes=True)
for n in c[note.Note]: print(n, n.tie)

Expected vs. actual behavior

Expected

{0.0 - 1.6667} <music21.stream.Measure 1 offset=0.0>
    {0.0 - 0.0} <music21.clef.BassClef>
    {0.0 - 0.0} <music21.meter.TimeSignature 2/4>
    {0.0 - 0.3333} <music21.note.Note C>
    {0.3333 - 0.6667} <music21.note.Note D>
    {0.6667 - 1.0} <music21.note.Note E>
    {1.0 - 2.0} <music21.note.Note E>
    {2.0 - 2.0} <music21.bar.Barline type=final>

<music21.note.Note C> None
<music21.note.Note D> None
<music21.note.Note E> <music21.tie.Tie start>
<music21.note.Note E> <music21.tie.Tie stop>

Actual

{0.0 - 1.6667} <music21.stream.Measure 1 offset=0.0>
    {0.0 - 0.0} <music21.clef.BassClef>
    {0.0 - 0.0} <music21.meter.TimeSignature 2/4>
    {0.0 - 0.3333} <music21.note.Note C>
    {0.3333 - 0.6667} <music21.note.Note D>
    {0.6667 - 1.0} <music21.note.Note E>
    {1.0 - 1.6667} <music21.note.Note E>
    {1.6667 - 1.6667} <music21.bar.Barline type=final>

<music21.note.Note C> None
<music21.note.Note D> None
<music21.note.Note E> <music21.tie.Tie start>
<music21.note.Note E> None

More information
Appears to be a problem with the intersection of autoExpires and the ending triplet state.

@mscuthbert mscuthbert self-assigned this Sep 18, 2024
mscuthbert added a commit that referenced this issue Sep 18, 2024
This is a bit of cleanup before fixing a bigger TN problem of Issue #1733.
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