-
Notifications
You must be signed in to change notification settings - Fork 129
Conversation
…4-nks-cut-reference-videos
…ture/PYPE-654-nks-cut-reference-videos
|
||
def process(self, instance): |
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.
'ExtractReview.process' is too complex (40)
too many blank lines (2)
This needs to be able to work also with clips that are already cut to the correct length. Currently if we have As discussed on the call, we'll extend each clip to it's full length with handles by copying a proxy frame before and after the footage. This will ensure that even plates with missing material can be published and worked on. |
For publish plugins where function has more than 50 lines, yes I prefer old formating (rather with args than dict). But like you've said it's my preferation. |
@jezscha can you please write explanation of what it does now exactly? I can't get it to work. Some cutting is happening, but can't get past review extractor. |
yeah I was just working on the schema @mkolar I hope it is more clear now ;) |
Still having issues publishing with this
|
could you send me those files for testing please? cannot replicate the issue on my side.. |
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
* Feature: Animation build with last version * skip if not hero * fix switch * get version hero for broken repre metadata (ynput#27) --------- Co-authored-by: kaa <kaalium@gmail.com>
This picture is showing the problem. Sometimes an input video defined as
review
family member has source video material only for part of the required subset properties. Either it is shorter as handles are missing at the start or the end of the clip, or it is far too long and needs to be shortened.This plugin is taking the source video file from the representation created in NKS
CollectReviews
where it has been tagged with either_cut-bigger
or_cut-smaller
nonpublic tags.Then after missing material is identified will add black empty frames on the spot where it should be.
Schema is explaining the details of procedure.