From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 093F54440E for ; Mon, 12 Dec 2022 15:28:53 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 16FC568BD21; Mon, 12 Dec 2022 17:28:52 +0200 (EET) Received: from mail.frobbit.se (mail.frobbit.se [85.30.129.176]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D68BD68BD0F for ; Mon, 12 Dec 2022 17:28:45 +0200 (EET) Received: from debian.lan (unknown [IPv6:2a00:66c0:a::72c]) by mail.frobbit.se (Postfix) with ESMTPSA id 6226C20556 for ; Mon, 12 Dec 2022 16:28:45 +0100 (CET) Message-ID: <22621577bd7ccd1da3bab3e16740528ffd1d82fb.camel@haerdin.se> From: Tomas =?ISO-8859-1?Q?H=E4rdin?= To: FFmpeg development discussions and patches Date: Mon, 12 Dec 2022 16:28:44 +0100 In-Reply-To: References: <20221207093122.553668-1-quinkblack@foxmail.com> User-Agent: Evolution 3.38.3-1 MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH v2 5/7] avcodec/mediacodecenc: remove the strategy to create DTS X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: ons 2022-12-07 klockan 17:31 +0800 skrev Zhao Zhili: > From: Zhao Zhili > > Use input PTS as DTS has multiple problems: > 1. If there is no reordering, it's better to just use the output > PTS as DTS, since encoder may change the timestamp value (do it > on purpose or rounding error). > > 2. If there is reordering, input PTS should be shift a few frames > as DTS to satisfy the requirement of PTS >= DTS. I can't find a > reliable way to determine how many frames to be shift. For example, > we don't known if the encoder use hierarchical B frames. The > max_num_reorder_frames can be get from VUI, but VUI is optional. > > 3. Encoder dropping frames makes the case worse. Android has an > BITRATE_MODE_CBR_FD option to allow it explicitly. Don't we already have code to parse this stuff from h.264 streams? /Tomas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".