Skip to content

Conversation

@cvolkcvolk
Copy link
Collaborator

@cvolkcvolk cvolkcvolk commented Jan 27, 2026

Summary

ObjectReference can be used in object placement (as Anchor Objects).

  • Provide ObjectReference with the required fields for it to be able to be used in the Object Placement

  • ObjectReference points to a sub-prim inside the parent's USD, not a separate USD file. Adds a function compute_local_bounding_box_from_prim() to get the bbox of the specific sub-prim &
    apply the parent asset's scale to the resulting bounding box.

  • Helper function to center, translate and scale AxisAlignedBoundingBox, removing some repetitive code.

  • Moved following functions to ObjectBase as they can be shared among Object and ObjectReference.

    def get_relations(self) -> list[RelationBase]:
    def get_spatial_relations(self) -> list[RelationBase]:

@cvolkcvolk cvolkcvolk changed the base branch from release/0.1.1 to main January 27, 2026 13:13
@cvolkcvolk cvolkcvolk marked this pull request as ready for review January 28, 2026 17:31
@cvolkcvolk cvolkcvolk marked this pull request as draft January 29, 2026 08:49
@cvolkcvolk cvolkcvolk marked this pull request as ready for review January 29, 2026 08:51
@cvolkcvolk cvolkcvolk marked this pull request as draft January 29, 2026 10:26
Copy link
Collaborator

@viiik-inside viiik-inside left a comment

Choose a reason for hiding this comment

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

Looks amazing. Just some minor suggestions from my side.

@cvolkcvolk cvolkcvolk marked this pull request as ready for review January 29, 2026 13:54
Copy link
Collaborator

@alexmillane alexmillane left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks for doing that!

@cvolkcvolk cvolkcvolk merged commit f3962a2 into main Jan 30, 2026
5 checks 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.

4 participants