Fixed selection highlighting removed on Maya object by isolate select. #198
Conversation
| @@ -478,7 +478,10 @@ class MayaPathMapper : public Fvp::PathMapper | |||
|
|
|||
| Fvp::PrimSelections | |||
| UfePathToPrimSelections(const Ufe::Path& appPath) const override { | |||
| return _piSi.UfePathToPrimSelectionsLit(appPath); | |||
| auto litPaths = _piSi.UfePathToPrimSelectionsLit(appPath); | |||
There was a problem hiding this comment.
Awkwardly, when selected, Maya objects have geometry under two Hydra prim hierarchies, with selection highlighting geometry (poly wire) rooted under /MayaHydraViewportRenderer/rprims, and the real mesh geometry under /MayaHydraViewportRenderer/rprims/Lighted. Both must be considered "selected" for isolate select to preserve them in the scene, otherwise the unlit selection highlighting geometry will be hidden.
There was a problem hiding this comment.
I recall we have a tech debt to remove /Lighted for lit prims, something worth a look later
There was a problem hiding this comment.
Without the fix, the test fails. With the fix, the test passes.
| cmds.mayaHydraCppTest('/MayaHydraViewportRenderer/rprims/pTorus1', | ||
| f='TestPrimPath.isVisible'); | ||
| cmds.mayaHydraCppTest('/MayaHydraViewportRenderer/rprims/Lighted/pTorus1', | ||
| f='TestPrimPath.isVisible'); |
There was a problem hiding this comment.
Both the selection highlighting and the real geometry are visible under isolate select.
| f='TestPrimPath.isVisible'); | ||
| cmds.mayaHydraCppTest('/MayaHydraViewportRenderer/rprims/Lighted/pTorus1', | ||
| f='TestPrimPath.isVisible'); | ||
| cmds.mayaHydraCppTest('/MayaHydraViewportRenderer/rprims/Lighted/group1', |
There was a problem hiding this comment.
Geometry that is not isolate selected is not visible.
There was a problem hiding this comment.
Add a testing utility to receive paths directly as Hydra prim paths, not application UFE paths.
ppt-adsk
added
the
ready-for-merge
lilike-adsk
deleted the
tremblp/HYDRA-1181/isolate_select_vs_Maya_selection_highlighting
branch
No description provided.