Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Ashton Larkin ashton@openrobotics.org
🦟 Bug fix
Fixes #647
Summary
While working on fixing the crash from launching
actor.sdf
(see #647), I noticed that there are issues with other actor example worlds as well. It seems like there is an issue using an<include>
tag for actors in SDF (removing actor-related<include>
s from the SDF resolves crashes on launch). I'm not sure when this became a problem, or why it is a problem. I need to look into this further (I haven't been involved with actor-related development for Ignition), so I've added a list of things I plan to investigate below.TODO:
<include>
tags became an issue (I need to look through the history/past PRs. One PR that I found so far that may be related is More actor components and follow plugin #157)<include>
scenarios if this test case hasn't been handled yetChecklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge