Skip to content

Further clarify TRS properties in the Object Model#2560

Merged
lexaknyazev merged 1 commit intoKhronosGroup:mainfrom
lexaknyazev:object-model-trs
Feb 3, 2026
Merged

Further clarify TRS properties in the Object Model#2560
lexaknyazev merged 1 commit intoKhronosGroup:mainfrom
lexaknyazev:object-model-trs

Conversation

@lexaknyazev
Copy link
Member

@lexaknyazev lexaknyazev commented Feb 3, 2026

Follow up to #2559 with more explicit language.

/cc @hybridherbst

@lexaknyazev lexaknyazev requested a review from javagl February 3, 2026 17:26
javagl
javagl approved these changes Feb 3, 2026
View reviewed changes
Copy link
Contributor

@javagl javagl left a comment •
edited
Loading

Choose a reason for hiding this comment

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

When I first read this, I added a comment:

~Why does it not also mention translation?

Then I read the 'Rationale' below, and deleted that comment.

So yes, mentioning translation here immediately can avoid a short moment of confusion.

@lexaknyazev lexaknyazev merged commit f966a3f into KhronosGroup:main Feb 3, 2026
2 checks passed
@lexaknyazev lexaknyazev deleted the object-model-trs branch February 3, 2026 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@javagl javagl javagl approved these changes

+1 more reviewer

@aaronfranke aaronfranke aaronfranke approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lexaknyazev @aaronfranke @javagl