[FFmpeg-devel] [PATCH] Add DICOM Support

Paul B Mahol onemda at gmail.com
Wed Jun 26 10:54:56 EEST 2019


On 6/26/19, Michael Niedermayer <michael at niedermayer.cc> wrote:
> 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 ?
>

To clarify, you mean frame metadata system?


More information about the ffmpeg-devel mailing list