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

Chore: Make sure implicit thumbnail in higher priority over the thumbnail source #3

Merged

Conversation

moonyuet
Copy link
Member

@moonyuet moonyuet commented Jul 17, 2024

ported from ynput/ayon-core#695

Changelog Description
Resolve #9
When users trying to create screengrab thumbnails and publish along with the review product type, it will error out the intgerate.py due to the duplicate of the thumbnail representation data. This PR is to make sure the extract thumbnail is always at the higher priority than the extract thumbnail sources

Additional info
please test on some other hosts with thumbnail extractors in review family, maybe similar bug is hit when you publish both screengrab and review family

Testing notes:
Launch Maya/Max
Create Review Instance
Create Screengrab in publisher tab
Publish

Copy link
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

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

I have captured multiple thumbnails on different publish instances and all publishes well. No any failures or errors shown during publishing and only Review takes precedence over user based thumbnail which is correct behavior.

image

and integrated on Ayon server:

image

All good imho!

@moonyuet moonyuet merged commit ddd4dbd into develop Jul 31, 2024
1 check passed
@moonyuet moonyuet deleted the bugfix/AY-5562_Maya-duplicate-thumbnail-integration-error branch July 31, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make sure implicit thumbnail in higher priority over the thumbnail source
2 participants