gazebosim / gazebosim/sdformat
Possible to preserve `//include/@merge` for output SDFormat files?
- Dominant language
- C++
- Stars
- 216
- Forks
- 125
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 14
Description
## Desired behavior
Per #749 and VC discussion, `//include/@merge=true` will not be preserved round-trip.
Would be nice to have it.
## Alternatives considered
N/A
## Implementation suggestion
Annotate included elements w/ stack of indicators about which `//include/@merge` element produced it.
Upon emitting the elements, remove them, and ensure original `//include` element is preserved.
## Additional context
\cc @azeey @jennuine
Contributor guide
Research direction
Start by reviewing issue #749 and the referenced VC discussion, then trace how included elements are expanded and how output SDFormat files are emitted. Done means `//include/@merge=true` survives a round trip while the resulting output remains valid SDFormat.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100