From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <ffmpeg-devel-bounces@ffmpeg.org> Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 833B04DA3B for <ffmpegdev@gitmailbox.com>; Mon, 21 Apr 2025 20:52:55 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C1E2A68834E; Mon, 21 Apr 2025 23:52:51 +0300 (EEST) Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11olkn2059.outbound.protection.outlook.com [40.92.20.59]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 94E0D687CED for <ffmpeg-devel@ffmpeg.org>; Mon, 21 Apr 2025 23:52:45 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=hSaVRg+JYPA7c4gPajs7bLWXOKq4Vm60/6DnjUM5ggK9cQRNsLFmMcWGWyKC2HeyjpYmn5WRMPMBfuz9Kz5nrD09AL1BR8qUMnNGEqicSb1/izXfIOxgKz0T66XAvVYvmHLW5wuTZEqz4Acc5KgrtPCtbJAwsiwkdnWenm49lpkxwKeZZ/hFLkXiqYCU4JrY75TehiIQ7szy5FfYIQZvrRaB9T6n8mG8ZmgL+9wjVf/0Ul6vJaebd/2CNYLVw/gzgnPsHAnmz/QqnZ0V90tKcA9IgBVLPJjQQDklyBt7M1xcUoTfcgoQnCy+bbyMqe8y/LDtD5PDEGBJrqYQ/6w4fg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=mOj1CEulSG8F9X2OpNXJ75kxoMu1mzeBCCvU9meyodg=; b=dgINY82wQXgnt91y6M24aiU/kIk8o1h2VrF75Wav5b1CkrC7CvZ3TLpYQajSrSAYLkRTex5k6RLvPB1neNH01DKXalM5SaIKXl4PfifLnOOMe8sHaEkIz0piQX1DYERAncXG8QTV2to0Rb70l/67PevLe5If9+417muivT9LOQ0gxCkFCDQGc8a6ULbdx5vvuTPM+1oEJqjorStZZLA+u89LuKDjX0vskvXJM5zXl32Rps7KGsz89OrzlK2hCUgeULha+JUaw+KGt8E+m7sZmsgugfNWsnM96iUovovAIvOyPi4cSKmXjqRsV+Q9vlDwf9FGqGAk49VxUaM2pBYQiQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hotmail.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=mOj1CEulSG8F9X2OpNXJ75kxoMu1mzeBCCvU9meyodg=; b=uh0dD5/VKGQ8ju4JpyyS1oajQVFagfXajGdgIT1BvvLjDfBP5gXQ7EEluAKLNQvnlT8VnvNgfUhCIgmxoB2rOclgH1Fmc/4G5B1AM6tmUj4mNer6iqTTSj/jGS9Y/kbJTz55b7XUCgbGKJKNAIPi58wTq/PQzMeA8ZkWT9plZsrueQQa496pcgmleiN1pLRLQ6jOjOwY1OuQCRPTzeLDjbag808T6RHJPqwb/PFb/QPU/y5ffrQ18uPPAo7JeMUbhRW4fMvCM5un1WUBEK2jn3/ulzLA+lHnhYXWPXlSqQ9HdR6neZedesRvX4l7/KSr0gR0KVWLIxf6hd23etis7w== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by DM8P223MB0270.NAMP223.PROD.OUTLOOK.COM (2603:10b6:5:317::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8655.35; Mon, 21 Apr 2025 20:52:42 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7%4]) with mapi id 15.20.8655.033; Mon, 21 Apr 2025 20:52:42 +0000 From: "softworkz ." <softworkz-at-hotmail.com@ffmpeg.org> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Thread-Topic: [FFmpeg-devel] [PATCH] avformat/dump: Change precision of stream start offsets Thread-Index: AQHbsjBgmnqibkcbuU+WybiBeSgUerOtjYkAgACP6iCAAGYiAIAABERwgAAIswCAAAOk8A== Date: Mon, 21 Apr 2025 20:52:42 +0000 Message-ID: <DM8P223MB036578CCED26965A7D0EC5ECBAB82@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> References: <pull.72.ffstaging.FFmpeg.1745179871512.ffmpegagent@gmail.com> <af9371b0-8684-404f-8bde-33e4dcaec555@gyani.pro> <DM8P223MB0365F6CE5ADDBFBD31C13CEFBAB82@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> <876061f7-1b81-1d4c-6dbc-51ac2119e2f3@passwd.hu> <DM8P223MB0365837752CD6F7A92A2F335BAB82@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM> <b5348676-35cc-1a29-cdc4-c70666a5ec61@passwd.hu> In-Reply-To: <b5348676-35cc-1a29-cdc4-c70666a5ec61@passwd.hu> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM8P223MB0365:EE_|DM8P223MB0270:EE_ x-ms-office365-filtering-correlation-id: b4bcdbbb-0889-4b1d-1b37-08dd811675e7 x-microsoft-antispam: BCL:0; ARA:14566002|8060799006|461199028|8062599003|15080799006|19110799003|7092599003|10035399004|102099032|3412199025|440099028|41001999003; x-microsoft-antispam-message-info: =?us-ascii?Q?sD4LVCUOzkChz4qIA0oZS17uX6qxBPODhAwXVX7tCXkcUisJMN6TYDeHtBxq?= =?us-ascii?Q?uFlxmIRAOrmnW1ihSiQGidv/n1SfludfG3iI2q7ajwQPHjCdsUkMyGHNRLZ9?= =?us-ascii?Q?Et+yLVjitl2wwuiDC8wa/s3bc8Wc7eiD32iwyPGT9LpO00G82knDjcX6XoFe?= =?us-ascii?Q?t9nIa4gfvCIEz3xRPLYD+tVH6zntvEKGYh3lBFEdRcCrRWz9MwlR6PCZLdnf?= =?us-ascii?Q?4CE9vY3NYkW9Guj7ALjPWAD2Q1MUsUbYOONqxUXiiKCoDRH3WaXYYQJHdMSi?= =?us-ascii?Q?zRJWnGFCft6st0FaYUB2cA+jSm0Z3sjV6zWij9v5AyFpV5PkllnjbkxlJS5/?= =?us-ascii?Q?wFpHrtE1CiQvBkUIsKHuqsS/6BPvw5xsvTcLWDO3Dfqu2ij3c3iyLStI2gZn?= =?us-ascii?Q?UVGsMDmh0LlZdY8aIFlav4KRm9ox8L+PK5Uf/qEI9tLd0f0vHPkIIud3Zzg6?= =?us-ascii?Q?kN8mSXXUOXCsq1hPleFvf7FIyoseWKKmCc44gq5wqjVFFBCepbT0Rt4Z/4Ks?= =?us-ascii?Q?6/iO/4dACIDrkfCd6RJBQaLDyxpS09kDlBM4lcIcJvn1withZ8Nu389sVbqy?= =?us-ascii?Q?J1vLIQh94zVCkh7sIE9nZ0DxIseLOwYUM163+P6W75OgkOggyn13Xewyx80l?= =?us-ascii?Q?MIkIgQShp/D/LNd3sLIxNZERvCWuuF49Pt0HFouCDN66g+WXY5t/Ym/X4Vxr?= =?us-ascii?Q?oF4A20ZR/3vOOxvO34s4MEx8JHsaHG4rhHOaA1sWQBnak57nI66AAnASaLzl?= =?us-ascii?Q?WEhB1wCCllPsTTYZiBEDBuizzHSrE23bmlcDWk2F85vJLs3G7HXnWxwtjH5V?= =?us-ascii?Q?n98PUu4qTUuALMZMEy1+nottKbqT9oZFKFR9RldhGD5k5oFeBhwi5iEai0AM?= =?us-ascii?Q?/i2qDt+mSoIfgyTmHfXzQx4pkDOagkTuBWQ3pP0oNwLFtzGlLVmJUTnJR8sI?= =?us-ascii?Q?ZGCOArIP0s5c/On3M7U/owMyUwP41Vfd2FY6CLS/FwqzrkrJYXHnA5y2uKTz?= =?us-ascii?Q?uYDjWLA4Xo0lMt7wSwog7Yb9QgwYQIIp9u8FwLGR0YMJk+t5POW3OM0gyFXH?= =?us-ascii?Q?oFKwbPtW37f7rT6GAuttMNKcM7gIhuN0VckHxfObfQ/o0l17t2NcoIia2dUR?= =?us-ascii?Q?/mY+xu0Fj1N+02O+5XI2zS+Ki/Ctw7GaZA8JX9vJINduu0wUsAMcCKSZkvXZ?= =?us-ascii?Q?aA4Qv5MVhqjYA0h7cKWk1zXqEUIHweBvMSdnnw=3D=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?8GQp6nDw4HJBDwCST/Zg8ojEVZ88AshxWZy3qFDHR8Gqss8kXM7YXad/btQT?= =?us-ascii?Q?TS6rVkk5vYCbRodXaXvhhk5DrCVrRS9rvLVFIXjEIMQoYiZw8PoeJD4W79D2?= =?us-ascii?Q?lHldwhZ3odCQZKMNpOcDXOCZrYjvwUqnaamsHKRodIgEIjDDHTvNyvxIlWLU?= =?us-ascii?Q?XqKI6LHh3LEw/eSz0rix2mpAjfdX+TQpL6i2H1t3RFbr1RvJdD7nD86gq+Wo?= =?us-ascii?Q?LdaHxFg6ShY/yOGhbcrjE7TAp57pV1IbnHREaz6NeHFD/zT0bsBv6y1Lbl8O?= =?us-ascii?Q?WwLDEqQ5C2eleEEqyFcV1vtTjkZATry8zL4JBazlZ+khOLyFrxsI/lVD9YuJ?= =?us-ascii?Q?mXiKJE2xIdLF4sUJAPfj9VWp0bUdMVORDZP6IIgRmJKgHceAEC56E9a7YNvX?= =?us-ascii?Q?fj0fSHcXUopJzK3GAdyM2TdI+23jiUpXqCbJcD5AcpfZAugrjVvmFxLKdNTZ?= =?us-ascii?Q?7GKR810HfgjcbSWaf+cFhQtoqcf8Fz2yG4rkozvVRP61pMW1Gr9tCBi89dCM?= =?us-ascii?Q?PLzqAWqRX89MeuqGUMASqLieQPDz3CZjy7/JhNGNvWBQ+KPkD/BApel9696E?= =?us-ascii?Q?cd1kwghA3VgDzw1w3IhoA5jG8DYC0OwQbZ7kSOTYR4C74/bZeMQN7bejIM8A?= =?us-ascii?Q?wQu5BZUntYDXgnL/t75pyBOeXt+MNub5hfkB6e+ByCkTaR7CQr2d1lWW8niW?= =?us-ascii?Q?hn+2I1kPrV5OblzMhkne9tMtBiSVMxtqETpWHwL6foA3zRH+Fy+1W+kfE3Pu?= =?us-ascii?Q?rTCBb2M6S/d8l5v3vWKuSWAK/QLqnkU0XiOSK7jIMM1roT/LNkKrJsJ4VgD2?= =?us-ascii?Q?6Yfc7nMgdoL9JdFV3Ur5cv0/kWQFyMDuJFZ7Q+pJ+Viy2GkBxRTY1UPlEoVm?= =?us-ascii?Q?8nu+JJs13Ahfv3CJx68ziOfiUExTfphRWVsDDp4trJu3mRiBnV9JGmw/s6pK?= =?us-ascii?Q?MDrE//NnmM/3Enuf3vQQT7+tZw9xGFph00Ic7SkY4A/FKCmIUEiG7kvbdU1S?= =?us-ascii?Q?gV4TOS+xb29ncI7RO4Wu6WYRfFJ7Rrq0rQKQejjpc+kdJ+MR44mbitG1Ta/D?= =?us-ascii?Q?Yfyxm/O9jwa0dw7sVseSMB/UfXgq4D9qC75+iG4fSBUCHjFnevV0V61igRUD?= =?us-ascii?Q?W//HxVD3/8Sb71ZRQ+qe0ItiwygmCiptBdohb6jgCJiXivLDqt6SXYfqYJwx?= =?us-ascii?Q?3eYXREyFLhGw3SsF30TqReyh3fSTestd0EQRIEm/86zoe+N8HuP6mxfeytg?= =?us-ascii?Q?=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-7719-20-msonline-outlook-92255.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: b4bcdbbb-0889-4b1d-1b37-08dd811675e7 X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Apr 2025 20:52:42.3704 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM8P223MB0270 Subject: Re: [FFmpeg-devel] [PATCH] avformat/dump: Change precision of stream start offsets X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org> List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>, <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe> List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel> List-Post: <mailto:ffmpeg-devel@ffmpeg.org> List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help> List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>, <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe> Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org> Archived-At: <https://master.gitmailbox.com/ffmpegdev/DM8P223MB036578CCED26965A7D0EC5ECBAB82@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM/> List-Archive: <https://master.gitmailbox.com/ffmpegdev/> List-Post: <mailto:ffmpegdev@gitmailbox.com> > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Marton Balint > Sent: Montag, 21. April 2025 22:18 > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] avformat/dump: Change precision of > stream start offsets > > > > On Mon, 21 Apr 2025, softworkz . wrote: > > > > > > >> -----Original Message----- > >> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > >> Marton Balint > >> Sent: Montag, 21. April 2025 21:32 > >> To: FFmpeg development discussions and patches <ffmpeg- > >> devel@ffmpeg.org> > >> Subject: Re: [FFmpeg-devel] [PATCH] avformat/dump: Change precision > of > >> stream start offsets > >> > >> > >> > >> On Mon, 21 Apr 2025, softworkz . wrote: > >> > >>> > >>> > >>>> -----Original Message----- > >>>> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > >> Gyan > >>>> Doshi > >>>> Sent: Montag, 21. April 2025 06:51 > >>>> To: ffmpeg-devel@ffmpeg.org > >>>> Subject: Re: [FFmpeg-devel] [PATCH] avformat/dump: Change > precision > >> of > >>>> stream start offsets > >>>> > >>>> > >>>> > >>>> On 2025-04-21 01:41 am, softworkz wrote: > >>>>> From: softworkz <softworkz@hotmail.com> > >>>>> > >>>>> Changing this to 6 digits to align with other > >>>>> printed times > >>>>> > >>>>> Signed-off-by: softworkz <softworkz@hotmail.com> > >>>>> --- > >>>>> avformat/dump: Change precision of stream start offsets > >>>>> > >>>>> Changing this to 6 digits to align with other printed times > >>>>> > >>>>> Signed-off-by: softworkz softworkz@hotmail.com > >>>>> > >>>>> Published-As: > https://github.com/ffstaging/FFmpeg/releases/tag/pr- > >>>> ffstaging-72%2Fsoftworkz%2Fsubmit_start_offsets-v1 > >>>>> Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr- > >>>> ffstaging-72/softworkz/submit_start_offsets-v1 > >>>>> Pull-Request: https://github.com/ffstaging/FFmpeg/pull/72 > >>>>> > >>>>> libavformat/dump.c | 2 +- > >>>>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>>>> > >>>>> diff --git a/libavformat/dump.c b/libavformat/dump.c > >>>>> index 8c7db7b275..1bd0424f3d 100644 > >>>>> --- a/libavformat/dump.c > >>>>> +++ b/libavformat/dump.c > >>>>> @@ -680,7 +680,7 @@ FF_ENABLE_DEPRECATION_WARNINGS > >>>>> > >>>>> if (st->start_time != AV_NOPTS_VALUE && st->start_time != > 0 > >> && > >>>> st->time_base.den && st->time_base.num) { > >>>>> const double stream_start = av_q2d(st->time_base) * > st- > >>>>> start_time; > >>>>> - av_log(NULL, AV_LOG_INFO, ", Start-Time %.3fs", > >>>> stream_start); > >>>>> + av_log(NULL, AV_LOG_INFO, ", Start-Time %.6fs", > >>>> stream_start); > >>>> > >>>> The camel case is incongruous with the formatting of the text > next > >> To > >>>> It. > >>> > >>> Hi Gyan, > >>> > >>> as far as I'm seeing it, captions/labels are title case and values > >> are > >>> lower case. I would consider "Start-Time" to be a caption/label. > >> Let's > >>> look at an example: > >>> > >>> Stream #0:0[0x8fd]: Video: h264 (High) ([27][0][0][0] / 0x001B), > >> yuv420p(tv, bt709, top first), 1920x1080 [SAR 1:1 DAR 16:9], Level > 40, > >> 25 fps, 50 tbr, 90k tbn, Start-Time 32476.588s > >>> Stream #0:1[0x907](dut): Audio: mp2 ([3][0][0][0] / 0x0003), > 48000 > >> Hz, stereo, fltp, 256 kb/s, Start-Time 32476.706s > >>> Stream #0:2[0x908](dut): Audio: ac3 (AC-3 / 0x332D4341), 48000 > Hz, > >> 5.1(side), fltp, 448 kb/s, Start-Time 32476.643s > >>> Stream #0:3[0x909](GOS): Audio: mp2 ([3][0][0][0] / 0x0003), > 48000 > >> Hz, stereo, fltp, 256 kb/s, Start-Time 32476.674s (visual impaired) > >> (descriptions) > >>> Stream #0:4[0x962](dut): Subtitle: dvb_subtitle ([6][0][0][0] / > >> 0x0006), Start-Time 32476.588s > >>> Stream #0:5[0x963](dut): Subtitle: dvb_subtitle ([6][0][0][0] / > >> 0x0006), Start-Time 32476.719s (hearing impaired) > >> > >> I think Gyan means fps, tbr or tbn. These are all lowercase and > >> abbreviations. > > > > Yes, I understood that. But these are units, not labels. > > > > > >> The one-line stream info should be as compact as > >> possible. > > > > Agreed. > > > >> So we should make it shorter, and we should lose the "s" unit as > >> well. > > > > I'm afraid, but omitting the unit makes no sense to me. > > > > Why? > > Because it is redundant. You always print the start time in seconds, > users > will have no problem guessing the time measurement unit of timestamps. > E.g. see the status line of ffplay or mpv. Timestamps or time > differences miss the measurement unit, and nobody complianed... It's redundant only for those who know what it is. Everybody else will need to guess whether it's s or ms or us and whether it's in the time base of the stream or not. With the same argument, you can omit kb/s, fps, tbr, tbn etc. (user can infer the units from the order and count of values) This is something that I totally hate when applications are doing it, and I need to dig down deep into the source code for getting a definitive answer. Letting users "guess" is not ok IMO. > >> Something like: > >> > >> "in 1234.567" > > > > What means "in"? > > in is short for inpoint. > > > > > Maybe "offset" if it's gotta be short..? > > Or simply "start", if you don't like "in". It's not about liking, it's just that "in" doesn't ring any bell for me in that context. The problem with start is that it could also mean the timestamp value where the stream (incl. container) starts, so "start-offset" or "stream-start" would be more clear imo, even though I wouldn't completely reject having just "Start". Regarding the casing: Example: Stream #0:0(eng): Video: h264 (High), yuv420p(progressive), 1920x1080 [SAR 1:1 DAR 16:9], Level 42, 60 fps, 60 tbr, 1k tbn, Start-Time 2.790s (default) Why is "Level" capitalized? This fits into my understanding that I explained before (Labels are capitalized) - how does it fit into the claim that it should all be lower case? Thanks, sw _______________________________________________ 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".