Re: Updates to bagpipe.ly (issue3825043)

[Carl Sorensen]

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,

Carl