-
Notifications
You must be signed in to change notification settings - Fork 2
Co2 shell #8
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
base: main
Are you sure you want to change the base?
Conversation
| @@ -0,0 +1,88 @@ | |||
| #!/usr/bin/env python3 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you convert these to 4 space indentation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry missed this now done in fee5a09
| return plume(x, y, z) | ||
|
|
||
|
|
||
| plume_base = plume_grid() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder, could place the plume in gazebo with some input variables from the CLI?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah any defined model you can place(spawn) in gazebo from the command line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You just gotta make sure it matches the virtual co2publisher or that we write something that looks at the current gazebo models
Also changed to just launching gazebo instead of a ros2 launch as the library arguments don't seem to be available in the ros2 launch. Note if there are multiple versions of ros/gazebo in the container this may cause issues

Starts gazebo in a mode(with libraries) so the spawner can be called.
Created a script that matches the virtual co2 publisher's plume misc/co2_plume_spawner.py
Sorry I had to open a new PR #7 would closed itself when was resting the branch one commit back and would not reopen.