CameraServer doesn't seem to work with 2016 default dashboard #172

Closed
virtuald opened this Issue Jan 11, 2016 · 3 comments

Projects

None yet

1 participant

@virtuald
Member

I just tried out the USBCamera class RobotPy 2016.0.0 with the default LabVIEW dashboard, and it doesn't work. Looking at tcpdump, the RoboRIO seems to be rejecting the incoming connections for some reason.

As @PeterJohnson mentioned, it also won't connect to a NetworkTables 2.0 client, and so that might have something to do with it.

If I can't get this resolved relatively quickly, then I'm not going to waste time on this, and remove it from RobotPy until someone else decides to fix it. The performance was terrible when we tried it last year, and OpenCV is a better solution.

@virtuald
Member

Ok, so the real problem is that pynivision wasn't installed, and it didn't warn me that it was using the dummy camera.

@virtuald virtuald self-assigned this Jan 11, 2016
@virtuald virtuald added this to the 2016.1.0 milestone Jan 11, 2016
@virtuald
Member

When I install pynivision, I get a "Connection reset by peer" error. The problem is that when the LabVIEW dashboard connects to the server and it encounters the wrong version of networktables (see robotpy/pynetworktables#21), it disconnects from the camera. When I block the NetworkTables connection, then the camera works fine.

It's pretty terrible, the CPU usage is at 51% without the robot code actually doing anything (as compared to ~7% without the camera code). I don't really want to spend time optimizing + fixing it, so I think my recommendation is that we delete the camera code for now, and focus on providing a more python friendly solution via mjpg-streamer or some such similar thing.

@virtuald virtuald modified the milestone: 2016.2.0, 2016.1.0 Jan 21, 2016
@virtuald
Member

For those who want to use this, you can install the alpha version of pynetworktables:

pip install -U --pre pynetworktables
@virtuald virtuald added a commit that closed this issue Nov 21, 2016
@virtuald virtuald Remove pynivision + related pieces
- Fixes #172
669410c
@virtuald virtuald closed this in 669410c Nov 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment