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

Hydrothermal vent #1187

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

Hydrothermal vent #1187

wants to merge 11 commits into from

Conversation

tjcanro
Copy link

@tjcanro tjcanro commented Apr 11, 2024

Description

I made the hydrothermal vent mission, along with the gazebo 3D modeling of the buoy for testing purposes. The mission consists on circumnavigating a red buoy. I also made it so the sub returns to origin after completing the mission

Screenshot or Video

Screenshot from 2024-04-11 11-25-50
Screenshot from 2024-04-11 11-26-17
Screenshot from 2024-04-11 11-26-53
Screenshot from 2024-04-11 11-27-27

Related Issues

- Closes #1143

Testing

run the following commands:
$ roslaunch subjugator_launch gazebo.launch --screen
$ gazebogui
$ amonitor kill
$ mission run Hydrothermal_vent

About This PR

  • I have updated documentation related to this change so that future members are aware of the changes I've made.

Comments:

The mission uses a hard-coded value for the buoy position. Once computer vision has the red buoy detection ready, it can be implemented in this mission

Copy link
Member

@cbrxyz cbrxyz left a comment

Choose a reason for hiding this comment

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

Thank you! (also, I updated your branch to match master)

# Computer vision aspect still required to detect the red buoy. For now, random position used
class HydrothermalVent(SubjuGatorMission):
async def run(self, args):
self.buoy_pos = np.array([3, 4, -2]) # Convert buoy position to numpy array
Copy link
Member

Choose a reason for hiding this comment

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

What is this array?

Copy link
Author

Choose a reason for hiding this comment

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

The array represents the position of the buoy in x, y, z coordinates relative to the sub. I spoke to Daniel about how the sub would actually view the buoy and I will now work on modifying the functionality so it uses position in x,y relative to the center of the camera and radius/

@uf-mil-bot
Copy link
Collaborator

Hello, it's your friendly InvestiGator bot here!

The docs preview for this PR is available at https://mil.ufl.edu/pr-docs/1187.

Last updated at:f444f936b73c0ca2d5da8f3d5add929c3912fca1
► 2024-04-16 14:51 EDT

Have a great day! Go gators! 🐊

@DaniParr DaniParr marked this pull request as draft April 16, 2024 22:13
@cbrxyz cbrxyz mentioned this pull request Jun 22, 2024
1 task
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.

Robosub 2024: Hydrothermal Vent | Buoy
3 participants