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.

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

cvOSCcv: restore connection #29

Closed
wants to merge 1 commit into from
Closed

Conversation

jfrey-xx
Copy link

Hello,

I am trying to find ways to command VCV Rack from outside, without the need to interact with the GUI. Your OSC module might just be what I need, however there is one inconvenience: upon start I still have to "enable" each module manually.

Hence here is a proposal to save the last state of the module (connection up or not) and restore this state when a patch is loaded.

@chichian
Copy link
Collaborator

Hi jfrey-xx,

We think this is a good addition, however some users may not want this. How do you feel if this was a setting on cvOSCcv (Auto-Reconnect On Load)?

We would make it pretty much the same, just an extra bool stored in the json and if true and it was enabled, then it would connect. I would have to add some place on the module or in context menu to set this, but that can be done.

@jfrey-xx
Copy link
Author

Indeed there might be some cases where one might not want to open too many ports at once, even thought it is about restoring a previous state. Actually I considered adding a specific option to let users device... but that was beyond my reach, I did not want to touch the UI. If you can find a spot for a checkbox that would be perfect :)

@chichian
Copy link
Collaborator

Hi.

We've added a little button for this in the OSC configuration screen in the latest release v0.6.3.

Thanks!

@jfrey-xx
Copy link
Author

Thank you for the feature, it works perfectly :)

@jfrey-xx jfrey-xx closed this Jun 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants