[FFmpeg-devel] [PATCH] Add DICOM Support

Michael Niedermayer michael at niedermayer.cc
Wed Jun 26 10:50:45 EEST 2019

On Tue, Jun 25, 2019 at 01:52:09PM +0530, Shivam wrote:
> On 6/25/19 2:12 AM, Michael Niedermayer wrote:
> >On Mon, Jun 24, 2019 at 09:18:13PM +0530, Shivam wrote:
> >>Hi!
> >>
> >>     The code is to add DICOM Support. The patch is only for uncompressed
> >>dicom files using explicit value representation. I would extend it, once i
> >>clarify some doubts.     As dicom image files contain lots of metadata about
> >>the patient. So, should i display that data while demuxing or should i
> >>ignore and only demux the image data ?. In the current patch, i have made an
> >>option "-metadata", which when used will print the data on the terminal
> >>while demuxing.
> >metadata should be exported to be usable by applications.
> >
> >For teh API design a one test is that it should be possible to have a
> >dicom file as input and a format with similar features as output and not
> >loose any significant data.
> >Printing to the terminal cannot achieve that easily.
> So, should i export it to a csv file ?

does it fit into the metadata system we have ?


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190626/7600a4bd/attachment.sig>

More information about the ffmpeg-devel mailing list