Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add new commands: #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add new commands: #9

wants to merge 2 commits into from

Conversation

jofemodo
Copy link

@jofemodo jofemodo commented Nov 2, 2016

  • 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

@jofemodo
Add new commands:
7258643 
 + 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
@jofemodo
Copy link
Author

jofemodo commented Nov 2, 2016 •
edited
Loading

Hi MOD developpers!

I've added some commands to mod-host that can be useful for getting more feedback from the server about the running effects, its presets and parameters. I've not added tests, but if you finally push the new commands into the main, i could add it.

Also, if you think that other command names would fit better, tell me, or change it directly ;-)

Kind Regards!

@jofemodo jofemodo closed this Nov 2, 2016
@jofemodo jofemodo reopened this Nov 2, 2016
@SrMouraSilva
Copy link

Please, update the README.md

@jofemodo
Copy link
Author

jofemodo commented Nov 3, 2016

OK! It's updated in my fork: https://github.com/zynthian/mod-host/

@guysoft
Copy link

guysoft commented Jul 25, 2018 •
edited
Loading

Hey, updated this commit to be mergable again at #26 .
Would still value a merge :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jofemodo @SrMouraSilva @guysoft