Adds beam collision avoidance to auto beaming (issue4290069)

2011-03-26 Thread mtsolo
Reviewers: , Message: Actualizes Han Wen's idea for putting everything in finalize. Seems to work... Cheers, Mike Description: Adds beam collision avoidance to auto beaming Please review this at http://codereview.appspot.com/4290069/ Affected files: M lily/beam-collision-engraver.cc ___

Re: PATCHES: 48-hour notice for ties and grace notes in lyrics

2011-03-26 Thread Graham Percival
On 3/26/11, Xavier Scheuer wrote: > On 26 March 2011 23:59, Graham Percival wrote: |>> Monday, 23:59pm UK time. Note: the quoted time for "On XYZ, Graham..." must be in French time or something. It's not yet 23:59 in UK time. > Would rather be 47-hour notice, due to daylight saving time that

Re: PATCHES: 48-hour notice for ties and grace notes in lyrics

2011-03-26 Thread Xavier Scheuer
On 27 March 2011 00:44, Xavier Scheuer wrote: > > Would rather be 47-hour notice, due to daylight saving time that starts > this night in Europe (and you said UK time). > We will loose one hour of sleep tonight. > > Good night anyway. ;D OK, no, forget. I saw you sent your e-mail at 23:59 but th

Re: PATCHES: 48-hour notice for ties and grace notes in lyrics

2011-03-26 Thread Xavier Scheuer
On 26 March 2011 23:59, Graham Percival wrote: > > Monday, 23:59pm UK time. > > Don't set 'tieMelismaBusy until old waiting heads have been cleared > http://codereview.appspot.com/4309052/ > > Suppress continuing LyricHyphen under grace note at start of line > http://codereview.appspot.com/4313047

Re: separating flags from noteheads in font (issue4273119)

2011-03-26 Thread Carl Sorensen
On 3/26/11 5:01 PM, "lemniskata.bernoull...@gmail.com" wrote: > Reviewers: , > > Message: > Hi, > > we are going to have multiple variants of each flag, fine-tuned to fit > particular stem lengths. This means LOTS of flag glyphs. > When experimenting some time ago i discovered that adding to

Re: a mistake in mf/GNUmakefile?

2011-03-26 Thread Francisco Vila
2011/3/26 Janek Warchoł : > W dniu 26 marca 2011 23:34 użytkownik Carl Sorensen > napisał: >> >> On 3/26/11 4:21 PM, "Francisco Vila" wrote: >> >> > 2011/3/26 Janek Warchoł : >> >> Hi, >> >> >> >> i found something strange in mf/GNUmakefile. >> >> lines 98-101 read: >> >> $(outdir)/feta20.otf-tab

separating flags from noteheads in font (issue4273119)

2011-03-26 Thread lemniskata . bernoullego
Reviewers: , Message: Hi, we are going to have multiple variants of each flag, fine-tuned to fit particular stem lengths. This means LOTS of flag glyphs. When experimenting some time ago i discovered that adding too many glyphs results in error. I suppose it means there's a limit of how many gly

PATCHES: 48-hour notice for ties and grace notes in lyrics

2011-03-26 Thread Graham Percival
Monday, 23:59pm UK time. Don't set 'tieMelismaBusy until old waiting heads have been cleared http://codereview.appspot.com/4309052/ Suppress continuing LyricHyphen under grace note at start of line http://codereview.appspot.com/4313047/ Cheers, - Graham _

Re: a mistake in mf/GNUmakefile?

2011-03-26 Thread Janek Warchoł
W dniu 26 marca 2011 23:34 użytkownik Carl Sorensen napisał: > > On 3/26/11 4:21 PM, "Francisco Vila" wrote: > > > 2011/3/26 Janek Warchoł : > >> Hi, > >> > >> i found something strange in mf/GNUmakefile. > >> lines 98-101 read: > >> $(outdir)/feta20.otf-table: $(outdir)/feta20.lisp \ > >>   

Re: a mistake in mf/GNUmakefile?

2011-03-26 Thread Carl Sorensen
On 3/26/11 4:21 PM, "Francisco Vila" wrote: > 2011/3/26 Janek Warchoł : >> Hi, >> >> i found something strange in mf/GNUmakefile. >> lines 98-101 read: >> $(outdir)/feta20.otf-table: $(outdir)/feta20.lisp \ >> $(outdir)/feta-noteheads23.lisp \ >>  

Re: a mistake in mf/GNUmakefile?

2011-03-26 Thread Francisco Vila
2011/3/26 Janek Warchoł : > Hi, > > i found something strange in mf/GNUmakefile. > lines 98-101 read: > $(outdir)/feta20.otf-table: $(outdir)/feta20.lisp \ > $(outdir)/feta-noteheads23.lisp \ > $(outdir)/parmesan20.lisp \ >  

a mistake in mf/GNUmakefile?

2011-03-26 Thread Janek Warchoł
Hi, i found something strange in mf/GNUmakefile. lines 98-101 read: $(outdir)/feta20.otf-table: $(outdir)/feta20.lisp \ $(outdir)/feta-noteheads23.lisp \ $(outdir)/parmesan20.lisp \ $(outdir)/feta-alphabet20.lis

Re: Adds beam collision avoidance to auto-beaming (issue4312058)

2011-03-26 Thread Han-Wen Nienhuys
On Sat, Mar 26, 2011 at 6:54 PM, m...@apollinemike.com wrote: >> Quick comment: I'd say that timestep-count as a property is not >> necessary.  The 'width' of an autobeam could also be garnered by >> looking at its bounds, eg beam->get_bound(LEFT)-> get_column().  From >> the column you could eit

Re: Adds beam collision avoidance to auto-beaming (issue4312058)

2011-03-26 Thread m...@apollinemike.com
On Mar 26, 2011, at 5:46 PM, Han-Wen Nienhuys wrote: > On Sat, Mar 26, 2011 at 6:28 PM, wrote: >> Reviewers: , >> >> Message: >> I think this is a better approach to getting beam collision resolution >> out of auto-beams. > > Quick comment: I'd say that timestep-count as a property is not > ne

Re: Adds beam collision avoidance to auto-beaming (issue4312058)

2011-03-26 Thread Han-Wen Nienhuys
On Sat, Mar 26, 2011 at 6:28 PM, wrote: > Reviewers: , > > Message: > I think this is a better approach to getting beam collision resolution > out of auto-beams. Quick comment: I'd say that timestep-count as a property is not necessary. The 'width' of an autobeam could also be garnered by looki

Adds beam collision avoidance to auto-beaming (issue4312058)

2011-03-26 Thread mtsolo
Reviewers: , Message: I think this is a better approach to getting beam collision resolution out of auto-beams. It only effects 2 regtests: repeat-percent.ly (the beamed C-Eb) collision-dots-move.ly (there's only one beam & it moves) I'd like to get this in 2.14.1 if possible in some form. Ch

landscape mode vs. new paper size

2011-03-26 Thread Graham Percival
Hi James, I managed to add a [landscape] option to @lilypond, but it prints music rotate 90 degrees. In retrospect, that should have been obvious -- that's exactly what we want landscape to do! But it's not useful for the NR 3 spacing chapter. How about defining a spacing-doc (icky name) page s

shortened flags affair, part 8: choosing appropriate flag (issue4312057)

2011-03-26 Thread lemniskata . bernoullego
Reviewers: mike_apollinemike.com, Message: Hi, here comes the main part: code that chooses flags. I'm aware that it's quite imperfect. I'm working on it, however some problems are perhaps beyond my skills. Since i'll be busy after the weekend, it would be great if we were able to discuss and pol

Re: Doc: Added new option 'papersize' to Usage (issue4289057)

2011-03-26 Thread percival . music . ca
LGTM, go ahead and push. I'll check out the landscape stuff. http://codereview.appspot.com/4289057/ ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: downstem 64th and 128th flag touchup (issue4273074)

2011-03-26 Thread Janek Warchoł
2011/3/25 > Could you send me the final patch for pushing? > > http://codereview.appspot.com/4273074/ > Here they are, sorry for the delay. Janek From 1d878568c268cb9753abda0f758939e2c990c15e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Janek=20Warcho=C5=82?= Date: Fri, 18 Mar 2011 11:33:55 +0100 S

Re: Suppress continuing LyricHyphen under grace note at start of line (issue4313047)

2011-03-26 Thread percival . music . ca
LGTM http://codereview.appspot.com/4313047/ ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Don't set 'tieMelismaBusy until old waiting heads have been cleared (issue4309052)

2011-03-26 Thread percival . music . ca
LGTM http://codereview.appspot.com/4309052/ ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Don't set 'tieMelismaBusy until old waiting heads have been cleared (issue4309052)

2011-03-26 Thread tdanielsmusic
Reviewers: , Message: This is a suggested fix to issue 742: Delayed ties influence lyrics (even with tieWaitForNote unset) http://code.google.com/p/lilypond/issues/detail?id=742 Reg tests run clean. Description: Don't set 'tieMelismaBusy until old waiting heads have been cleared - fixes 742