Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

General: Extract review single frame output #4064

Merged

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Nov 3, 2022

Brief description

Extract review have different outputs when input is single frame.

Description

The goal is to avoid creating of single frame movie files but also give ability to create different review output and define it's resolution and tags. We've added one more additional filtering to output defintinions. It is possible to tell that the output will be created always (no matter how many frames input has), when input has 1 frame image or when input has 2+ images (or is movie).

Added default output definition to create HD png output if input is single frame and h264 is created only if input is video or has multiple frames.

Testing notes:

Your current settings should work as they did before (nothing should break with older settings).

To test the feature: Change your settings to create an output definition on single frame, another output definition on multi frame and another output definition which is created always. You can add letterboxes with different colors to know which is which.

Run publishing in all of those cases e.g. by using tray publisher where can be passed reviewable.

@ynbot
Copy link
Contributor

ynbot commented Nov 3, 2022

Task linked: OP-4353 extract review single frame output

@github-actions github-actions bot added this to the next-patch milestone Nov 3, 2022
@iLLiCiTiT iLLiCiTiT marked this pull request as ready for review November 3, 2022 10:33
@iLLiCiTiT iLLiCiTiT self-assigned this Nov 3, 2022
@iLLiCiTiT iLLiCiTiT added the type: enhancement Enhancements to existing functionality label Nov 3, 2022
@iLLiCiTiT iLLiCiTiT merged commit 3a6a6c3 into develop Nov 7, 2022
@iLLiCiTiT iLLiCiTiT deleted the feature/OP-4353_extract-review-single-frame-output branch November 7, 2022 16:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants