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

TODO: switch to official SICK drivers #35

Closed
moriarty opened this issue Feb 27, 2019 · 2 comments
Closed

TODO: switch to official SICK drivers #35

moriarty opened this issue Feb 27, 2019 · 2 comments

Comments

@moriarty
Copy link
Contributor

We depend on ros-melodic-sick-tim package from https://github.com/uos/sick_tim/ which for the purpose of the upcoming release to melodic ZebraDevs/fetch_ros#63 is fine.

But we should switch over to the official drivers: https://github.com/SICKAG/sick_scan which is ros-melodic-sick-scan at some point in the future.

This won't be for the initial ros-melodic release, because we want that to get out with the next melodic sync and won't have time for the additional tests.

@chadrockey are you aware of any major differences?

@chadrockey
Copy link

No, but I haven't looked into the TIM stuff in forever.

I'll look into this in the coming weeks.

@moriarty
Copy link
Contributor Author

I'm going to close this now and open a ticket internally as it's not an issue, the ros-melodic-sick-tim package is working now.

@rctoris the original request came via an email.

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

No branches or pull requests

2 participants