Replace boost hash usage with TfHash in pxr/usd/usd

[nvmkuruc]

Description of Change(s)

Requires #2173
To remove the dependency of pxr/usd/usd on boost's hashing functions

• boost::hash_combine is replaced with TfHash::Combine, delegating hashing of containers instead of iterating
• boost::hash is replaced with TfHash
• Hashing test coverage has been extended to cover UsdStageLoadRules, UsdObject derived classes, and UsdCollectionMembershipQuery.

Fixes Issue(s)

-#2172 (additional PRs forthcoming)

• I have verified that all unit tests pass with the proposed changes

• I have submitted a signed Contributor License Agreement

[spiffmon]

@nvmkuruc , it looks like all the dependent/upstream files got pulled in, indavertently?

[nvmkuruc]

@nvmkuruc , it looks like all the dependent/upstream files got pulled in, indavertently?

I asked Nick about this and he said that it should be fine and resolve itself as the upstream dependency branches got pulled in, and I should just note in the pull request what the PR was dependent on. If there's a better way to have organized the stacking of PRs to avoid this, happy to reorganize.

[nvmkuruc]

@nvmkuruc , it looks like all the dependent/upstream files got pulled in, indavertently?

I asked Nick about this and he said that it should be fine and resolve itself as the upstream dependency branches got pulled in, and I should just note in the pull request what the PR was dependent on. If there's a better way to have organized the stacking of PRs to avoid this, happy to reorganize.

I think I can take a pass at making most of these PRs dependent on just the pxr/base/tf changes which should hopefully make these easier to look at.

[nvmkuruc]

I think I can take a pass at making most of these PRs dependent on just the pxr/base/tf changes which should hopefully make these easier to look at.

All the pull requests have been updated to be dependent on only either dev or the tfhash pull request.

[spiffmon]

Awesome - thanks, Matt!

…

--spiff

On Mon, Jan 23, 2023 at 12:43 PM nvmkuruc ***@***.***> wrote: I think I can take a pass at making most of these PRs dependent on just the pxr/base/tf changes which should hopefully make these easier to look at. All the pull requests have been updated to be dependent on only either dev or the tfhash pull request. — Reply to this email directly, view it on GitHub <#2187 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABPOU2CFVNGMEMKJHUVVLATWT3UOFANCNFSM6AAAAAAUCESUO4> . You are receiving this because you commented.Message ID: ***@***.***>

[sunyab]

Filed as internal issue #USD-7907