-
Notifications
You must be signed in to change notification settings - Fork 8
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
Make special BELA pattern for bela-specific command line arguments. #21
Comments
…ds sensestage/supercollider/BelaPlatform#21.
I drafted this in giuliomoro@2fd434d it is NOT ready to merge, I did not check your default options and you may want to add a bit more logic to check valid combinations of values, requires quite some cleanup, but shows the workflow. Basically you can set defaults in Bad things: 1) I am using a global variable 2) The BelaDriver depends now on what happens in The "advantage" of this approach is that you do not have to implement all of Bela's command-line options every time one is added. Feel free to rearrange as needed (or dump completely) the code in my commit, this shows the path. |
I've started in this branch [1] with porting it, but there still seems to be an issue with starting scsynth from the web IDE. [1] https://github.com/sensestage/supercollider/commits/bela_hackery_v02-belaoptions |
there are peeps in the SC community thinking about integrating reading a |
Is it related to this?
#42
|
not really... there's no issue for this yet. but I can ask around. |
talked. |
Continues here. |
If we keep adding Bela options to SC we could quickly run out of letters.
We could think to redesign the SC Bela-related command line options so that they take something like
--bela "-C 8 --pga-left -9 -H 0"
so that we do not need to implement that on both sides AND we save letters.
The text was updated successfully, but these errors were encountered: