Mike, was your patch on Jan 10 a serious one? I'm not certain where we stand on this particular issue... I suspect that various other changes (especially Keith's work on 1472) may change this. And you're doing fantastic work on issue 37 right now, so I don't want to distract you by getting back into this issue.
And I also see that the patchset included a fix for 37, which I know has changed drastically over the past few weeks.
I'm going to mark the patch "needs work" for now, and when other stuff settles down, you can either revise this patch, or just put it on codereview.
Labels: -Patch Patch-needs_work
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Beam collisions as currently implemented do not deal with this sort of collision automatically. I'm not sure why, as clefs are accounted for in the beam collision engraver. I may have some time to do some investigating this weekend.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Originally posted by: v.villenave
Why "low"?
Labels: -Priority-Low Priority-Medium
Originally posted by: percival.music.ca@gmail.com
Historically, all collisions were low.
Labels: -Priority-Medium Priority-High
Owner: ---
Originally posted by: nez...@gmail.com
http://lists.gnu.org/archive/html/lilypond-devel/2011-01/msg00301.html
Labels: Patch
Originally posted by: percival.music.ca@gmail.com
Mike, was your patch on Jan 10 a serious one? I'm not certain where we stand on this particular issue... I suspect that various other changes (especially Keith's work on 1472) may change this. And you're doing fantastic work on issue 37 right now, so I don't want to distract you by getting back into this issue.
And I also see that the patchset included a fix for 37, which I know has changed drastically over the past few weeks.
I'm going to mark the patch "needs work" for now, and when other stuff settles down, you can either revise this patch, or just put it on codereview.
Labels: -Patch Patch-needs_work
Originally posted by: mts...@gmail.com
My most recent patch for Issue 37 fixes this issue.
http://codereview.appspot.com/4022045
Originally posted by: ColinPKC...@gmail.com
Graham/Mike: given Graham's reg test and Mike's comment, should this issue be marked Fixed or perhaps Patch-Review?
Originally posted by: mts...@gmail.com
Beam collisions as currently implemented do not deal with this sort of collision automatically. I'm not sure why, as clefs are accounted for in the beam collision engraver. I may have some time to do some investigating this weekend.
Originally posted by: mts...@gmail.com
Sorry - spoke too soon.
The workaround is:
\relative {
c8 [ \clef bass a, ] c,4
}
The beam collision engraver cannot account for automatic beams (yet).
Originally posted by: mts...@gmail.com
Removed with the beam collision engraver (version 2.13.57).
Status: Fixed
Originally posted by: ColinPKC...@gmail.com
Verified with 2.13.57
Status: Verified
Originally posted by: julien.r...@gmail.com
(No comment was entered for this change.)
Labels: -Patch-needs_work