Re: Fix issue 4355 -- broken beam subdivision (issue 226700043 by address@hidden)

[Carl Sorensen]

On 4/25/15 6:37 AM, "Urs Liska" <address@hidden> wrote:

>>
>> In that context I'd like to raise the question if baseMoment really is
>> the appropriate property for configuring beamSubdivision or if it
>> shouldn't have a dedicated property. Changing baseMoment might have
>> side-effects on other aspects of the engraving that may or may not be
>> intended.
>>
>
>What would you think of the following interface:
>
>\set beamSubdivision = ##t behaves as currently (fetch value from
>baseMoment)
>\set beamSubdivision = #(ly:make-moment 1 16) sets the division
>frequency to an explicit value.
>
>That way we
>- don't need a new property
>- decouple subdivision frequency from baseMoment
>- don't break existing code

I actually think that baseMoment is intended precisely for subdivision.

baseMoment sets the subdivision interval.

beatGrouping combines the baseMoment together to get beaming groups.

My first fix will not include a new property, and then we'll see where we
go from that.

Thanks,

Carl