-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Globe anchor support #453
Globe anchor support #453
Conversation
@weegeekps orientations are all looking good in my tests! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Architecturally everything feels well thought out and fits in nicely. I do want to walk through some of the more complex areas offline just so I confirm my understanding.
I also haven't tried running it yet, but I wanted to get this first pass review out.
exts/cesium.usd.plugins/cesium/usd/plugins/CesiumUsdSchemas/__init__.pyi
Outdated
Show resolved
Hide resolved
exts/cesium.omniverse/cesium/omniverse/ui/attributes/globe_anchor_attributes_widget.py
Outdated
Show resolved
Hide resolved
exts/cesium.omniverse/cesium/omniverse/ui/attributes_widget_controller.py
Show resolved
Hide resolved
exts/cesium.usd.plugins/cesium/usd/plugins/CesiumUsdSchemas/_cesiumUsdSchemas.so
Outdated
Show resolved
Hide resolved
@@ -754,4 +831,23 @@ RenderStatistics Context::getRenderStatistics() const { | |||
return renderStatistics; | |||
} | |||
|
|||
void Context::addGlobeAnchorToPrim(const pxr::SdfPath& path) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From offline discussion: are globe anchors added to the registry when loaded from a file? Do we need to watch for globe anchors in processPrimAdded
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed during stand-up, I will be fixing this in a subsequent PR. I opened #489 for this.
@lilleyse I'm ready for you to take a second pass at this when you're able. In the morning I'm going to fix the things I left untouched today. |
Resolves #235.
This PR implements Globe Anchor support for any Xformable prims via a custom API schema. When a user adds a Globe Anchor to a prim, a new transform op with a
cesium
suffix is added. This transform is used by the anchoring code to place the object in the USD stage. User's can use the transformation widgets as normal, or can set an exact Geographic Coordinate using either Lat/Long/Alt or ECEF. Updating any one of these values updates all of the others.NOTE: I plan on squashing and merging this PR due to the sheer number of commits, but I'm leaving them here so we can keep the history.
2023-10-04.17-38-22.mp4