gazebosim / gazebosim/sdformat

Possible to preserve `//include/@merge` for output SDFormat files?

Open
#769 0 comments 0 reactions 0 assignees View on GitHub
enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.