Skip to content

moving the source message callback up a level, so it doesn't get repl… - #5538

Merged
avacreeth merged 2 commits into
vision_servicefrom
vision_service_fixed_source_message_callback
Aug 25, 2025
Merged

moving the source message callback up a level, so it doesn't get repl…#5538
avacreeth merged 2 commits into
vision_servicefrom
vision_service_fixed_source_message_callback

Conversation

@tmaneri

@tmaneri tmaneri commented Aug 25, 2025

Copy link
Copy Markdown
Contributor

…aced for each smart source

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Tom Maneri seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@tmaneri

tmaneri commented Aug 25, 2025

Copy link
Copy Markdown
Contributor Author

obs.NodeObs.RegisterSourceMessageCallback only supports a single callback, so each time a smart source loads it ends up overriding it, and preventing other smart sources from getting notified of changes

@avacreeth
avacreeth merged commit 4119369 into vision_service Aug 25, 2025
2 of 9 checks passed
@avacreeth
avacreeth deleted the vision_service_fixed_source_message_callback branch August 25, 2025 21:34
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.

3 participants