-
Notifications
You must be signed in to change notification settings - Fork 55
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
Add new commands #26
base: master
Are you sure you want to change the base?
Add new commands #26
Conversation
+ list_uris => List current effect instances URIs + list_bundles => List current effect instances bundle names + preset_list %i => List Presets URIs for a given effect instance ID + param_info %i %s => Get range info for a given instance's parameter Update README.md
Trevis build here failed on some stupid wget error:
|
@falkTX Can you at least give a comment if you plan to pull this or not? |
I am not associated with MOD anymore. ask @ricardocrudo or perhaps @jkbd |
I will discuss your pull request with @ricardocrudo next week. |
@guysoft we decided to not merge your pull request now because we don't seem to be using these features anytime soon and we have other priorities now. |
Thanks for your reply. We also in Zynthian moved not to use these features and we are using |
Hey,
Not sure you want it, but in zynthian we have a few extra commands we added for our stuff. If you like, you are welcome to PR them bach. work of @jofemodo .
If there is anything you want changed in order to merge also please tell us.
It would make our maintenance easier.
Update README.md