Use the nearestToCamera mode in HdxIntersector

pmolodo commented 5 years ago

Description (this won't be part of the changelog)

By making use of this new mode, we can significantly simplify some of the selection code. Should also be slightly more efficient, as we avoid doing any processing or transfer of hits we won't actually end up using / needing.

Note that this PR branches off of https://github.com/AnimalLogic/AL_USDMaya/pull/150, as it relies on changes made in usd-0.19.5.

Changelog

Changed

Make selection using HdxIntersector's nearestToCamera mode, instead of doing the work ourselves

Checklist (Please do not remove this line)

[X] Make sure the Title and Description of the PR make sense and provide sufficient context for your work
[X] Do any added files have the correct AL Apache Licence Header?
[X] Are there Doxygen comments in the headers?
[X] Are any new features, behaviour changes documented in the .md format documentation?
[ ] Have you added, updated tests to cover new features and behaviour changes?
[X] Have you filled out at least one changelog entry?
[X] Is the branch's history clean? (only relevant commits)

murphyeoin commented 5 years ago

Hi @elrond79 if we merge as is, we'll be forcing everyone to update to 19.05 to use it. Adding guards to the code to enable both options kind of defeats the purpose of it though.... What are your thoughts?

pmolodo commented 5 years ago

Yeah, my thinking was just that that we'd wait on this until after we merged in https://github.com/AnimalLogic/AL_USDMaya/pull/150, which it branches off of, and we've made 19.05 the official build requirement.

murphyeoin commented 5 years ago

@elrond79 sounds like a good plan

AnimalLogic / AL_USDMaya