On Sun, Apr 16, 2017 at 9:00 AM, Martin Tarenskeen <m.tarensk...@zonnet.nl> wrote: > > On Thu, 13 Apr 2017, Christopher Heckman wrote: > >> I have written a patch to fix this. Inside of the class Duration (line >> 129), replace the dump function with: > > Hi Chris, thanks for trying to improve the midi2ly tool. > > I like the idea and really appreciate your efforts, but I tried your patch > and it doesn't seem to work (yet), at least not on my system. I can't > convert a midifile after applying your patch. Did anyone try Christophers > proposal succesfully? (I installed modified version of midi2ly as > /usr/bin/midi2ly_ on my system)
Hm. My Python might be a bit rusty, but the syntax doesn't differ from the original code. However, you need to answer the "usual questions": (1) Did midi2ly work before? (2) Is this failing on all midi files, or just one in particular? (Followup: If it's failing all the time, then post a URL to one of your files.) (3) What OS are you using? (I tested midi2ly on the Ubuntu machine at work, and I had no problems. I can't run it on OSX, maybe because I have the "wrong type" of Python on it.) One other thing to mention is that a patch has been posted at git, and it includes the other change I mentioned. (For instance, e4*1/2 now prints as e8, and e8*7/8 now prints as e2.. ) ================================== Christopher Carl Heckman Honors Faculty Lecturer 33° 25' 7'' N, 111° 55' 55'' W ECA 212 School of Mathematical & Statistical Sciences Arizona State University (T1N R4E Sect22 NW1/4) Tempe, AZ 85287 - 1804 United States of America North America Planet Earth Solar System Gould Belt Local Bubble Local Fluff Milky Way Galaxy, Orion Arm Local Group Virgo Supercluster Pisces-Cetus Supercluster Complex Universe Multiverse ================================== _______________________________________________ bug-lilypond mailing list bug-lilypond@gnu.org https://lists.gnu.org/mailman/listinfo/bug-lilypond