[Octave-bug-tracker] [bug #48816] Dicom package - dicominfo incorectly parses multiframe DICOM

[anonymous]

URL:
  <http://savannah.gnu.org/bugs/?48816>

                 Summary: Dicom package - dicominfo incorectly parses
multiframe DICOM 
                 Project: GNU Octave
            Submitted by: None
            Submitted on: Thu 18 Aug 2016 04:14:25 PM UTC
                Category: Octave Forge Package
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: Incorrect Result
                  Status: None
             Assigned to: None
         Originator Name: Matthew Clemence
        Originator Email: address@hidden
             Open/Closed: Open
         Discussion Lock: Any
                 Release: 4.0.3
        Operating System: Any

    _______________________________________________________

Details:

Hi

When parsing multiframe with dicominfo the nested sequences do not appear in
the final structure. Item_XX labels missing and only
one instance of each sequence item stored.

Tested on Linux and windows, Octave 4.0.3, dicom 0.1.1 released
and also compiled from latest mercurial sources.

Example - philips MR enhanced dicom

There seems to be two issues

1)With the code as written, when using assign on octave_map with a map as the
second option, the maps are merged rather than a new sub element created -
requires cast to octave value. (is this a bug with octave_map?)

2) The test file used has sequences of undefined length (see Dicom standard
http://dicom.nema.org/dicom/2013/output/chtml/part05/sect_7.5.html) but these
report an item length of 1 in grassroots dicom. Iterators work ok however.

I have compiled from latest sources on hg and attached my current working
versions. I have left the original sequence dumping fucntions (except for
octave value cast as above) and added a second for the case when Dicom
sequence of undefined length.

the two change sets are 
51 - to fix the build errors (hack). Building with gdcm 2.6.4 from source and
octave-4.0.3 from source. configure doesn't find the libraries correctly in
the final makefile

52 - additional seqdump function for multiframe (with extra logging
information)


The test data I used is rather large to attach - but any enhanced dicom with
sequences of undefined length should show the problem - I work with Philips
data and see this issue there.

happy to discuss, help out and work on this if required and would also like to
get a new release out


Matthew






    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Thu 18 Aug 2016 04:14:25 PM UTC  Name: 51-buildfix.patch  Size: 2kB  
By: None
change sets v source forge dicom package
<http://savannah.gnu.org/bugs/download.php?file_id=38254>
-------------------------------------------------------
Date: Thu 18 Aug 2016 04:14:25 PM UTC  Name: 52-multiframefix.patch  Size: 5kB
  By: None
change sets v source forge dicom package
<http://savannah.gnu.org/bugs/download.php?file_id=38255>

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?48816>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/