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

References to grace notes in Slurs are incomplete #12

Closed
mscuthbert opened this issue Nov 11, 2013 · 2 comments
Closed

References to grace notes in Slurs are incomplete #12

mscuthbert opened this issue Nov 11, 2013 · 2 comments

Comments

@mscuthbert
Copy link
Member

Reported by bret.aarden, Jun 25, 2012

Hello,

I just stumbled upon an interesting behavior: references to grace notes in slurs seem to be incomplete. In "mozart/k458/movement3.mxl", for instance, m. 53 has slurs from grace notes to eighth notes. In the slur spanners the eighth notes have intact DefinedContexts, but the grace notes have lost the references to their containing measures. The copies in the slurs also no longer think they're grace notes. Regular notes in the same slur have no such problem. (I first spotted this in the first movement, which has slurs between two grace notes, and in that case both grace notes are afflicted.)

s=corpus.parse('mozart/k458/movement3.mxl')
s[1][167]
<music21.stream.Measure 53 offset=208.0>
s[1][167][1]
<music21.note.Note G>
s[1][167][1].isGrace
True
s[1][167][1].getSites()
[None, <music21.stream.SpannerStorage 4344967056>, <music21.stream.Measure 53 offset=208.0>]
s[1][112]
<music21.spanner.Slur <music21.note.Note G><music21.note.Note F>>
s[1][112][0].isGrace
False
s[1][112][0].getSites()
[None, <music21.stream.SpannerStorage 4344967056>]
s[1][112][1].getSites()
[None, <music21.stream.SpannerStorage 4344967056>, <music21.stream.Measure 53 offset=208.0>]

I'm using v.1.0 on MacOS 10.7 with Python 2.7.

No big problem, now that I know to check for this behavior, but I wanted to bring it to your attention.

Cheers,
-Bret.

@mscuthbert
Copy link
Member Author

I'm following the Enhancement model suggested at http://brianegranger.com/?p=249 -- closing the issue here; if others believe it is important, they can reopen the issue.

@mscuthbert
Copy link
Member Author

probably fixed with some changes since 2015.

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

1 participant