Modify dx.py to correctly extract information from some more DR systems

Issue #482 resolved
David Platten
created an issue

Fuji Go digital mobiles Once imported, acquisition protocol is blank , and requested procedure name contains the acquisition protocol.

Toshiba Radrex digital rooms Acquisition protocols not imported at the moment.

Comments (21)

  1. Ed McDonagh

    As confirmation, I have run the test locally with your latest additions and everything works fine:

    (veOpenREM)me@ubuntu:~/research/bbOpenREM$ cd openrem/
    (veOpenREM)me@ubuntu:~/research/bbOpenREM/openrem$ python manage.py test remapp
    Creating test database for alias 'default'...
    ..............................
    ----------------------------------------------------------------------
    Ran 30 tests in 32.678s
    
    OK
    Destroying test database for alias 'default'...
    (veOpenREM)me@ubuntu:~/research/bbOpenREM/openrem$ 
    
  2. David Platten reporter

    I need to make a further change. For two of our Carestream digital mobiles the requested procedure name is not populated on import. For these systems the data is present in the StudyDescription location (0008,1030). This location is already used to retrieve requested procedure names for Canon CXDI equipment studies.

  3. David Platten reporter

    Added code to extract requested procedure name from StudyDescription location for Carestream DRX-Revolution digital mobiles. The extractor only resorts to this location if it has failed to find it in other more sensible places first. This fixes an issue with two of my digital mobiles. References issue #482

    → <<cset 70dd74bb4db4>>

  4. David Platten reporter

    Added a test to see if the requested procedure name is imported correctly from a skeleton Carestream DRX Revolution radiographic DICOM file. Couldn't check if this worked locally as it was unable to create a database, and then asked if it could delete my existing database. References issue #482.

    → <<cset 56075f80226a>>

  5. Log in to comment