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

Optimize output data from Maya USD animation export #115

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

BigRoy
Copy link
Contributor

@BigRoy BigRoy commented Sep 17, 2024

Changelog Description

Reduce amount of data extracted to USD from Maya USD animation.

Additional info

Draft.

Testing notes:

  1. Publish maya USD for animation product - it should work.
  2. Check the resulting published data in the USD file, confirm it does not contain materials, UVs, etc.

@BigRoy BigRoy added type: enhancement Improvement of existing functionality or minor addition sponsored This is directly sponsored by a client or community member labels Sep 17, 2024
@BigRoy BigRoy self-assigned this Sep 17, 2024
Comment on lines +430 to +431
options["exportMaterials"] = False
options["exportAssignedMaterials"] = False
Copy link
Member

@moonyuet moonyuet Oct 24, 2024

Choose a reason for hiding this comment

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

I tested out with these two options and also do some research on the mayaUSD. It seems that these two options haven't been exposed to Python as it errors out in maya 2025.1 during extraction.
image

image

Suggested change
options["exportMaterials"] = False
options["exportAssignedMaterials"] = False

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's odd. But they are in the list here: https://github.com/Autodesk/maya-usd/blob/dev/lib/mayaUsd/commands/Readme.md#mayausdexportcommand

Can you check which maya-usd addon version you're using?

Copy link
Member

@moonyuet moonyuet Oct 24, 2024

Choose a reason for hiding this comment

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

I am at 0.28.0 not the latest 0.30.0. That could be the reason. I would download the latest version and try it again.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks - yes. Seems added in 0.29.0 -> https://github.com/Autodesk/maya-usd/blob/v0.29.0/doc/CHANGELOG.md

Will take it into consideration. 👍

@moonyuet moonyuet self-requested a review October 24, 2024 11:33
@moonyuet
Copy link
Member

I tried with the latest version of maya 2025.3 with mayausd 0.30.0
The animation is published successfully.
image

@BigRoy Do we need to implement some backward compatibility for just in case users are using maya-usd 0.28.0 or lower?

@BigRoy
Copy link
Contributor Author

BigRoy commented Oct 24, 2024

I tried with the latest version of maya 2025.3 with mayausd 0.30.0 The animation is published successfully. image

@BigRoy Do we need to implement some backward compatibility for just in case users are using maya-usd 0.28.0 or lower?

Yes, that's the idea. Do keep in mind this PR is still in draft. There are other things to solve too, like:

  • This may need to be optional or configurable in settings. (especially for backwards compatibility with existing projects)
  • Shouldn't crash on older USD addons, but if the setting is used on the instance what should it do then? Log a warning that the output might not consider that and hence may be different than what the user expects from the toggles?

@moonyuet
Copy link
Member

I tried with the latest version of maya 2025.3 with mayausd 0.30.0 The animation is published successfully. image
@BigRoy Do we need to implement some backward compatibility for just in case users are using maya-usd 0.28.0 or lower?

Yes, that's the idea. Do keep in mind this PR is still in draft. There are other things to solve too, like:

  • This may need to be optional or configurable in settings. (especially for backwards compatibility with existing projects)
  • Shouldn't crash on older USD addons, but if the setting is used on the instance what should it do then? Log a warning that the output might not consider that and hence may be different than what the user expects from the toggles?

This should be configurable in settings, but we need to inform users about the missing functionalities if they are using elder versions of maya usd.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sponsored This is directly sponsored by a client or community member type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AY-6684_Maya USD: Publish animation with only point positions, transforms, bounding boxes as overrides
2 participants