Re: PATCH: Issue 638 Autobeaming

[Carl Sorensen]

On 12/17/09 9:53 AM, "David Kastrup" <address@hidden> wrote:

> Carl Sorensen <address@hidden> writes:
> 
>> With the revised code and adding an autobeaming rule for 1/64 notes to
>> the default beam settings, the beaming is consistent.
>> 
>> Without the addition of an autobeaming rule for 1/64 notes the beaming
>> appears to be inconsistent.  I will investigate this further.
> 
> I was picking that example because the 2/2 default pattern implies
> breaking 1/32 after quarter notes and 1/64 after half notes.  So that
> could have implied having to revise a _break_ when a 1/64 note appears.
> Because your code appears to assume that smaller timing will always
> imply more breaks, and I was seeing a rule that differed from this
> assumption.
> 
> Of course, the results were quite different from what I half expected to
> see.

Yes.  That difference is due to a pre-existing bug in the code.  The
consider_end check used the current duration, not the shortest duration in
the beam.

That bug has now been fixed, and your example now beams the whole measure
(as expected).  Patch update soon to arrive.

Thanks,

Carl