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 00D11452A8 for ; Mon, 16 Jan 2023 14:00:20 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 3F90368BE13; Mon, 16 Jan 2023 16:00:17 +0200 (EET) Received: from FRA01-PR2-obe.outbound.protection.outlook.com (mail-pr2fra01on2043.outbound.protection.outlook.com [40.107.12.43]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 6F319680106 for ; Mon, 16 Jan 2023 16:00:10 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=m01qiQzHfqpXko9tPXN92vSdcu/xkVye1Jd1YCPeuchdw7zpM0tut5CSpURGz/6Ym00RZZE+LrmtCx2EFQdVPlUDecgRMtChSIAl6VtCIH2E1aUgarUPWMnR8/+GcOlSXu/ZcsC1FNZOkh57he3FCHPpGX6q3oRrXvBl2M+2+kVTfhcJ4mpiS7wDryhrqoZ7UYNa1/m5hV0Q4nkIdf6Bj+sx64mLmVk15a1CDbM8PxevjSgSoiN8qJBC4+X0h7L034edcFiTGDoQEx6hhpkIIji1iFJZihXtZAVFaSV6sfO6Fw60GFRbP2Yf2MtsK4KMblk3/3kXFvjLztHi81V52A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=xG3Ah6LA5yDfHtRwkJoYy5UzWMbf+0zv2TX4k11T9kw=; b=hl2YMVC79thkW1kWtYmCXtHUpLOcFQ/zJkua4+DXQmP0Gd3m/0mG4a/RMUYXUadBLg3hJhC1dVISsZ99kRZl5Jjrypq4q+a90VMt4O8eILusnN2k6j4OaGm6EHfwE369YRx7ToG/R43o8syHW8eJztx01Saoi6bVOwB6r1AZQRi8QFnbyP6KvEti8ukMJmCOXEuG8pZhw+epU2zlbqh6s1WSmQiWoC0yxxEMjAmDkCzsJgjDUchCpk0FU7d+hC1H675ZlZSW7P0tqQqeTaSUS713tlYHbeHHLNIQOkXR/QPcRAlLQCGRPLbidXkG7EFX2JspNyuwyEK7MvGtNBRXWA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cji.paris; dmarc=pass action=none header.from=cji.paris; dkim=pass header.d=cji.paris; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cjiparis.onmicrosoft.com; s=selector1-cjiparis-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=xG3Ah6LA5yDfHtRwkJoYy5UzWMbf+0zv2TX4k11T9kw=; b=XYDYDZCet/4WPRxWUo6ZMg8XcCYKGFpcNnDDTK3uS6yAUrM2oM7tNbnz10qnQTzojT0000HV1gIND1NssLcLlAoGrXhpRna/X4BHM6Y7GWX16DEQD/ArMc3DL9soEujhXPhmnHdNiGjAljsFzsr7jVjoGVf5bAz7mwK11u6e72qrLwieNuGCkNRajaYHCmLsE8bFd4XZwV+4FH/QKl97Y4RSYOzEJdes1htsbZaxZCS4sJMS6kobm9NSBrHmYAnNi8dqzBWWDKHvdzq3Xeg4KnVG9frkFGx9MekyjjzezmEa93dFOlz+J7p15l4bRRpVyC/nA59QMV+Cc5i49c/evQ== Received: from MR1P264MB2483.FRAP264.PROD.OUTLOOK.COM (2603:10a6:501:34::23) by MRZP264MB1670.FRAP264.PROD.OUTLOOK.COM (2603:10a6:501:a::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5986.23; Mon, 16 Jan 2023 14:00:08 +0000 Received: from MR1P264MB2483.FRAP264.PROD.OUTLOOK.COM ([fe80::8ff5:d8f7:f4e1:748f]) by MR1P264MB2483.FRAP264.PROD.OUTLOOK.COM ([fe80::8ff5:d8f7:f4e1:748f%5]) with mapi id 15.20.6002.012; Mon, 16 Jan 2023 14:00:08 +0000 From: Nicolas Gaullier To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] avformat/mxfenc: fix stored/sampled/displayed width/height Thread-Index: AQHZKC+j5eSWmbHhmkiLRCzZk9GPE66hCc6g Date: Mon, 16 Jan 2023 14:00:08 +0000 Message-ID: References: <0af7e930-9ec7-dc81-95bc-939eb21e2da0@mediaarea.net> In-Reply-To: <0af7e930-9ec7-dc81-95bc-939eb21e2da0@mediaarea.net> Accept-Language: fr-FR, en-US Content-Language: fr-FR X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=cji.paris; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MR1P264MB2483:EE_|MRZP264MB1670:EE_ x-ms-office365-filtering-correlation-id: baea82a9-c2d0-4448-79e3-08daf7c9fa0d x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: d18Qjg4F6dokrmLlHza8AnUKxL9CWddthNY0pB50CCpV++W5D1zhucroYLLuBBY8HcN5H7qOXjQhn7ynrm9lXbK8zFVyWom6ASCIPxhEaYZ8cUrsJqt8vR3Tq6JjrWflZTpmIdr+VefSISI0VFo+UgoRfOXpMIW7Oc0BUphprGzFakmpKfIOzoXwktAF3XQi+HjqFOW/WP3ESVDRlh0wtbjeELg6UwzFT3sCRPOM3v9EQtxb9mTRggE9WgwUbrOxJkC0LqlAPjcE8F4kNVtKOyPeuBzVVepxAcWEFZQccGt4gd8a7f/vVYC7j9k7d+BQgYreYEo5l9n3M3sXiOAeep25XfWjWgGx0IQi2pyaTJPpxztYDuk/q5wQ/f4I6PqOubbkCDQlyGFtiaPJe0CMK7A/w7N7z7fYzC+VEOicwYa2WGoXRVcvamePaxeekEbKq3HuaugEynZLHau8KE36UMDslNXCArSrTZogqYKF3ehWL5o/ozsejb0cwENUeKHoF9IHfLCWKwaqdG2+2n6a5DhxK316cb1igAoSrLq7KFZXxl3GprlfVccf790gSEtbBDvja7CVcSNssYrcOveo3QYx/3KH5Y6Eq+P9SKjGutietlyEdvddnVxyanUZcB1MWnw+/Cwg15ICo3O2P6HVCEcjli+5Fm38Tuor76WWsJbXrQN4cjfzZgIon9plvdR5qW0oOladsKKOP+JusqGz2gVJiSSzi8P12bx7HVkpBek= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MR1P264MB2483.FRAP264.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(13230022)(366004)(396003)(136003)(39850400004)(376002)(346002)(451199015)(86362001)(55236004)(33656002)(26005)(6916009)(8676002)(64756008)(186003)(9686003)(66946007)(66556008)(76116006)(66476007)(66446008)(41300700001)(316002)(71200400001)(6506007)(55016003)(7696005)(478600001)(122000001)(44832011)(38100700002)(2906002)(38070700005)(52536014)(83380400001)(5660300002)(8936002)(46492015); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?eUFkcDRPVFgvS2dOeHlwZXE4aTBJYThvdUxHblByUlVLanlFYW1EOTU5ZGN2?= =?utf-8?B?czFUd0JtbmdkUGM4bGZGVllSdHgvbVQ4Wkt0d0EwcXZXdy9uR3UvMkQ1cVZj?= =?utf-8?B?Z1MvTDhqUmNwaC8zRHA4Y3FOalY2OWZpN2d0d2FrSzdNQjl4UDk2aEJoSkRh?= =?utf-8?B?QWlUTFNVdmZQa0dMTjdsaHpuRmJaNlh2VTgrTHN4K2Y2YzQ4NFdxZlNIcXlh?= =?utf-8?B?NUhsODY3eXd2TUpmMEJYWWpITm5sZDlqSEhSRVFSRERScG9vaEt1TnIvdWww?= =?utf-8?B?SS8wOGc3MW0rWGJ2Z3VmdGszZDBDeTdMUmJmSC81UEpGc1ZENlNPcU1EK2pG?= =?utf-8?B?WncwVWJuWjVWaXBIaXF4VEVtdWEvbGwyRlRYRWF2T0lpZ0w2RXlvREpBUnZB?= =?utf-8?B?dllkWDNiajdXY3hXV1BHSDBSOVJyTnJ5ektsT1R3djNwT0FodUdVeU9WOW96?= =?utf-8?B?TWJoWFYwc3dHQW0vTzVxQ0g4WWtqcW40U291Y0xISEJFYnF1Z0NEQlZaQUdr?= =?utf-8?B?Nnl0YVlhMCtqSmpwRktqNEFQZ1I1dDZNbUR1RFljZlh5TGx4UXpTWmE1Yjdz?= =?utf-8?B?My81QkltY3FzNTVFRjBNZUtmdE9Jc0NlYjBNRlRGZGozRE55Y1RFRkZkQ2tW?= =?utf-8?B?UldHQzFSdnlDNlYzckhsdXNIdHlNakRqYUxld1MybzFsb3ZHMmtDN0swKzBk?= =?utf-8?B?NW11Z1o1ME9UMWYxUmg0cnZwNXlwQnRyZm9aVUVuMCtRNlVxYTl1MVJKOVl5?= =?utf-8?B?WWJLY0xuN0hwTC8xTUlOcnBBTTVKUWdFT0FVRVZtSCtkZHJrU05aOVMzTlpN?= =?utf-8?B?ZW0zMzhQN3c3NUtmb1VVSlY2RE9Fc2lJcGlpNXFqek1JRWpKMHpSSm5QVEQ1?= =?utf-8?B?ek9USmhLZVVkbFhHR2dQU3NJZU5OUGNLZFBLSWdqeElWS1BLVnRjU3B1KzNI?= =?utf-8?B?S09Lc3ZPUkZWOGFDTUg0SmJWLzMwNVVuUnFObEdJL2xqMlAxVUlGNVF2RGxl?= =?utf-8?B?NjR2TjNwMHVKR09Hc3VMdmNhOHdKVXllUjlUOEZPRVNFYitPYlJEUEtrSWhu?= =?utf-8?B?ZXcxb21sZFdwZ20vRXFab3ovOWtMUVFzbDlTdk4zSERYZHNSZW5GYnJsRktK?= =?utf-8?B?TFNCVTRmUGRrZW9lVTN1R2duanZLZzMxWVhManVTQnp6djlvT0xUOUYwR05m?= =?utf-8?B?bk55ZExnV29SZWhHYnFVMHlxcFZLaTNpTlByT3ZtaHRBWTNqRWI0ODlEMWp3?= =?utf-8?B?SmhCZGZaU3hxeHFhRXlJYlV5ekJ5QXJZKzhldkVENHZrNzV5Tk9EdHZFYVE3?= =?utf-8?B?akFaYURCQ1N0ampwbWV5WGpvUXdwS3hPSGs5STlwd2tBM2F3Ri82WmtmcThC?= =?utf-8?B?SUtpbEJQbnRYcHprZktOTnpHR1JUWGRVM0JyYytGak9IYllGaTdBU09XeWFP?= =?utf-8?B?MmVjcnUzQnFTamwzbjFJQWlQSURnb2JQTDBUVWlITUZ1Yk5JZllwM05sUm5N?= =?utf-8?B?NnAwRmEzTWVFQXJyMUZjNXkwanR1YjNKK0s4SC95ZWpxQTZzbnZUK1ROaHpw?= =?utf-8?B?NW5yNk40M2s4TWlXK3d0TGRqMHlYRHY1a3BmcmYxWXV5UDNhUHNlV0ZtZ0ZU?= =?utf-8?B?QzZ5Sm95eTdWRUFORW0vd3dWRWhURG1td3NtV3dqNlB1NmVvM2VINkQ0WFJ3?= =?utf-8?B?WmJPOEFJYmd3Ulh1emkzc3NqWjZIQ0ZFektlWlZyUER0WGN3djhDc2xKTnpl?= =?utf-8?B?dkR3K3pIcHZNdlczY0svSUc2SW9XRFNMOFQzTCtTSS9hZVp3aWpuSXdBcFhB?= =?utf-8?B?MlUzSjlWVUFuSnZBbnU4cCtKQmk4UGtyaXlYamlaRFdIUFRIM2ZwNkdHdUJU?= =?utf-8?B?eExHQ2ZQcml5Q1pBdXJkbXRyUGdhOE8xUnJRaURpYk9NMXpwT0tKd01CUEZV?= =?utf-8?B?QkY3S1pKL1hVV1VHRXRrdUpDdmNqZlVBN1c3TUtPVkZrWlhGOVQ4UkJwZ1pH?= =?utf-8?B?WnhCYVNxYXNDckZNZ05pMjF0V2l2RmFjNnlMOXMzdUxKbFRDNmhjcHJLU1Nu?= =?utf-8?B?WTNXbjR5S0lzam9kNXloSW50NVIzRk00YlRWRjJaOWV2bEdBTHFZelJBUVBP?= =?utf-8?B?ZTRMalVFUVNhekJVdWl4bTBMdGRUR0hRYndGd1lUSWJqOUtUd2NkTnp6YThQ?= =?utf-8?B?WEE9PQ==?= MIME-Version: 1.0 X-OriginatorOrg: cji.paris X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MR1P264MB2483.FRAP264.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-Network-Message-Id: baea82a9-c2d0-4448-79e3-08daf7c9fa0d X-MS-Exchange-CrossTenant-originalarrivaltime: 16 Jan 2023 14:00:08.1539 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: fc5390bc-7da7-4e42-b72b-a373be305b62 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: na0vi1fvvIETcSXMm3JYqhUjUNVBuXDlIgjl4Bf5On+PQcNNcHWH80KaTO/oNBVggN5+mkDJQDEUDOqb5j7OWQ+bIBa+/i5MmIyaquwbDjU= X-MS-Exchange-Transport-CrossTenantHeadersStamped: MRZP264MB1670 Subject: Re: [FFmpeg-devel] avformat/mxfenc: fix stored/sampled/displayed width/height 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: >Before the patch: >- stored values were rounded to upper 16 multiple also for formats not using macroblocks (should be st->codecpar->width and >st->codecpar->height when not MPEG formats; note that I found no other >muxer doing the rounding for AVC, only for MPEG-2 Video, but I find no reason in specs for doing the difference so I kept the rounding for AVC) >- sampled and displayed widths were stored width (should be >st->codecpar->width like it is already done for height, with the >DV50/100 exception) The width is one thing; for whatever reason, there is a divergence between DV100 on one hand and AVCI/XDCAMHD35 on the other. In my understanding, in current practices, DV obey s337 (stored width includes scaling) but xdcam&avci does not, so current code is fine but maybe this is an opportunity to document this ? The height is another topic, and in my information (checked against some samples from K2/Harmonic/bmx), DV height should not be rounded : maybe this patch is an opportunity to fix this ? Note: please mind update fate (make fate-lavf-mxf_opatom GEN=1). Nicolas _______________________________________________ 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".