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

Make breadcrumb static after specified time #90

Merged
merged 13 commits into from
May 7, 2020
Merged

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Apr 24, 2020

Added <disable_physics_time> param to the breadcrumbs plugin, which allows users to make the breadcrumb model static after the specified time. The timer starts when the model is spawned into the world.

Note that the model is currently made static by attaching it to a static entity. We can switch to using an ignition-physics API for doing this when that feature is available.

Signed-off-by: Ian Chen [email protected]

@iche033 iche033 requested a review from azeey April 24, 2020 23:00
@iche033 iche033 self-assigned this Apr 27, 2020
Signed-off-by: Ian Chen <[email protected]>
Copy link
Contributor

@azeey azeey left a comment

Choose a reason for hiding this comment

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

LGTM! Just a minor comment

#include "ignition/gazebo/components/Geometry.hh"
#include "ignition/gazebo/components/Link.hh"
#include "ignition/gazebo/components/DetachableJoint.hh"
Copy link
Contributor

Choose a reason for hiding this comment

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

Duplicate

Suggested change
#include "ignition/gazebo/components/DetachableJoint.hh"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed. 1824e78

examples/worlds/breadcrumbs.sdf Show resolved Hide resolved
@nkoenig nkoenig force-pushed the breadcrumbs_exit_early branch 2 times, most recently from ded47ae to b8ac88b Compare May 6, 2020 22:21
@iche033 iche033 changed the base branch from breadcrumbs_exit_early to ign-gazebo2 May 7, 2020 00:16
Signed-off-by: Ian Chen <[email protected]>
@iche033 iche033 merged commit 2cc8e8f into ign-gazebo2 May 7, 2020
@iche033 iche033 deleted the breadcrumbs_static branch May 7, 2020 19:32
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.

3 participants