Re: [FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
Thanks, Stefano, I agree it’s not ideal with just having magic numbers. If I think of a creative solution I'll let you know. Appreciate your time, On Sun, Mar 10, 2024 at 09:56 Stefano Sabatini wrote: > On date Saturday 2024-03-09 19:56:49 -0600, Marth64 wrote: > > Following up on this from December 2023. I simplified the content and > > hopefully addressed the feedback. > > > > Signed-off-by: Marth64 > > --- > > doc/bitstream_filters.texi | 15 +++ > > 1 file changed, 15 insertions(+) > > > > diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi > > index e06de1a73a..61539d2473 100644 > > --- a/doc/bitstream_filters.texi > > +++ b/doc/bitstream_filters.texi > > @@ -213,6 +213,21 @@ To remove all AUDs, SEI and filler from an H.265 > stream: > > ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' > OUTPUT > > @end example > > > > +To remove all user data from a MPEG-2 stream, including Closed Captions: > > +@example > > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=178' OUTPUT > > +@end example > > + > > +To remove all SEI from a H264 stream, including Closed Captions: > > +@example > > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=6' OUTPUT > > +@end example > > + > > +To remove all prefix and suffix SEI from a HEVC stream, including > Closed Captions and dynamic HDR: > > +@example > > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=39|40' > OUTPUT > > +@end example > > + > > Not against, but I'm not super convinced this is super useful as it > does not really explain what these values come from. Probably it > would be more useful a table, or even better make the parser somehow > expose the supported types with the meaning (this would enable having > e.g. a symbolic type "cc" abstracting the containter format). > > OTOH I agree thius would provide some practical examples, therefore > I'll apply while we have no smarter way to expose the logic in a more > effective way. > > Thanks. > ___ 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".
Re: [FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
On date Saturday 2024-03-09 19:56:49 -0600, Marth64 wrote: > Following up on this from December 2023. I simplified the content and > hopefully addressed the feedback. > > Signed-off-by: Marth64 > --- > doc/bitstream_filters.texi | 15 +++ > 1 file changed, 15 insertions(+) > > diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi > index e06de1a73a..61539d2473 100644 > --- a/doc/bitstream_filters.texi > +++ b/doc/bitstream_filters.texi > @@ -213,6 +213,21 @@ To remove all AUDs, SEI and filler from an H.265 stream: > ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' OUTPUT > @end example > > +To remove all user data from a MPEG-2 stream, including Closed Captions: > +@example > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=178' OUTPUT > +@end example > + > +To remove all SEI from a H264 stream, including Closed Captions: > +@example > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=6' OUTPUT > +@end example > + > +To remove all prefix and suffix SEI from a HEVC stream, including Closed > Captions and dynamic HDR: > +@example > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=39|40' OUTPUT > +@end example > + Not against, but I'm not super convinced this is super useful as it does not really explain what these values come from. Probably it would be more useful a table, or even better make the parser somehow expose the supported types with the meaning (this would enable having e.g. a symbolic type "cc" abstracting the containter format). OTOH I agree thius would provide some practical examples, therefore I'll apply while we have no smarter way to expose the logic in a more effective way. Thanks. ___ 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".
[FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
Following up on this from December 2023. I simplified the content and hopefully addressed the feedback. Signed-off-by: Marth64 --- doc/bitstream_filters.texi | 15 +++ 1 file changed, 15 insertions(+) diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi index e06de1a73a..61539d2473 100644 --- a/doc/bitstream_filters.texi +++ b/doc/bitstream_filters.texi @@ -213,6 +213,21 @@ To remove all AUDs, SEI and filler from an H.265 stream: ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' OUTPUT @end example +To remove all user data from a MPEG-2 stream, including Closed Captions: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=178' OUTPUT +@end example + +To remove all SEI from a H264 stream, including Closed Captions: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=6' OUTPUT +@end example + +To remove all prefix and suffix SEI from a HEVC stream, including Closed Captions and dynamic HDR: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=39|40' OUTPUT +@end example + @section hapqa_extract Extract Rgb or Alpha part of an HAPQA file, without recompression, in order to create an HAPQ or an HAPAlphaOnly file. -- 2.34.1 ___ 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".
Re: [FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
On date Saturday 2023-12-09 19:35:50 -0600, Marth64 wrote: > --- > doc/bitstream_filters.texi | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi > index c63c20370f..d2ad6e78ff 100644 > --- a/doc/bitstream_filters.texi > +++ b/doc/bitstream_filters.texi > @@ -213,6 +213,24 @@ To remove all AUDs, SEI and filler from an H.265 stream: > ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' OUTPUT > @end example > > +SEI messages are commonly used to contain Closed Captions as well as other > metadata such as dynamic HDR. > +This filter can be used to remove the SEI messages if desired. > + > +To remove SEI messages (including CC) from a MPEG-2 stream: > +@example > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=178' OUTPUT > +@end example > + > +To remove SEI messages (including CC) from a H264 stream: > +@example > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=6' OUTPUT > +@end example > + > +To remove SEI messages (including CC and dynamic HDR) from a HEVC stream: > +@example > +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=39' OUTPUT > +@end example Shall be good, but probably we can still do a bit better, by making explicit the common pattern. Something as: ... SEI messages are commonly used to contain Closed Captions as well as other metadata such as dynamic HDR. This filter can be used to remove the SEI messages, including CC, given the bitstream SEI unit type (which changes depending on the video stream). To remove SEI messages (including CC) from a MPEG-2 stream: ... ___ 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".
Re: [FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
Thank you! On Sat, Dec 9, 2023 at 9:06 PM Leo Izen wrote: > On 12/9/23 20:35, Marth64 wrote: > > --- > > doc/bitstream_filters.texi | 18 ++ > > 1 file changed, 18 insertions(+) > > > > Make sure when you generate the second version of a patch with git > format-patch you pass the -v2 argument, e.g. > > git format-patch -v2 -o outgoing/ master > > Otherwise it can get a little bit difficult for reviewers to track the > versions. I have no comments on the patch itself, in either direction. > > - Leo Izen (Traneptora) > ___ > 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". > ___ 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".
Re: [FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
On 12/9/23 20:35, Marth64 wrote: --- doc/bitstream_filters.texi | 18 ++ 1 file changed, 18 insertions(+) Make sure when you generate the second version of a patch with git format-patch you pass the -v2 argument, e.g. git format-patch -v2 -o outgoing/ master Otherwise it can get a little bit difficult for reviewers to track the versions. I have no comments on the patch itself, in either direction. - Leo Izen (Traneptora) ___ 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".
[FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
--- doc/bitstream_filters.texi | 18 ++ 1 file changed, 18 insertions(+) diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi index c63c20370f..d2ad6e78ff 100644 --- a/doc/bitstream_filters.texi +++ b/doc/bitstream_filters.texi @@ -213,6 +213,24 @@ To remove all AUDs, SEI and filler from an H.265 stream: ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' OUTPUT @end example +SEI messages are commonly used to contain Closed Captions as well as other metadata such as dynamic HDR. +This filter can be used to remove the SEI messages if desired. + +To remove SEI messages (including CC) from a MPEG-2 stream: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=178' OUTPUT +@end example + +To remove SEI messages (including CC) from a H264 stream: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=6' OUTPUT +@end example + +To remove SEI messages (including CC and dynamic HDR) from a HEVC stream: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=39' OUTPUT +@end example + @section hapqa_extract Extract Rgb or Alpha part of an HAPQA file, without recompression, in order to create an HAPQ or an HAPAlphaOnly file. -- 2.34.1 ___ 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".
Re: [FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
Appreciate the good feedback, will update. Thank you. On Sat, Dec 9, 2023 at 6:53 PM Stefano Sabatini wrote: > On date Saturday 2023-12-09 13:31:13 -0600, Marth64 wrote: > > --- > > doc/bitstream_filters.texi | 17 + > > 1 file changed, 17 insertions(+) > > > > diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi > > index 145aa8684e..b01ede0c93 100644 > > --- a/doc/bitstream_filters.texi > > +++ b/doc/bitstream_filters.texi > > @@ -213,6 +213,23 @@ To remove all AUDs, SEI and filler from an H.265 > stream: > > ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' > OUTPUT > > @end example > > > > > +By nature of removing SEI messages, the filter can also remove Closed > Captions. > > I think you might expand this to shortly explain the relation between > Closed Captions and SEI message. Also, I'd avoid the expression "by > nature" and replace with a plainer/more explicit expression (as most > of the readers are not English speakers). > > [...] > ___ 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".
Re: [FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
On date Saturday 2023-12-09 13:31:13 -0600, Marth64 wrote: > --- > doc/bitstream_filters.texi | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi > index 145aa8684e..b01ede0c93 100644 > --- a/doc/bitstream_filters.texi > +++ b/doc/bitstream_filters.texi > @@ -213,6 +213,23 @@ To remove all AUDs, SEI and filler from an H.265 stream: > ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' OUTPUT > @end example > > +By nature of removing SEI messages, the filter can also remove Closed > Captions. I think you might expand this to shortly explain the relation between Closed Captions and SEI message. Also, I'd avoid the expression "by nature" and replace with a plainer/more explicit expression (as most of the readers are not English speakers). [...] ___ 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".
[FFmpeg-devel] [PATCH] doc/bitstream_filters: add filter_units practical examples for removing closed captions
--- doc/bitstream_filters.texi | 17 + 1 file changed, 17 insertions(+) diff --git a/doc/bitstream_filters.texi b/doc/bitstream_filters.texi index 145aa8684e..b01ede0c93 100644 --- a/doc/bitstream_filters.texi +++ b/doc/bitstream_filters.texi @@ -213,6 +213,23 @@ To remove all AUDs, SEI and filler from an H.265 stream: ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=35|38-40' OUTPUT @end example +By nature of removing SEI messages, the filter can also remove Closed Captions. + +To remove SEI messages (including CC) from a MPEG-2 stream: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=178' OUTPUT +@end example + +To remove SEI messages (including CC) from a H264 stream: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=6' OUTPUT +@end example + +To remove SEI messages (including CC and dynamic HDR) from a HEVC stream: +@example +ffmpeg -i INPUT -c:v copy -bsf:v 'filter_units=remove_types=39' OUTPUT +@end example + @section hapqa_extract Extract Rgb or Alpha part of an HAPQA file, without recompression, in order to create an HAPQ or an HAPAlphaOnly file. -- 2.34.1 ___ 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".