Importing Philips Digital Diagnost DX images fails with multiple filters
Issue #593
resolved
As reported by Mark Pearce on the google groups:
Traceback (most recent call last):
File "/usr/local/bin/openrem_dx.py", line 24, in <module>
dx(filename)
File "/usr/local/lib/python2.7/dist-packages/celery/local.py", line 191, in __call__
return self._get_current_object()(*a, **kw)
File "/usr/local/lib/python2.7/dist-packages/celery/app/task.py", line 380, in __call__
return self.run(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/openrem/remapp/extractors/dx.py", line 815, in dx
_dx2db(dataset)
File "/usr/local/lib/python2.7/dist-packages/openrem/remapp/extractors/dx.py", line 714, in _dx2db
_generalstudymoduleattributes(dataset, g)
File "/usr/local/lib/python2.7/dist-packages/openrem/remapp/extractors/dx.py", line 621, in _generalstudymoduleattributes
_projectionxrayradiationdose(dataset, g, ch)
File "/usr/local/lib/python2.7/dist-packages/openrem/remapp/extractors/dx.py", line 443, in _projectionxrayradiationdose
_irradiationeventxraydata(dataset, proj, ch)
File "/usr/local/lib/python2.7/dist-packages/openrem/remapp/extractors/dx.py", line 404, in _irradiationeventxraydata
_irradiationeventxraysourcedata(dataset, event)
File "/usr/local/lib/python2.7/dist-packages/openrem/remapp/extractors/dx.py", line 281, in _irradiationeventxraysourcedata
_xray_filters_prep(dataset, source)
File "/usr/local/lib/python2.7/dist-packages/openrem/remapp/extractors/dx.py", line 164, in _xray_filters_prep
xray_filter_thickness_minimum, source
File "/usr/local/lib/python2.7/dist-packages/openrem/remapp/extractors/dx.py", line 71, in _xrayfilters
if material.strip().lower() == 'molybdenum':
AttributeError: 'MultiValue' object has no attribute 'strip'
Error confirmed on my install.
Comments (3)
-
reporter -
reporter Modified the creation of the MultiValue filter material element to replicate the issue picked up in ref
#593- test will now fail.→ <<cset 3f7e3aa1dee1>>
-
reporter - changed status to resolved
Merged in issue593PhilipsDiDifilterimport (pull request #160)
Changed test for filter materials as list, allow non-US spelling of aluminum/aluminium; fixes
#593→ <<cset 3bbb6b81020c>>
- Log in to comment
Changed test for filter materials as list, allow non-US spelling of aluminum/aluminium; both to ref
#593→ <<cset 39cd0bd819d7>>