Re: Adding note color handling to musicxml2ly

2009-07-15 Thread David Kastrup
Bret Aarden writes: >>> +def print_note_color (self, object, rgb=None): >>> +if rgb: >>> +str = ("\override %s #'color = #(rgb-color %s %s %s)" % >>> + (object, rgb[0], rgb[1], rgb[2])) >>> >> >> I suppose this should be \\override (i.e. escape the b

Re: Adding note color handling to musicxml2ly

2009-07-14 Thread Bret Aarden
Ah, drat, I used the wrong subject. Here it is again. I was about to suggest the same. The \revert is never inserted. But then, you don't want an \override anyway, because the setting should only apply to that one note, which has the color attribute, not to all following ones. So, you really

Re: Adding note color handling to musicxml2ly

2009-07-14 Thread Reinhold Kainhofer
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Am Dienstag, 14. Juli 2009 18:07:10 schrieb Carl Sorensen: > On 7/13/09 1:35 AM, "Bret Aarden" wrote: > > There is no doubt a much more elegant way to do this, but the included > > patch works for me, [...] > > I saw that you had provision in print_no

Re: Adding note color handling to musicxml2ly

2009-07-14 Thread Carl Sorensen
On 7/13/09 1:35 AM, "Bret Aarden" wrote: > > > I've written a music search tool that exports MusicXML and colors the > matching notes, and I'd like those colors to show up in LilyPond > typesetting. > > There is no doubt a much more elegant way to do this, but the included > patch works for

Adding note color handling to musicxml2ly

2009-07-14 Thread Bret Aarden
I've written a music search tool that exports MusicXML and colors the matching notes, and I'd like those colors to show up in LilyPond typesetting. There is no doubt a much more elegant way to do this, but the included patch works for me, and it would be great to see this functionality adde

Adding note color handling to musicxml2ly

2009-07-13 Thread Bret Aarden
I've written a music search tool that exports MusicXML and colors the matching notes, and I'd like those colors to show up in LilyPond typesetting. There is no doubt a much more elegant way to do this, but the included patch works for me, and it would be great to see this functionality added to