> > +    ctx->width  = ctx->coded_width  = vp9->frame_width;
> > +    ctx->height = ctx->coded_height = vp9->frame_height;
> > +
> > +    ctx->pict_type = fh->intra_only ? AV_PICTURE_TYPE_I :
> > AV_PICTURE_TYPE_P;
> 
> I found it is different as previous version, so I took a look at spec and
> cbs_vp9_syntax.
> 
> I believe the intra_only is only usable for non-key frame, so it should be
> something like:
>    ctx->pict_type = is_key_frame ? AV_PICTURE_TYPE_I : (fh->intra_only ?
> AV_PICTURE_TYPE_I : AV_PICTURE_TYPE_P );

Any update? I am looking forward this patch can be applied with this fix.
_______________________________________________
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".

Reply via email to