Re: Updates to bagpipe.ly (issue3825043)

[Carl Sorensen]

On 1/2/11 8:02 AM, "Sven Axelsson" <address@hidden> wrote:

> On 2 January 2011 15:30, Carl Sorensen <address@hidden> wrote:
>> On 1/2/11 4:56 AM, "Sven Axelsson" <address@hidden> wrote:
>> 
>>> On 2 January 2011 12:23, Sven Axelsson <address@hidden> wrote:
>>>> On 2 January 2011 00:35,  <address@hidden> wrote:
>>>>> looks mostly ok, but I don't know what's up with those beaming rules.
>>>>> 
>>>>> 
>>>>> http://codereview.appspot.com/3825043/diff/1/ly/bagpipe.ly
>>>>> File ly/bagpipe.ly (right):
>>>>> 
>>>>> http://codereview.appspot.com/3825043/diff/1/ly/bagpipe.ly#newcode72
>>>>> ly/bagpipe.ly:72: #(override-auto-beam-setting '(end * * * *) 1 2
>>>>> 'Staff)
>>>>> err, why are you removing the good beaming rules, and replacing them
>>>>> with old beaming rules?
>>>> 
>>>> Because the new beaming rules don't do what they should. I guess I'd
>>>> better look into how to make new beaming rules work right instead of
>>>> using my old ones, though.
>>> 
>>> Just to clarify, the beaming should be strict - if I say beam to
>>> quarter notes, the autobeamer should not break at other intervals if
>>> it feels like it.
>> 
>> I think that beaming is strict with the new rules.
>> 
>> The difference between what you have above and the new autobeaming code is
>> that *all* beaming rules are done by time signature now; you can't put a
>> setting to have beaming to quarter notes in all time signatures.
>> 
>> I'd be happy to help you write a music function to get the desired behavior
>> if you'll explain it to me.
> 
> Thanks, that would be most helpful. I see that it was you who changed
> it to use beamExceptions in the first place ...

Are you using 2.13.x?  The autobeam rules have changed in 2.13.x.  The
2.12.x rules don't work any more.

> 
> Attached are examples using \quarterBeaming with an empty
> beamExceptions list (what is currently in bagpipe.ly), and using
> override-auto-beam-setting the way I want it to look.

override-auto-beam-setting doesn't work in 2.13.  Please try your example in
2.13.

Thanks,

Carl