Skip to content

Conversation

@quarium
Copy link
Contributor

@quarium quarium commented Jan 30, 2026

Add upipe_interlace pipe to preform temporal field interlacing.

@quarium quarium requested a review from nto January 30, 2026 15:31
@quarium quarium force-pushed the upipe_interlace branch 2 times, most recently from 59d7ace to 41213f1 Compare January 30, 2026 15:52
Change uref_pic_progressive and uref_pic_tff attributes from void to
boolean. With void attribute, there is no possibility to explicitly set
interlaced or bottom field first on a flow definition packet.
Add upipe_interlace pipe to preform temporal field interlacing.
* @return an error code
*/
static int upipe_interlace_set_option(struct upipe *upipe, const char *option,
const char *value)
Copy link
Collaborator

Choose a reason for hiding this comment

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

The SET_OPTION control command should only be used to pass string parameters to an underlying API. Standard parameters are to be implemented using dedicated control commands, otherwise they are inconvenient to use since native values cannot be used and static typing is gone.

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.

2 participants