lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Fix Issue 1290 (issue3832046)


From: Carl Sorensen
Subject: Re: Fix Issue 1290 (issue3832046)
Date: Sat, 8 Jan 2011 07:19:16 -0700

On 1/7/11 9:16 PM, "Keith OHara" <address@hidden> wrote:

> On Fri, 07 Jan 2011 15:50:40 -0800, Carl Sorensen <address@hidden> wrote:
>> 
>> A default value of skyline-horizontal-padding of 1.2 gets
>> skyline-horizontal-padding.ly to work well.
>> 
>> An override to System 'skyline-horizontal-padding = #0 in
>> stem-length-estimation.ly gets it working well.  Since we already have
>> overrides to make this regtest work, I don't see it as a big problem to add
>> another.
>> 
> 
> I'll argue once more for putting the override to sklyine-horizontal-padding in
> the reg-test for skyline-horizontal-padding, so you can use a default tuned to
> realistic use.
> 
> A default of 1.2 strikes me as rather large, although the three files I
> checked last evening do alright 1.2.
> 
> We should err on the side of too low a padding, at first.  Users suffering
> from too much interleaving have some hope to find the override by searching
> for 'interleaving' or by searching 'padding' and sorting through the results.
> Users needing to fit music (that looks just a bit more interleaved than
> stem-estimation length.ly) will look to the Notation Reference section on
> "fitting music onto fewer pages".  Let's teach sklyine-horizontal-padding in
> that doc before making the default anything that a user might want to reduce.
> 
Isn't this an argument for having a *bigger* default value?  If the place we
put the teaching obout skyline-horizontal-padding is a section on "fitting
music onto fewer pages", we're talking about making it tighter, not looser.
And we get tighter by reducing skyline-horizontal-padding.  Oh, perhaps you
mean that "If we add a section on skyline-horizontal-padding to the NR, then
we can make skyline-horizontal-padding bigger".  That seems to make sense.

Anyway, you have more practical experience with what looks good on spacing
variables, so I trust you implicitly.


I think you missed my latest patch.

Default is 0.5 as you suggested.

Regtest has 1.5.

Thanks,

Carl




reply via email to

[Prev in Thread] Current Thread [Next in Thread]