Skip to content

Maya: abc options for Pointcache/Animation family - AY-970#336

Merged
mkolar merged 236 commits intodevelopfrom
enhancement/OP-5920_abc-options-for-Pointcache-Animation-family
Apr 30, 2024
Merged

Maya: abc options for Pointcache/Animation family - AY-970#336
mkolar merged 236 commits intodevelopfrom
enhancement/OP-5920_abc-options-for-Pointcache-Animation-family

Conversation

@tokejepsen
Copy link
Copy Markdown
Contributor

@tokejepsen tokejepsen commented Apr 1, 2024

Changelog Description

Port of ynput/OpenPype#5173

  • Validate alembic defaults settings

tokejepsen and others added 30 commits June 21, 2023 17:10
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

# Conflicts:
#	openpype/hosts/maya/plugins/create/create_animation.py
tokejepsen and others added 12 commits April 15, 2024 14:58
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
@tokejepsen tokejepsen requested review from BigRoy and iLLiCiTiT April 27, 2024 10:30
@mkolar mkolar merged commit c918bea into develop Apr 30, 2024
@mkolar mkolar deleted the enhancement/OP-5920_abc-options-for-Pointcache-Animation-family branch April 30, 2024 13:40
@BigRoy
Copy link
Copy Markdown
Collaborator

BigRoy commented Apr 30, 2024

Massive work everyone! :)

@BigRoy
Copy link
Copy Markdown
Collaborator

BigRoy commented May 1, 2024

Getting this error after this PR:

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\validate_alembic_options_defaults.py", line 46, in process
    attributes = self._get_publish_attributes(instance)
  File "C:\Users\Maqina-02\AppData\Local\Ynput\AYON\addons\core_0.3.1-dev.3\ayon_core\hosts\maya\plugins\publish\validate_alembic_options_defaults.py", line 33, in _get_publish_attributes
    cls.plugin_name(
TypeError: 'str' object is not callable

@BigRoy
Copy link
Copy Markdown
Collaborator

BigRoy commented May 1, 2024

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'

@BigRoy
Copy link
Copy Markdown
Collaborator

BigRoy commented May 1, 2024

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,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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,
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Collaborator

@BigRoy BigRoy May 1, 2024

Choose a reason for hiding this comment

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

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

host: Maya size/L type: enhancement Improvement of existing functionality or minor addition

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.