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

Make special BELA pattern for bela-specific command line arguments. #21

Closed
sensestage opened this issue Oct 6, 2016 · 7 comments
Closed

Comments

@sensestage
Copy link

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.

giuliomoro added a commit to giuliomoro/supercollider that referenced this issue Oct 9, 2016
@giuliomoro
Copy link

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 main(), then pass the command line options straight to Bela's parser, then override/check those values again.

Bad things: 1) I am using a global variable 2) The BelaDriver depends now on what happens in main().

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.
The disadvantage is that you depend more on the command line options assigned by Bela, which are slightly more likely to change meaning than the fields of the gBelaSettings structure.
As usual, the behaviour of SC heavily depends on the current version of the Bela files. At some point it may be worth adding a Bela subrepo with a tagged version to the SC repo .

Feel free to rearrange as needed (or dump completely) the code in my commit, this shows the path.

@sensestage
Copy link
Author

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

@LFSaw
Copy link

LFSaw commented Aug 9, 2017

there are peeps in the SC community thinking about integrating reading a config.yaml to scsynth. also to replace the (crude) commandline parsing by using a lib... I guess that's help a lot for this issue as well.

@giuliomoro
Copy link

giuliomoro commented Aug 9, 2017 via email

@LFSaw
Copy link

LFSaw commented Aug 9, 2017

not really... there's no issue for this yet. but I can ask around.

@LFSaw
Copy link

LFSaw commented Aug 9, 2017

talked.
here are stub issues, will be filled in detail later-ish.
But I'd definitely wait before fiddling with the scsynth cli interface, and take part in these discussions.

supercollider#3117
supercollider#3118

@giuliomoro
Copy link

Continues here.

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

No branches or pull requests

3 participants