-
Notifications
You must be signed in to change notification settings - Fork 129
Conversation
Task linked: OP-2913 Entity top-level metadata editing (dialog?) |
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.
If DVXHD on MXF is configured in Settings
My ffmpeg output args were:
-codec:v dnxhd
-pix_fmt yuv422p
-b:v 115M
Here is Traceback:
Traceback (most recent call last):
File "C:\CODE\__PYPE\OpenPype\.venv\lib\site-packages\pyblish\plugin.py", line 522, in __explicit_process
runner(*args)
File "C:\CODE\__PYPE\OpenPype\openpype\plugins\publish\extract_review_slate.py", line 95, in process
KeyError: 'timecode'
Another try. Tested on Prores, DNxHD, MXF and H264 mp4, with and withouth sound. |
This might close Nuke: Nuke Slate MXF review fails on 23.976 FPS project #3014 |
Hey @jrsndl, shouldn't this be marked as review ready? I had just tested it and all is working fine as expected ;) I am ready to approve and merge. |
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.
tested and works fine
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
Does it also fix #3014 ? |
|
Isn't this duplicate of #3162 ? |
…audio # Conflicts: # openpype/plugins/publish/extract_review_slate.py
Brief description
Add timecode to reviews with Nuke slate.
Description
Source Timecode is read from the review. Slate is assumed to be one frame. Source timecode is offseted -1 frame and added to the slate encode.
Additional info
This PR also has an initial attempt to solve adding slate to the review with audio. Concatenating slate with review with sound no longer fails, but sometimes produces two frame slate. More work is needed for clean audio concatenation.
Testing notes: