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.

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

USB Cameras Going Rogue #27

Open
flynn248 opened this issue Oct 13, 2021 · 1 comment
Open

USB Cameras Going Rogue #27

flynn248 opened this issue Oct 13, 2021 · 1 comment
Assignees
Labels
bug Something isn't working development foundational enhancement

Comments

@flynn248
Copy link
Collaborator

flynn248 commented Oct 13, 2021

Background Info

Currently the PID's of each device is saved in a file in the logs/ directory. This is then read by stop_data_collection.sh to kill these devices.

Description

For some reason the PID saved in the file for one of the USB cameras does not correlate to the PID outputted by
$ sudo fuser /dev/video{port}
This results in the camera not being killed successfully. Then when the devices are started up again, the ROS script that runs the camera fails because it cannot open another stream for that camera since it still exists.

Currently each script's output is sent to the logs/ directory and not to the terminal. There are currently no automatic checks to see if a device truly booted up successfully.

Possible Solutions

  1. Develop a more robust system that will inform the user right away if a device did not boot up successfully. Should be done anyways.
  2. When stopping the devices, add extra checks for the USB cameras to see if they were successfully killed. If not, then forcefully kill them using
    $ sudo fuser /dev/video{port}
@flynn248 flynn248 self-assigned this Oct 13, 2021
@flynn248 flynn248 added bug Something isn't working development foundational enhancement labels Oct 13, 2021
@flynn248
Copy link
Collaborator Author

flynn248 commented Oct 13, 2021

Currently implemented solution 2. Added an extra check to see if the USB camera did in fact get killed in stop_data_collection.sh.

This has not been tested fully. But on some quick tests, it should be functioning properly.

Solution 1 should also be worked on, but may be put into a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working development foundational enhancement
Projects
None yet
Development

No branches or pull requests

1 participant