-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve performance.ini format #558
Comments
Looks like the infrastructure for this might land in Circle sooner or later. |
I'm not really sure what the complexity offers over the existing format. Once preset performances are defined, most people would only be saving them from the UI, so won't really benefit from a more user-readable format anyway? If we did anything (and I'm not convinced we should tbh) then moving to a binary sysex format (as described here: #533) probably makes more sense, especially if tools will be required to read/write the new files...? And if we're writing tools, then a converter from a format like the above to the existing format probably is a lot easier than adding support directly in MiniDexed (even if added to circle)? Kevin |
Would have made it easier (a bit) to read and write by hand. But possibly not worth the effort, especially now that tools and performances already exist. |
We should improve https://github.com/probonopd/MiniDexed/blob/main/src/performanceconfig.h/.cpp so that we can use something like
Maybe we need to investigate something like https://github.com/pulzed/mINI to read and write a more complete
.ini
format?The text was updated successfully, but these errors were encountered: