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

Workspace with the Gazebo World #30

Merged
merged 29 commits into from
Aug 22, 2024
Merged

Conversation

YuZhong-Chen
Copy link
Collaborator

Achieving the same objective as in #6, but incorporating two additional worlds.

In contrast to the previous pull request, I've opted not to use the git submodule. This decision stems from the potential need to modify the world file in future projects, such as incorporating additional plugins.

@YuZhong-Chen YuZhong-Chen requested a review from j3soon May 3, 2024 04:30
@YuZhong-Chen YuZhong-Chen self-assigned this May 3, 2024
Copy link
Owner

@j3soon j3soon left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Most changes look good to me, but I'm curious about the decision on using Google Drive (commits 6150ccf, 4564568, 8659a83) instead of tracking the files in Git.

I prefer tracking files in Git unless it would cause significant issues, like large files making Git clones excessively slow.

The total file size here seems to be around a few hundred MB when compressed. I think we can simply track them in Git?

@YuZhong-Chen
Copy link
Collaborator Author

YuZhong-Chen commented Aug 13, 2024

Tracking models in Git seems a bit unusual since it implies tracking model weights or something similar, which we typically wouldn't modify manually. However, I'm not too keen on using Google Drive either. I might want to try the method in this repository ( assets.zip ) in the future. What do you think?

If we want to revert to using Git to track the model, we just need to delete a few commits and make some minor adjustments. It's not a difficult task. I can make the change now, and we can explore it further when we have more time or opportunity in the future.

@j3soon
Copy link
Owner

j3soon commented Aug 13, 2024

Yeah I think we can revert to track models in Git for now. Thank you!

We can explore better methods in the future. (further discussions in #42)

This package is designed for opening Gazebo.
…in citysim.

The original repository is not designed for ROS, so the package structure differs from a typical package.
To ensure Gazebo's satisfaction, remove the plugin from the world file, as we won't be building those plugins.
…ts.txt in turtlebot3_gazebo.

As we solely require the world file and won't use the TurtleBot3, it's likely unnecessary to build the TurtleBot3 driver.
…ital_world.

Since we have downloaded the model and track the model by Git, there's no need to execute the download program in the package build process.
…gzclient.

If gzclient is not launched, it will run in headless mode without opening the GUI. This is useful when running the code on a server or when the GUI is not needed.
Allow the user to select the TurtleBot3 world file when launching the simulation.
@YuZhong-Chen YuZhong-Chen requested a review from j3soon August 21, 2024 20:21
@YuZhong-Chen
Copy link
Collaborator Author

YuZhong-Chen commented Aug 21, 2024

Since workspaces like husky_ws and kobuki_ws need to be updated to the latest template_ws format, I have made and tested some modifications for husky_ws separately in the gazebo_world_ws branch. I’m requesting a review for this PR first and will open other PRs for husky_ws, kobuki_ws, and others shortly afterward.

The source `clearpath_playpen.world` file retrieved by `wget` has a trailing newline. Just a small modification to keep them in sync.
@j3soon j3soon self-requested a review August 22, 2024 12:08
Copy link
Owner

@j3soon j3soon left a comment

Choose a reason for hiding this comment

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

Tested most of the worlds on my machine, all of them works well. Thank you for opening this PR!

@j3soon j3soon merged commit f572b54 into j3soon:main Aug 22, 2024
@YuZhong-Chen YuZhong-Chen deleted the gazebo_world_ws branch August 22, 2024 12:29
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