-
Notifications
You must be signed in to change notification settings - Fork 2
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
move some common-use components to swmain + further discussion #3
Comments
For the first part, I don't really think there needs to be any new code, right? Just find/create a tmux session based on a configurable name, then send a couple sets of commands, with a unique entrypoint script (the |
Gonna split into two issues. 1/ tmux components. Agree. But there are already several users using a fork of camstack and relying on this being standalone-ish (albeit already broken since the FG code has moved to a private repo already), so... The alternative is what we discussed this morning, submodule/subpackage camstack into swmain. Not great, not terrible. |
eh fair enough. I vote we put the appropriate code in |
Well if we go and merge camstack fully into swmain we can mark current camstack to be legacy and the people who want to keep up with camstack in the future clone the entire swmain? |
No regression in features through the merge but full power to reshuffle files. |
Yeah but it creates much more overhead inside swmain, like think of all the extra scripts and dependency management that comes with that. I do support somehow forking camstack so that we don't have to make decisions locked into the current architecture. Perhaps a |
Spitballing a couple ideas here, feel free to split into different issues if you like.
I think some of the scaffolding code for setting up the tmux daemons should be upstreamed to
software-main
(e.g., thecam-*start
scripts), that way it can be used for device control daemons. In doing this, I think we should make the scripts python, and perhaps could consider some different CLI approaches.For example, we could greatly simplify the
cam-*start
stack with an interface likecamstart <cam_name> [<mode>]
. This argument parsing is easy in python, and presumably the cameras can be modularized and listed out easily in python.Thoughts @DasVinch?
The text was updated successfully, but these errors were encountered: