Menu
▾
▴
1 Attachments
#40 collision glissando accidental (includes workaround)
Originally created by: *anonymous
Originally created by: gpermus@gmail.com
Originally owned by: k-ohara5...@oco.net
\header {
texidoc = "
the glissando line doesn't stop before an accidental.
I'd give this an even lower priority if possible.
Workaround:
@code{ \\once \\override Glissando #'gap = #0.5 }
@code{ \\once \\override Glissando #'extra-offset = #'(-0.5 . 0)}
produces a gliss which does an (IMO) acceptable job of avoiding
collision.
"
reportedin = "2.7.5"
}
\version "2.9.16"
\layout{ragged-right=##t}
\relative c'' {
\override Glissando #'thickness = #3
c4\glissando ces
}
Originally posted by: v.villenave
(Reproduced with 2.11.47)
Originally posted by: percival.music.ca@gmail.com
(No comment was entered for this change.)
Labels: -Priority-Low Priority-Medium
Owner: ---
Originally posted by: mts...@gmail.com
Fix posted here.
http://codereview.appspot.com/4801083
Originally posted by: percival.music.ca@gmail.com
(No comment was entered for this change.)
Labels: Patch-review
Originally posted by: percival.music.ca@gmail.com
(No comment was entered for this change.)
Labels: -type-Collision Type-Ugly
Originally posted by: mts...@gmail.com
Fixed with [rb3c0b773bde05b3971f087ed6c5f4fb40a1b20b1].
Status: Fixed
Originally posted by: ColinPKC...@gmail.com
Pushed as [rb3c0b773bde05b3971f087ed6c5f4fb40a1b20b1] by Mike Solomon.
Originally posted by: brownian.box@gmail.com
Should be tagged as "fixed_2_15_9" ?
Originally posted by: percival.music.ca@gmail.com
yes
Labels: -Priority-Medium fixed_2_15_9
Originally posted by: brownian.box@gmail.com
This is fixed, but I've submitted another one, issue 1860.
Status: Verified
Originally posted by: lemzw...@googlemail.com
At least in version release/2.17.16-1-45-g7ea1588, this is broken again.
Attached is a current rendering of the original issue. Do we have a regtest for this?
Labels: -Patch-review -fixed_2_15_9 Regression
Status: Accepted
Originally posted by: PhilEHol...@googlemail.com
glissando-accidental.ly
Originally posted by: lemzw...@googlemail.com
Thanks. And indeed, it fails.
Originally posted by: k-ohara5...@oco.net
First bad commit is
[r987fc85646498b0d8bfa89b85ad1375d942a53f0]
Date: Fri Sep 14 07:20:22 2012 +0200
Uses exact horizontal skylines for accidental placement.
issue 2811
Originally posted by: k-ohara5...@oco.net
It turns out the wrong patch set was pushed for this issue, while the final reviewed patch set is more robust and co-exists with the patch for issue 2811.
I pushed a correction to bring the code in agreement with the final patch from Mike's review, as [rb47350d1efb1128b2a8ea13f340dd5e3a53f0735].
Labels: fixed_2_17_25
Owner: k-ohara5...@oco.net
Status: Fixed
Originally posted by: Elu...@gmail.com
(No comment was entered for this change.)
Status: Verified
Originally posted by: k-ohara5...@oco.net
Issue 2942 has been merged into this issue.