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

Revert upstream changes to to run initialization on a separate thread #29

Open
wants to merge 1 commit into
base: por_upstream_sync
Choose a base branch
from

Conversation

malban
Copy link

@malban malban commented Sep 16, 2020

Revert upstream changes to to run initialization on a separate thread, instead have the driver unload itself or shutdown if no device is found.

This will rely on roslaunch respawn to accomplish polling for the device if it isn't present. I think this is a bit simpler and avoids needing a special thread for initialization.

…. Instead have the driver unload itself or shutdown if no device is found.
@130s
Copy link

130s commented Nov 4, 2020

As described in #31, POR is moving toward more up-to-date Realsense ROS wrapper. I suggest to 1) re-target to por_X_2020 (forgot the branch name), 2) test this PR.

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