LilyIssues / Issues / #2527 Finger-flag collision

Google Importer - 2012-08-27

Originally posted by: colingh...@gmail.com

With lilypond-2.16.0 on x86_64 GNU/Linux the output looks very similar to the PNG in the initial entry of this issue tracker. Source and output attached.

bug.preview.png

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-08-27

Originally posted by: colingh...@gmail.com

Sorry, here's the bug.ly source for my last comment, Comment #28.

As Keith suggested in Comment #23 I tried:

\override Fingering #'add-stem-support % syntax error, my ignorance on that one

\override Fingering #'add-stem-support = ##t % no fingering at all in output

\override Fingering #'add-stem-support = ##f % same output as my Comment #28.

Cheers,
Colin.

bug.ly

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-08-28

Originally posted by: mts...@gmail.com

Thanks Colin!
I'll have a look at it next week.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-11

Originally posted by: mts...@gmail.com

Uses single algorithm for side-position spacing.

To implement this, uses grob-extent based boxes for simple skylines
instead of stencil-extent based boxes. Stencils are still used for
more complicated skylines.

Generalizes algorithm used in Fingeing_column code, allowing for better
skyline spacing and fixing some collisions.

http://codereview.appspot.com/6827072

Labels: Patch-new

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-11

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests. Reg tests here: https://www.yousendit.com/download/WUJaOGNVdVVRWUp1a3NUQw

Labels: -Patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-13

Originally posted by: ColinPKC...@gmail.com

Per Keith's comments on Rietveld, this is not ready for countdown.

Labels: -Patch-review Patch-needs_work

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-15

Originally posted by: mts...@gmail.com

Uses single algorithm for side-position spacing.

To implement this, uses grob-extent based boxes for simple skylines
instead of stencil-extent based boxes. Stencils are still used for
more complicated skylines.

Generalizes algorithm used in Fingeing_column code, allowing for better
skyline spacing and fixing some collisions.

http://codereview.appspot.com/6827072

Labels: -Patch-needs_work Patch-new

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-16

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests. Reg tests here: https://www.yousendit.com/download/WUJiZGVWeWE5RmFKUmNUQw

Labels: -Patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-18

Originally posted by: ColinPKC...@gmail.com

Per Kieth's comments on Rietveld, this needs further work.

Labels: -Patch-review Patch-needs_work

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-19

Originally posted by: mts...@gmail.com

Uses single algorithm for side-position spacing.

To implement this, uses grob-extent based boxes for simple skylines
instead of stencil-extent based boxes. Stencils are still used for
more complicated skylines.

Generalizes algorithm used in Fingeing_column code, allowing for better
skyline spacing and fixing some collisions.

http://codereview.appspot.com/6827072

Labels: -Patch-needs_work Patch-new

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-20

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests. Reg tests here: https://www.yousendit.com/download/WUJZT2pKbWdOMUJMWE1UQw

Labels: -Patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-26

Originally posted by: ColinPKC...@gmail.com

(No comment was entered for this change.)

Labels: -Patch-review Patch-countdown

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-28

Originally posted by: k-ohara5...@oco.net

In 'finger-chords.ly' fingering -5 clears the flag, which is nice, but...

Breaks 'finger-chords.ly'
"possible to add horizontal fingerings to notes" -- no longer lined-up horizontally
"Fingering clears ... if @code{'add-stem-support} is set." -- now always in effect

Breaks 'fingering-column.ly'
"Non-intersecting Fingerings are left alone" --they are shifted half-space up
the tall column of six finger-numbers becomes un-evenly spaced

Labels: -Patch-countdown Patch-needs_work

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-29

Originally posted by: mts...@gmail.com

Uses single algorithm for side-position spacing.

To implement this, uses grob-extent based boxes for simple skylines
instead of stencil-extent based boxes. Stencils are still used for
more complicated skylines.

Generalizes algorithm used in Fingeing_column code, allowing for better
skyline spacing and fixing some collisions.

http://codereview.appspot.com/6827072

Labels: -Patch-needs_work Patch-new

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-29

Originally posted by: mts...@gmail.com

Uses single algorithm for side-position spacing.

To implement this, uses grob-extent based boxes for simple skylines
instead of stencil-extent based boxes. Stencils are still used for
more complicated skylines.

Generalizes algorithm used in Fingeing_column code, allowing for better
skyline spacing and fixing some collisions.

http://codereview.appspot.com/6827072

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-29

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests. Reg tests here:

https://www.yousendit.com/download/WUJaVWRqY1N3TGgzZU1UQw

Labels: -Patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-11-29

Originally posted by: ColinPKC...@gmail.com

(No comment was entered for this change.)

Labels: -Patch-review Patch-countdown

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-12-03

Originally posted by: ColinPKC...@gmail.com

Counted down to 20121203, but I'm not sure if Keith's comment in #21 on Rietveld was addressed. If so, please push; if not, please mark as "needs-work".

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-12-06

Originally posted by: ColinPKC...@gmail.com

Per Mike's comment on Rietveld.

Labels: -Patch-countdown Patch-needs_work

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-12-16

Originally posted by: mts...@gmail.com

Uses single algorithm for side-position spacing.

To implement this, uses grob-extent based boxes for simple skylines
instead of stencil-extent based boxes. Stencils are still used for
more complicated skylines.

Generalizes algorithm used in Fingeing_column code, allowing for better
skyline spacing and fixing some collisions.

http://codereview.appspot.com/6827072

Labels: -Patch-needs_work Patch-new

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-12-16

Originally posted by: mts...@gmail.com

Uses single algorithm for side-position spacing.

To implement this, uses grob-extent based boxes for simple skylines
instead of stencil-extent based boxes. Stencils are still used for
more complicated skylines.

Generalizes algorithm used in Fingeing_column code, allowing for better
skyline spacing and fixing some collisions.

http://codereview.appspot.com/6827072

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-12-16

Originally posted by: mts...@gmail.com

The most recent patchset makes NoteColumn cross-staff when it contains a cross-staff stem. Using this model, most (if not all) problems from the previous patch-sets are cleared up.

Sorry for the commented-out printf's - they'll disappear with the final push.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-12-16

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests. Reg Tests: https://www.yousendit.com/download/WUJZblFJQTZEbUlPd3NUQw

Labels: -Patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-12-17

Originally posted by: mts...@gmail.com

Uses single algorithm for side-position spacing.

To implement this, uses grob-extent based boxes for simple skylines
instead of stencil-extent based boxes. Stencils are still used for
more complicated skylines.

Generalizes algorithm used in Fingeing_column code, allowing for better
skyline spacing and fixing some collisions.

http://codereview.appspot.com/6827072

Labels: -Patch-review Patch-new

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2012-12-17

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests. Reg Tests: https://www.yousendit.com/download/WUJiZm1lcTJQb0xtcXNUQw

Labels: -Patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Finger-flag collision

Issue Tracker for LilyPond

Searches

Help

#2527 Finger-flag collision

Related

Discussion