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

Nav tube update #1184

Merged
merged 3 commits into from
Apr 7, 2024
Merged

Nav tube update #1184

merged 3 commits into from
Apr 7, 2024

Conversation

andrew-aj
Copy link
Member

Description

Adds in the code required to access depth data from the new nav tube.

Related Issues

- Closes #1123 #1121 and #1120

Testing

Tested code in lab to verify driver works and verified launch file works.

DaniParr
DaniParr previously approved these changes Apr 7, 2024
Copy link
Contributor

@DaniParr DaniParr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@uf-mil-bot
Copy link
Collaborator

uf-mil-bot commented Apr 7, 2024

Hola, your friendly InvestiGator bot here with another message!

Because this PR was closed/merged, I'm going to remove the docs preview for now.

Have a great day! Go gators! 🐊

Copy link
Contributor

@DaniParr DaniParr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@andrew-aj andrew-aj merged commit f6ea764 into master Apr 7, 2024
4 checks passed
@andrew-aj andrew-aj deleted the nav_tube_update branch April 7, 2024 18:35
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.

Sub9 Navigation Tube ROS Client
3 participants