Thanks, this is really cool. I was also thinking that the decoding text was getting too long, especially for users who use the GUI. I'll shorten the text by using abbreviations, but only for common abbreviations that someone with only moderate experience with a logic analyzer or MIDI would likely know. I reviewed some manuals for MIDI devices as a guideline. Some examples:
ch = Channels SysEx = System Exclusive RPN = Registered Parameter Number NRPN = Non-Registered Parameter Number I will not use abbreviations for less common phrases. For example, I've seen "CC" used for "Control Change" but only in audio chip datasheets. CC seems too technical. -Chris ---------------------------------------- > Date: Tue, 23 Aug 2016 12:31:43 +0200 > From: [email protected] > To: [email protected] > CC: [email protected] > Subject: Re: [sigrok-devel] Substantial improvement to decoding of MIDI > messages > > Hi, > > On Fri, Aug 19, 2016 at 10:26:02AM -0700, Chris Dreher wrote: >> Here is moderate-sized submission that substantially improves decoding of >> MIDI messages. > > Great stuff, thanks a lot for doing this! > > The PD changes as well as the dumps and unit tests are merged, thanks! > I did some minor whitespace/cosmetics consistency changes and added some > missing ")" in lists.py as well as changed api_version back to 2 > (there's no the PD version, that api_version is meant for other purposes). > > There's some "frames/second" strings in lists.py, maybe we could use > "fps" there, it should be a well-known abbreviation (?) > > > Cheers, Uwe. > -- > http://hermann-uwe.de | http://randomprojects.org | http://sigrok.org ------------------------------------------------------------------------------ _______________________________________________ sigrok-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/sigrok-devel
