Skip to content

Address REP-158 peer review - #30

Merged
gbiggs merged 2 commits into
openrobotics:mainfrom
zakmat:mzak/adjust-158
Jul 13, 2026
Merged

Address REP-158 peer review#30
gbiggs merged 2 commits into
openrobotics:mainfrom
zakmat:mzak/adjust-158

Conversation

@zakmat

@zakmat zakmat commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

This PR follow #29 and is addressing the community feedback from https://discourse.openrobotics.org/t/draft-rep-158-openusd-conventions-for-simulation-asset-interoperability-in-open-source-robotics/55526

The initial batch tackles the following items:

  • This PR adjusts the REP to not introduce the ext_physics incubation layers. Instead it adheres to AOUSD ratification process for the Physics attributes.
  • With the above change explicit degradation reporting is even more important. This PR adds requirement for explicit signalling in case the backend does not support property or vendor layer or when it drops the fidelity

* remove the ext_physics incubation namespace.
* suggest rep_sanitizer to maintain cross-vendor dialect mappings
* make degradation reporting normative requirement

Signed-off-by: Mateusz Żak <mateusz.zak@robotec.ai>
@zakmat
zakmat requested review from gbiggs and tfoote as code owners July 13, 2026 10:30
Signed-off-by: Mateusz Żak <mateusz.zak@robotec.ai>
@zakmat zakmat changed the title Drop ext_physics namespace in favor of vendor layers Address REP-158 peer review Jul 13, 2026
@gbiggs
gbiggs merged commit b56cd16 into openrobotics:main Jul 13, 2026
1 check passed
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.

2 participants