adapt to UsdImagingDelegate::PopulateSelection param change #345
Conversation
|
@mattyjams We missed our internal update of USD by a few days. I don't believe there is any rush in this PR, so we can keep it open for a week or two? |
… index path With USD_IMAGING_API_VERSION 11, PopulateSelection() expects to be given a USD path instead of a Hydra "index path". This helps to avoid some confusion about the types of paths being passed around. This change corresponds to core USD commit 760904c80d8c38d83a4fca079d2414b8288e22cf. (Internal change: 2046886)
This identifies core USD commit 893c8e4 (post 20.02 release) as the most recent supported version of USD.
mattyjams
force-pushed
the
pr/usdImaging_PopulateSelection_usdPath_param
branch
from
ef9af3b to
fb155cb
Compare
|
@kxl-adsk This change shouldn't require the latest version of USD, as the "new" code is gated on I just rebased it following the merge of #346. |
|
How often USD_IMAGING_API_VERSION is updated? At every breaking change? |
|
I'm not quite sure exactly how rigorous we're being about that, but at least for this particular change, the API version was bumped for this specific API change. The core USD commit was PixarAnimationStudios/OpenUSD@760904c. |
|
Roger that. Then we should be able to get it in sooner. Thank you for clarification @mattyjams |
…ace instead of free functions (#345) * MAYA-127453: combine util.cpp/h and utils.cpp/h; add mayahydra namespace instead of free functions * MAYA-127453: remove using ns
With
USD_IMAGING_API_VERSION11,UsdImagingDelegate::PopulateSelection()expects to be given a USD path instead of a Hydra "index path". This helps to avoid some confusion aboutthe types of paths being passed around.