-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add missing extractor for staticmesh product type #70
Add missing extractor for staticmesh product type #70
Conversation
@moonyuet should the testing include the new |
@LiborBatek no need to include that template. it would use the existing unreal template instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gives me following error when trying to Publish:
Traceback (most recent call last):
File "C:\Users\lbate\AppData\Local\Ynput\AYON\dependency_packages\ayon_2406251801_windows.zip\dependencies\pyblish\plugin.py", line 528, in __explicit_process
runner(*args)
File "c:\Work\REPO\ayon-unreal\client\ayon_unreal\plugins\publish\extract_fbx.py", line 35, in process
task.set_editor_property('object', asset_names)
TypeError: AssetExportTask: Failed to convert type 'str' to property 'Object' (ObjectProperty) for attribute 'object' on 'AssetExportTask'
TypeError: NativizeProperty: Cannot nativize 'str' as 'Object' (ObjectProperty)
TypeError: NativizeObject: Cannot nativize 'str' as 'Object' (allowed Class type: 'Object')
…staticMesh_product_type
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changelog Description
This PR is to add missing fbx extractor for staticMesh product type and also use
staticMesh
as product type instead ofunrealStaticMesh
to align with the unreal workflow to other dccsAdditional info
We need to discuss the use case of this product type in Unreal to other DCCs
Testing notes: