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.

Sign up for GitHub

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

Jump to bottom

Integrate buoy following with action server #168

Closed
toyat522 opened this issue Jan 22, 2025 · 1 comment · Fixed by #191
Closed

Integrate buoy following with action server #168

toyat522 opened this issue Jan 22, 2025 · 1 comment · Fixed by #191
Assignees
@quantum9Innovation @popkarsd @JanelleCai
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone
Fish&Ships: Integ...

Comments

@toyat522
Copy link
Contributor

No description provided.

@quantum9Innovation quantum9Innovation added enhancement New feature or request good first issue Good for newcomers labels Jan 22, 2025
@quantum9Innovation
Copy link
Member

Call the action server (navigation_server.py) from the waypoint finder (waypoint_finder.py). See rviz_waypoint_sender::send_path for an example implementation. Simulation testing should allow following buoys, but there may be underlying issues in either the navigation server or waypoint finder that result in unexpected behavior to investigate further.

@JanelleCai JanelleCai linked a pull request Feb 14, 2025 that will close this issue
buoy action server integration #191
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@quantum9Innovation quantum9Innovation

@popkarsd popkarsd

@JanelleCai JanelleCai

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
Fish&Ships: Integration Test
Development

Successfully merging a pull request may close this issue.

buoy action server integration ArcturusNavigation/all_seaing_vehicle
4 participants
@toyat522 @quantum9Innovation @popkarsd @JanelleCai