Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HYDRA-472 : Fix the scene indices chain used for Maya Hydra, for USD … #3293

Merged
merged 3 commits into from
Aug 24, 2023

Conversation

lanierd-adsk
Copy link
Collaborator

Fix the scene indices chain used for Maya Hydra, for USD 23.08 and 23.11+.
Including the code changes from another PR : #3275 made by @dj-mcg.

…23.08 and 23.11+.

Including the code changes from another PR : #3275.
Copy link
Collaborator

@seando-adsk seando-adsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm approving the review, but I would like for another dev from Hydra team to review the changes as I'm not familiar with this code.

@seando-adsk seando-adsk added the core Related to core library label Aug 23, 2023
@lanierd-adsk lanierd-adsk added the adsk Related to Autodesk plugin label Aug 24, 2023
@seando-adsk seando-adsk added do-not-merge-yet Development is not finished, PR not ready for merge ready-for-merge Development process is finished, PR is ready for merge and removed do-not-merge-yet Development is not finished, PR not ready for merge labels Aug 24, 2023
@seando-adsk
Copy link
Collaborator

Reviewed internally by another MayaHydra team member. Marking as ready-for-merge.

@seando-adsk seando-adsk merged commit 50a1883 into dev Aug 24, 2023
@seando-adsk seando-adsk deleted the lanierd/HYDRA-472 branch August 24, 2023 15:18
@lanierd-adsk
Copy link
Collaborator Author

lanierd-adsk commented Aug 25, 2023 via email

@seando-adsk
Copy link
Collaborator

@lanierd-adsk Just to confirm. We are still building MayaUsd with older USD versions: Maya 2020, 2022, 2023 = USD v21.11 and Maya 2024 = USD v22.11. Are those still supported by this change?

@lanierd-adsk
Copy link
Collaborator Author

lanierd-adsk commented Aug 25, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adsk Related to Autodesk plugin core Related to core library ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants