>>> 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 ? >> >>AFAIK: >>- DV in MXF: found old Omneon with no scaling for stored value, no sampled >>value (so stored value), scaled value for displayed value, old Quantel with >>scaling everywhere. From my understanding of spec, I would keep the scaling. >>- MPEG-2 Video including XDCAMHD35 in MXF obey "including any decoder >>scaling or padding" wording with a 16x16 rounding for height, I have no >>file not 1920 or 3840 width >>- AVC in MXF: found old Omneon or old Quantel or old Telestream with no >>padding value for stored value (height of 540 for interlaced). I don't >>understand why it is not same as with MPEG-2 Video so I don't touch FFmpeg >>behavior >there (rounding). Actually checking >again SMPTE ST 381-2013, there >>is an explicit example: "1088: 1080-line progressive". > >I totally agree they are so many weird things in the wild, particularly >looking at some early implementations. I also have fully broken DV100 and >XDCAMHD35 Omneon records with release v6.1 (2010) at the early stages of HD, >but it was fixed afterwards (with many other >issues too!). Looking at GVG, >1440x1088i stored size was implemented from the early beginnings (2010 too) : >sample clips are still available here : >http://www.gvgdevelopers.com/concrete/products/k2/test_clips/ >There is also "kind of" reference sony implementation available here both for >xdcamhd35/avc-1440: https://www.sonycreativesoftware.com/catalystbrowse >Anyway, I think we all agree not to change anything related to MPEG2 and AVC. > >>I don't have DV in MXF with non multiple of 16 (I thought that DV is >>only 720x576 or 720x480 or 1280x720 or 1920x1080, all values multiple of >>16) and don't know about video encoding in DV so I didn't want to change the >>behavior of FFmpeg when I don't know, but case AV_CODEC_ID_DVVIDEO: >>line could be definitely removed if it is fine for you. >DV is questionable. Currently, the dust is under the rug (as a defaults >behaviour), which is an issue with very little concern. >Now, with the patch, the dust become visible, the DV rule is made explicit and >moreover it is presented as an exception, sharing code with macroblock >codecs... I think it is time to fix, even if it was not your prior intention. >I don't have an extensive experience with DV too, I just have samples here and >there like you, but it seems we share the same information. >Let see if someone else react and ask for keeping the current 1088 lines for >DV stored height, but if nobody does, I think it should be okay. > >> Do you want me to add a comment line e.g. "obey 'including any decoder >> scaling or padding' from SMPTE ST 377"? >I am not a core developer and will let others give their feedback. My personal >opinion is that the spec is supposed to be well known and does not require >commenting, but that it would be interesting to explicit why we make a >difference between DV and MPEG2/AVC. And >personally, I don't have the answer >to this question. If nobody has one, maybe a comment could say "because this >is the observed common practice". > >Nicolas
Some weeks later now and no replies, maybe time to go on ? I think the "case AV_CODEC_ID_DVVIDEO:" can be removed as discussed, fate updated and that should be ok for everybody. (Ideally, it could have been an opportunity to document why we have this "DV exception", but I understand it is not very comfortable to write as there is no meaningful reason, so forget about this, this won't hold up the patch anyway) For information, there was a long thread recently on ffmpeg-user about a "bug" in dnxhd stored_height (will be fixed with your patch): https://ffmpeg.org/pipermail/ffmpeg-user/2023-February/056111.html 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-requ...@ffmpeg.org with subject "unsubscribe".