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

Fix: aftereffects review not displayed in ftrack #660

Merged
merged 9 commits into from
Jun 18, 2024

Conversation

kalisp
Copy link
Member

@kalisp kalisp commented Jun 17, 2024

Changelog Description

Solved bug caused review not getting created.

Testing notes:

  1. render locally (single frame, multiframe)
  2. render via Deadline
  3. check that review is present on DL.

This plugin depends on review in families logic which got put there only after this plugin (because of previous move order to accommodate deadline collection data).
This plugin must run after collect_review BUT before collect DL credentials!
Without it will render frame 0 which might result in black image instead of correct frame (1001).
Folder path messed up os.path.join, resulting in wrong file path pointing to c:/file_name.png. Folder path is actually not needed here, it is already included in base_dir.
@ynbot
Copy link
Contributor

ynbot commented Jun 17, 2024

Task linked: AY-5750 AE review not displayed in Ftrack

@m-u-r-p-h-y m-u-r-p-h-y added the sponsored This is directly sponsored by a client or community member label Jun 17, 2024
@64qam 64qam self-requested a review June 18, 2024 08:12
@iLLiCiTiT
Copy link
Member

BTW please bump version of AE and Deadline

@iLLiCiTiT iLLiCiTiT merged commit 86d5ae6 into develop Jun 18, 2024
1 check passed
@iLLiCiTiT iLLiCiTiT deleted the bugfix/AY-5750_AE-review-not-displayed-in-Ftrack branch June 18, 2024 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: Deadline size/XS sponsored This is directly sponsored by a client or community member type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

6 participants