Conversation
Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
…cache-Animation-family # Conflicts: # openpype/hosts/maya/api/plugin.py # openpype/hosts/maya/plugins/create/create_animation.py # openpype/hosts/maya/plugins/create/create_pointcache.py # openpype/hosts/maya/plugins/publish/extract_pointcache.py
…cache-Animation-family # Conflicts: # openpype/hosts/maya/plugins/create/create_animation.py # openpype/settings/defaults/project_settings/maya.json
…cache-Animation-family
…cache-Animation-family
…cache-Animation-family # Conflicts: # openpype/hosts/maya/plugins/create/create_animation.py
client/ayon_core/hosts/maya/plugins/publish/validate_alembic_options_defaults.py
Outdated
Show resolved
Hide resolved
client/ayon_core/hosts/maya/plugins/create/create_animation_pointcache.py
Outdated
Show resolved
Hide resolved
client/ayon_core/hosts/maya/plugins/publish/collect_animation.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
…intcache.py Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
…ptions_defaults.py Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
…ache-Animation-family # Conflicts: # server_addon/maya/server/version.py
…n-family' of github.com:ynput/ayon-core into enhancement/AY-970_abc-options-for-Pointcache-Animation-family
|
Massive work everyone! :) |
|
Getting this error after this PR: |
|
Next error after that is: DEBUG: Extracting pointcache..
Traceback (most recent call last):
File "C:\Users\Maqina-02\AppData\Local\Ynput\AYON\dependency_packages\ayon_2404022047_windows.zip\dependencies\pyblish\plugin.py", line 527, in __explicit_process
runner(*args)
File "C:\Users\Maqina-02\AppData\Local\Ynput\AYON\addons\core_0.3.1-dev.3\ayon_core\hosts\maya\plugins\publish\extract_pointcache.py", line 165, in process
"writeCreases", self.writeCreases
AttributeError: 'ExtractAnimation' object has no attribute 'writeCreases' |
|
Next error is: Traceback (most recent call last):
File "C:\Users\User\AppData\Local\Ynput\AYON\dependency_packages\ayon_2403071252_windows.zip\dependencies\pyblish\plugin.py", line 527, in __explicit_process
runner(*args)
File "E:\dev\ayon-core\client\ayon_core\hosts\maya\plugins\publish\extract_pointcache.py", line 218, in process
extract_alembic(**kwargs)
TypeError: extract_alembic() got an unexpected keyword argument 'userAttr'I get the feeling this PR was merged too quickly? :D |
| "renderableOnly": False, | ||
| "stripNamespaces": True, | ||
| "uvsOnly": False, | ||
| "uvWrite": False, |
There was a problem hiding this comment.
Do we really feel this is a sensible studio default we should offer from AYON by default. With this False by default Alembics will be written without UVs - pretty sure we'll want to flip that. Right @tokejepsen @mkolar ?
| "bake_attributes": [], | ||
| "bake_attribute_prefixes": [], | ||
| "dataFormat": "ogawa", | ||
| "eulerFilter": False, |
There was a problem hiding this comment.
This defaulted to True before, by making this False with this PR this now means there is a difference in default behavior.
| "writeFaceSets": False, | ||
| "writeNormals": True, | ||
| "writeUVSets": False, | ||
| "writeVisibility": False |
There was a problem hiding this comment.
This defaulted to True before this PR - with this now False any animated visibilities (or even any visilities whatsoever) will not be published along. Do we feel that's a sensible default value for AYON?
| frameRange="", | ||
| noNormals=False, | ||
| preRoll=False, | ||
| preRollStartFrame=0, |
There was a problem hiding this comment.
Pretty sure this functionality does not work?
I'm seeing this in my extraction logs:
# Warning: Ignoring unsupported flag: -preRollStartFrame
# Warning: Ignoring unsupported flag: 0
There was a problem hiding this comment.
It seems preRollStartFrame is not a job argument but a direct argument to AbcExport. See logic here. So the current implementation here is broken. @tokejepsen that will have to be rewritten a different way then.
Changelog Description
Port of ynput/OpenPype#5173