Re: midi2ly: Fraction Reduction + 2 minor issues

[Simon Albrecht]

Dear fellow Bug Squad members,

unfortunately I can’t access my sourceforge account right now. Could 
anyone please open tracker issues for this (I think three separate ones 
would be in order)?


Am 14.04.2017 um 02:18 schrieb Christopher Heckman:
> (1) When midi2ly is run, it will print fractions with large numerators
> and denominators,
[…]
> I have written a patch to fix this.

[…]

Am 15.04.2017 um 12:10 schrieb Simon Albrecht:
> Am 14.04.2017 um 02:18 schrieb Christopher Heckman:
>
> > (2) In line 181 of midi2ly.py, the authors talk about how the 7th of a
> > minor scale should be raised, but this is not notated. The real issue
> > is that there are two types of minor scale involved, the harmonic
> > minor (1 2 b3 4 5 b6 7) and the natural minor (1 2 b3 4 5 b6 b7). The
> > reason that "C# is not put in the key signature of D minor" is that
> > the minor scale is the natural minor, not the harmonic minor.
>
> I’d rather say that all these musicotheoretical musings don’t belong 
> there at all. Instead one could just say:
> # induce correct spelling of 7th scale degree in minor keys


> (3) At line 677, there's a cryptic comment
>
> # urg, this will barf at meter changes

i.e. an enhancement request to fix this

Best, Simon