[Top][All Lists]
[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
- Re: Fix Issue 1290 (issue3832046), (continued)
- Re: Fix Issue 1290 (issue3832046), k-ohara5a5a, 2011/01/04
- Re: Fix Issue 1290 (issue3832046), percival . music . ca, 2011/01/05
- Re: Fix Issue 1290 (issue3832046), Carl . D . Sorensen, 2011/01/07
- Re: Fix Issue 1290 (issue3832046), joeneeman, 2011/01/07
- Re: Fix Issue 1290 (issue3832046), n . puttock, 2011/01/07
Re: Fix Issue 1290 (issue3832046), n . puttock, 2011/01/07
Re: Fix Issue 1290 (issue3832046), k-ohara5a5a, 2011/01/07
Re: Fix Issue 1290 (issue3832046), Carl . D . Sorensen, 2011/01/07
Re: Fix Issue 1290 (issue3832046), Carl . D . Sorensen, 2011/01/07
Re: Fix Issue 1290 (issue3832046), Keith OHara, 2011/01/07