ability to only run specified Stream Decks instead of all attached decks by violet4 · Pull Request #227 · StreamController/StreamController · GitHub
Skip to content

ability to only run specified Stream Decks instead of all attached decks#227

Draft
violet4 wants to merge 2 commits into
StreamController:mainfrom
violet4:only_attach_specified_decks
Draft

ability to only run specified Stream Decks instead of all attached decks#227
violet4 wants to merge 2 commits into
StreamController:mainfrom
violet4:only_attach_specified_decks

Conversation

@violet4

@violet4 violet4 commented Sep 9, 2024

Copy link
Copy Markdown

this is achieved by adding --list and --paths command line arguments.

  • --list lists all the attached Stream Deck devices with some basic information, including their "paths", and exits
  • --paths takes 1 or more paths (found via --list) and filters out paths not specified before initializing the devices

if anyone cares about this change enough to want to merge it, but it's terrible or doesn't match the project requirements, please point me in the correct direction and i'd be happy to improve it and make it acceptable. however, at the moment, this makes it so i can use this software with one of my stream decks and my own program with my other stream deck simultaneously.

…cks. achieved by adding `--list` and `--paths` command line arguments.

`--list` lists all the attached	Stream Deck devices with some basic information, including their "paths", and exits
`--paths` takes	1 or more paths	(found via `--list`) and filters out paths not specified before	initializing the devices
@Core447

Core447 commented Sep 9, 2024

Copy link
Copy Markdown
Member

@violet4

violet4 commented Sep 16, 2024

Copy link
Copy Markdown
Author

tl;dr go ahead and close this PR when you see my message and i'll eventually submit a better one that handles more cases.

also my PR doesn't work so great if the externally managed stream deck is connected while StreamController is already running, since StreamController seems to watch new stream deck connections and attach to them automatically.

so clearly, this naïve/lazy PR isn't going to work for most use cases and i'm gonna have to put some more work into making it more robust and generally acceptable.

i'm not in any rush to make this super fully integrated and handle all cases, but i'm thinking this PR should be closed and i can use my own branch for a while over the course of a period of time and make small fixes until it's been a while since i've seen any issues. i can continuously rebase from this repo so it stays in sync and eventually make a PR when i've fixed all (currently unhandled) cases i've encountered (plus the one you mentioned).

i'll let you see this message, feel free to close the PR and i'll (slowly) work on improving it on my branch. at the moment my main concern is ensuring compatibility with my own awful stream deck implementation that i depend upon daily until i get the functionality moved over to StreamController.

((for example, i'll have to learn how the plugin system works so i can port my cute little stream deck calculator app.))

thank you for your time and patience!

@violet4

violet4 commented Sep 16, 2024

Copy link
Copy Markdown
Author

also i created a ticket for myself to keep track of specific needed fixes violet4#2

@Core447

Core447 commented Sep 18, 2024

Copy link
Copy Markdown
Member

@Core447 Core447 marked this pull request as draft September 18, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants