Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Pierre-Anthony Lemieux <pal@sandflow.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v2 2/2] fate/mxf: add JPEG 2000 test
Date: Tue, 6 Sep 2022 14:26:02 -0700
Message-ID: <CAF_7JxC0X646KPMv7UaAKB1CTwTmEyR9Wtk8XkSZLPoUJGxM5Q@mail.gmail.com> (raw)
In-Reply-To: <GV1P250MB0737182ECF7132776F96DA6D8F7E9@GV1P250MB0737.EURP250.PROD.OUTLOOK.COM>

On Tue, Sep 6, 2022 at 2:24 PM Andreas Rheinhardt
<andreas.rheinhardt@outlook.com> wrote:
>
> pal@sandflow.com:
> > From: Pierre-Anthony Lemieux <pal@palemieux.com>
> >
> > ---
> >  tests/fate/mxf.mak           |  4 ++
> >  tests/ref/fate/mxf-probe-j2k | 78 ++++++++++++++++++++++++++++++++++++
> >  2 files changed, 82 insertions(+)
> >  create mode 100644 tests/ref/fate/mxf-probe-j2k
> >
> > diff --git a/tests/fate/mxf.mak b/tests/fate/mxf.mak
> > index 3ab936b5de..38d1c2ef38 100644
> > --- a/tests/fate/mxf.mak
> > +++ b/tests/fate/mxf.mak
> > @@ -29,6 +29,10 @@ FATE_MXF_PROBE-$(call ENCDEC, DNXHD, MXF) += fate-mxf-probe-dnxhd
> >  fate-mxf-probe-dnxhd: SRC = $(TARGET_SAMPLES)/mxf/multiple_components.mxf
> >  fate-mxf-probe-dnxhd: CMD = run $(PROBE_FORMAT_STREAMS_COMMAND) -i "$(SRC)"
> >
> > +FATE_MXF_PROBE-$(call ENCDEC, JPEG2000, MXF) += fate-mxf-probe-j2k
>
> ENCDEC? Since when does ffprobe run encoders at all?

I did wonder, but ended-up using the same template as fate-mxf-probe-dnxhd.

Should they all be changed to DEC?

>
> > +fate-mxf-probe-j2k: SRC = $(TARGET_SAMPLES)/imf/countdown/countdown-small.mxf
> > +fate-mxf-probe-j2k: CMD = run $(PROBE_FORMAT_STREAMS_COMMAND) -i "$(SRC)"
> > +
> >  FATE_MXF_PROBE-$(call ENCDEC2, DVVIDEO, PCM_S16LE, MXF) += fate-mxf-probe-dv25
> >  fate-mxf-probe-dv25: SRC = $(TARGET_SAMPLES)/mxf/Avid-00005.mxf
> >  fate-mxf-probe-dv25: CMD = run $(PROBE_FORMAT_STREAMS_COMMAND) -i "$(SRC)"
> > diff --git a/tests/ref/fate/mxf-probe-j2k b/tests/ref/fate/mxf-probe-j2k
> > new file mode 100644
> > index 0000000000..2dbf2ac37d
> > --- /dev/null
> > +++ b/tests/ref/fate/mxf-probe-j2k
> > @@ -0,0 +1,78 @@
> > +[STREAM]
> > +index=0
> > +codec_name=jpeg2000
> > +profile=1798
> > +codec_type=video
> > +codec_tag_string=[0][0][0][0]
> > +codec_tag=0x0000
> > +width=640
> > +height=360
> > +coded_width=640
> > +coded_height=360
> > +closed_captions=0
> > +film_grain=0
> > +has_b_frames=0
> > +sample_aspect_ratio=1:1
> > +display_aspect_ratio=16:9
> > +pix_fmt=rgb48le
> > +level=-99
> > +color_range=unknown
> > +color_space=unknown
> > +color_transfer=bt709
> > +color_primaries=bt709
> > +chroma_location=unspecified
> > +field_order=progressive
> > +refs=1
> > +id=N/A
> > +r_frame_rate=24/1
> > +avg_frame_rate=24/1
> > +time_base=1/24
> > +start_pts=0
> > +start_time=0.000000
> > +duration_ts=24
> > +duration=1.000000
> > +bit_rate=N/A
> > +max_bit_rate=N/A
> > +bits_per_raw_sample=16
> > +nb_frames=N/A
> > +nb_read_frames=N/A
> > +nb_read_packets=N/A
> > +DISPOSITION:default=0
> > +DISPOSITION:dub=0
> > +DISPOSITION:original=0
> > +DISPOSITION:comment=0
> > +DISPOSITION:lyrics=0
> > +DISPOSITION:karaoke=0
> > +DISPOSITION:forced=0
> > +DISPOSITION:hearing_impaired=0
> > +DISPOSITION:visual_impaired=0
> > +DISPOSITION:clean_effects=0
> > +DISPOSITION:attached_pic=0
> > +DISPOSITION:timed_thumbnails=0
> > +DISPOSITION:captions=0
> > +DISPOSITION:descriptions=0
> > +DISPOSITION:metadata=0
> > +DISPOSITION:dependent=0
> > +DISPOSITION:still_image=0
> > +TAG:file_package_umid=0x060A2B340101010501010F201300000035E05073878E4B2FB69D2369F25ADFC9
> > +TAG:file_package_name=File Package: SMPTE ST 422 / ST 2067-5 frame wrapping of JPEG 2000 codestreams
> > +TAG:track_name=Image Track
> > +[/STREAM]
> > +[FORMAT]
> > +format_name=mxf
> > +duration=1.000000
> > +bit_rate=577792
> > +TAG:operational_pattern_ul=060e2b34.04010101.0d010201.01010100
> > +TAG:uid=f1994e51-a844-49e4-9459-1ddd622eb65d
> > +TAG:generation_uid=1be151ac-cc95-4314-b09f-7420eda9932b
> > +TAG:company_name=Sandflow Consulting LLC
> > +TAG:product_name=dcdm2imf
> > +TAG:product_version_num=0.0.0.0.0
> > +TAG:product_version=1.0-beta1
> > +TAG:product_uid=927fc4d1-89a3-4f88-88bb-d363ed33084a
> > +TAG:modification_date=2022-01-07T22:05:01.000000Z
> > +TAG:toolkit_version_num=2.10.38.27240.1
> > +TAG:application_platform=win32
> > +TAG:material_package_umid=0x060A2B340101010501010F201300000072BAF0557DA749308C14738BCD4FA116
> > +TAG:material_package_name=Material Package
> > +[/FORMAT]
>
> _______________________________________________
> 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".
_______________________________________________
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".

  reply	other threads:[~2022-09-06 21:26 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-05 18:29 [FFmpeg-devel] [PATCH v2 1/2] avformat/mxf: set stream frame rates for ST 422 essence containers pal
2022-09-05 18:29 ` [FFmpeg-devel] [PATCH v2 2/2] fate/mxf: add JPEG 2000 test pal
2022-09-06 11:47   ` Tomas Härdin
2022-09-06 21:23   ` Andreas Rheinhardt
2022-09-06 21:26     ` Pierre-Anthony Lemieux [this message]
2022-09-06 21:33       ` Andreas Rheinhardt
2022-09-06 21:52         ` Pierre-Anthony Lemieux

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAF_7JxC0X646KPMv7UaAKB1CTwTmEyR9Wtk8XkSZLPoUJGxM5Q@mail.gmail.com \
    --to=pal@sandflow.com \
    --cc=ffmpeg-devel@ffmpeg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git