8000 Added support for camera selection by DiTheRX · Pull Request #41 · Project-Babble/ProjectBabble · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Added support for camera selection #41

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

Merged
merged 3 commits into from
Aug 13, 2024

Conversation

DiTheRX
Copy link
Contributor
@DiTheRX DiTheRX commented Aug 9, 2024

No description provided.

@RamesTheGeneric
Copy link
Contributor
RamesTheGeneric commented Aug 12, 2024

Hey there, I have just tested your branch and really like this idea.
During my test I found a few small things.

in misc_utils.py
It seems FilterGraph() can only be called once so once a UVC source is selected, it crashes. I simply assigned graph the result of FIlterGraph at the top of the file and removed the assignments from the functions.

in camera_widget.py
On line 267 There is no check for "COM" or "/dev/tty" so when these serial ports are selected, "http://" is appended. Adding these checks allowed me to connect my serial cameras.

Can you please add these changes so it can be merged?

@DiTheRX
Copy link
Contributor Author
DiTheRX commented Aug 12, 2024

Hey there, I have just tested your branch and really like this idea. During my test I found a few small things.

in misc_utils.py It seems FilterGraph() can only be called once so once a UVC source is selected, it crashes. I simply assigned graph the result of FIlterGraph at the top of the file and removed the assignments from the functions.

in camera_widget.py On line 267 There is no check for "COM" or "/dev/tty" so when these serial ports are selected, "http://" is appended. Adding these checks allowed me to connect my serial cameras.

Can you please add these changes so it can be merged?

I have made these changes, please try again

Copy link
Contributor
@RamesTheGeneric RamesTheGeneric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything seems to work as expected.

@RamesTheGeneric RamesTheGeneric merged commit 67637fd into Project-Babble:main Aug 13, 2024
1 check passed
LordOfDragons added a commit to LordOfDragons/ProjectBabble that referenced this pull request Sep 13, 2024
This reverts commit 67637fd, reversing
changes made to 9f596eb.
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.

2 participants
0