The idea itself looks good to me, but the patch is corrupted. Regarding the patch: 1) >>// Keyframe overrides previous assignment. It might be better instead of: setting the frame type -> if key frame then overwrite to do: check for key frame and if it's not, set other PICTURE_TYPE to avoid unnecessary overwriting (use AV_FRAME_FLAG_KEY instead of deprecated key_frame, as already noted under the second patch) 2) The patch includes logic for AV_CODEC_ID_H264 and AV_CODEC_ID_HEVC, but not for AV_CODEC_ID_AV1.( FORCE_INSERT_SEQUENCE_HEADER AMF_VIDEO_ENCODER_AV1_FORCE_FRAME_TYPE ) _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
