Fix UsdGeomPointInstancer ComputeExtents when prototypes are under an over

cbillingham commented 3 weeks ago

Description of Change(s)

Allow UsdGeomBBox to take in a custom prim traversal predicate so that UsdGeomPointInstancer can compute bboxes of prototypes that exist under an over as described by the docs

Fixes Issue(s)

https://github.com/PixarAnimationStudios/OpenUSD/issues/3395

Checklist

[x] I have created this PR based on the dev branch
[x] I have followed the coding conventions
[x] I have added unit tests that exercise this functionality (Reference: testing guidelines)
[x] I have verified that all unit tests pass with the proposed changes
[x] I have submitted a signed Contributor License Agreement (Reference: Contributor License Agreement instructions)

cbillingham commented 3 weeks ago

I took a stab at addressing this, but let me know if I went down the the wrong path

jesschimein commented 3 weeks ago

Filed as internal issue #USD-10392

:heavy_exclamation_mark: Please make sure that a signed CLA has been submitted!

jesschimein commented 3 weeks ago

/AzurePipelines run

azure-pipelines[bot] commented 3 weeks ago

Azure Pipelines successfully started running 1 pipeline(s).

cbillingham commented 3 weeks ago

Filed as internal issue #USD-10392

❗ Please make sure that a signed CLA has been submitted!

I should have one sent in Feb 2021. Let me know if I need to send in a new one

pixar-oss commented 3 weeks ago

Thank you for flagging that! We have your CLA on file; so sorry for the confusion!

PixarAnimationStudios / OpenUSD