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.

Sign up for GitHub

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

Jump to bottom

CameraServer: LabVIEW Dashboard is not compatible with OpenCV sources #407

Closed
PeterJohnson opened this issue Dec 18, 2016 · 3 comments · Fixed by #1412
Closed

CameraServer: LabVIEW Dashboard is not compatible with OpenCV sources #407

PeterJohnson opened this issue Dec 18, 2016 · 3 comments · Fixed by #1412
Labels
bug in dependency component: cscore CameraServer library type: bug Something isn't working.
Projects
2020 CameraServer

Comments

@PeterJohnson
Copy link
Member

The current LabVIEW dashboard (Beta 4) requires the source type to be either usb: or ip: and does not support cv:. To work around this, it's necessary to use a source type of usb: for OpenCV sources as well.
@PeterJohnson PeterJohnson mentioned this issue Dec 18, 2016
Merged
@333fred 333fred added bug in dependency type: bug Something isn't working. labels Dec 18, 2016
PeterJohnson added a commit that referenced this issue Dec 18, 2016
@PeterJohnson
CameraServer: Workaround LabVIEW dashboard OpenCV compatibility. (#406)
0ee4cad 
The current LabVIEW dashboard (Beta 4) requires the source type to be either
"usb:" or "ip:" and does not support "cv:".  To work around this, use a source
type of "usb:" for OpenCV sources as well.

Workaround for #407.
@Kevin-OConnor
Copy link
Contributor

This should be fixed in the 2018 Kickoff release

@PeterJohnson PeterJohnson added the component: cscore CameraServer library label Apr 30, 2018
@ThadHouse
Copy link
Member

Was this ever tested to be fixed? I know we still have the shim in CameraServer to work around this.

@Kevin-OConnor
Copy link
Contributor

This is definitely fixed on the Dashboard side. We should be able to remove the shim if we want to.

@PeterJohnson PeterJohnson mentioned this issue Nov 7, 2018
Merged
2020 CameraServer automation moved this from To do to Done Nov 9, 2018
pjbuterbaugh pushed a commit to pjbuterbaugh/allwpilib that referenced this issue Jun 15, 2023
@Daltz333 @AustinShalit
don't recommend IE (wpilibsuite#407)
e79ee81
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug in dependency component: cscore CameraServer library type: bug Something isn't working.
Projects
No open projects
2020 CameraServer
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

CameraServer: Change opencv sources to publish "cv:" type PeterJohnson/allwpilib
4 participants
@PeterJohnson @333fred @ThadHouse @Kevin-OConnor