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

[GSOC-46] Fuel model usage and migration to ROS 2 launch files #2

Merged
merged 6 commits into from
Jun 20, 2024

Conversation

rakeshv24
Copy link
Collaborator

This PR introduces two features:

  • Fuel Model Integration Examples: We've added comprehensive examples demonstrating how to utilize models from Fuel, making it easier to incorporate these models into your projects.

  • ROS 2 Launch File Migration: Launch files have been updated to ROS 2, ensuring compatibility with the latest ROS version.


1. Set the resource path to your local model files:
```bash
export GZ_SIM_RESOURCE_PATH=<Path-to-dave_ws>/src/dave/dave_model_description

Choose a reason for hiding this comment

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

gazebosim/ros_gz#492 Could you try this to get rid of this step?

Copy link

@woensug-choi woensug-choi Jun 19, 2024

Choose a reason for hiding this comment

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

I've no experience with hooks but it seems it's another alternative
Reference: https://github.com/gazebosim/ros_gz/tree/ros2/ros_gz_sim_demos

Also... hard to ask, but with best wishes, it seems ros_gz's demo is now all in python. What do you think of working with python from this and now on? With power of chatgpt

Choose a reason for hiding this comment

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

All other parts worked fine! Thank you so much for investigating the previous dave structure and attempt to follow how it was done 👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing, I'll make those changes!

@rakeshv24
Copy link
Collaborator Author

@woensug-choi, I've incorporated your comments into the commit.

@rakeshv24 rakeshv24 requested a review from woensug-choi June 20, 2024 11:32
Copy link

@woensug-choi woensug-choi left a comment

Choose a reason for hiding this comment

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

So QUICK! Amazing..! 👍

  1. no need _bare_model for bare model. Will add _something as we add plugins or attachments (e.g. arms)

  2. I've treid universal upload_model.launch.py and model_in_empty_world.launch at https://github.com/IOES-Lab/dave/tree/fuel-models-universal.

ros2 launch dave_demos model_in_empty_world.py model_name:='nortek_dvl500_300'
ros2 launch dave_demos model_in_empty_world.py model_name:='mossy_cinder_block'

I think you can use git merge fuel-models-universal on your PR if things look ok (please test! and make mods if necessary). I haven't modified README for demos. you may commit on top of fuel-models-universal and merge to your PR.

  1. Any idea why Support <gazebo_ros> in package.xml exports gazebosim/ros_gz#492 didn't work? maybe it's not on release yet? added note on package.xml at dave_model_description

  2. There was a typo error at README. cd <path-to-dave_ws>/src/dave_model_description -> cd <path-to-dave_ws>/src/dave/dave_model_description

@rakeshv24
Copy link
Collaborator Author

rakeshv24 commented Jun 20, 2024

Awesome, thank you so for the quick and detailed review!

I like the idea of just using one universal launch file! I have merged the branch after making minor mods.

Any idea why gazebosim/ros_gz#492 didn't work? maybe it's not on release yet?

  • It seems likely that the pull request isn’t included in the release yet. For now, let's use the hook and revisit this once it becomes available or if I find a way to make it work with Jazzy.

@rakeshv24 rakeshv24 requested a review from woensug-choi June 20, 2024 16:13
Copy link

@woensug-choi woensug-choi left a comment

Choose a reason for hiding this comment

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

Great! Our first PR! Thank you!!! 👍
Take pleasure to push merge button

@rakeshv24 rakeshv24 merged commit 06b2c3a into ros2 Jun 20, 2024
@rakeshv24 rakeshv24 deleted the fuel-models branch June 20, 2024 20:59
@woensug-choi woensug-choi changed the title Fuel model usage and migration to ROS 2 launch files [WLT-46] Fuel model usage and migration to ROS 2 launch files Jun 20, 2024
@woensug-choi woensug-choi changed the title [WLT-46] Fuel model usage and migration to ROS 2 launch files [GSOC-46] Fuel model usage and migration to ROS 2 launch files Jun 20, 2024
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