Adapt rdsr.py to import DX RDSR from Canon and Carestream
No description provided.
Comments (17)
-
reporter -
reporter Should only check acquisition device type if this is a projection rdsr. Should fix failed tests. Refs
#464→ <<cset 48f8238f6bf5>>
-
reporter Now populates average_xray_tube_current from the xray_tube_current if it isn't populated. Refs
#464→ <<cset 1efc6ded4664>>
-
reporter Showing anatomy, laterality and targe region in interface. Should review exports... Refs
#464→ <<cset b76b459c98e7>>
-
reporter If num radiographic frames not populated, counts number of events for dx summary display instead. Refs
#464→ <<cset 76dc84a41c60>>
-
reporter @tcdewit I think this is good to go now, I just need to clean up the test files you sent me so I can write some tests
You can see the changes by comparing the branch to develop - mostly as you would expect changes to
rdsr.py
, plus minor changes to the dx templates. -
reporter Realised that the fix for average mA had previously been implemented for mammo RDSR! Refs
#371,#464→ <<cset a9bc1632380a>>
-
reporter See also
#466 -
reporter Adding anonymised Canon RDSR file from a DaRt. Will be used for testing ref
#464→ <<cset 5d385441bfa9>>
-
reporter Carestream DRX Evolution RDSR for testing. Refs
#464→ <<cset bf8956c4c3ac>>
-
reporter Merged issue464DXRDSR into develop. Test scripts need to be written before ref
#464is closed.→ <<cset adcdea0a46b7>>
-
Adding stub dx rdsr tests, Refs
#464→ <<cset 25b719e73993>>
-
Added all tests for Canon dx file and some for Carestream, Refs
#464→ <<cset bf75522b1b24>>
-
Added remaining tests for Carestream dx rdsr file, Refs
#464→ <<cset 312e4650a122>>
-
reporter Commit 9a043b1 on 8th February fixed issues with missing content in Philips Eleva MD as referenced in pull request #90, but a missing hash in the commit message meant it didn't end up here!
-
reporter - changed status to resolved
-
reporter - changed milestone to 0.8.0
-
assigned issue to
- Log in to comment
Added try except to two of the content sequences as CodeMeaning exists but NumericValue doesn't in DX RDSR. Changed logic for identifying fluoro vs radiographic. Explicit field exists, not yet seen! Refs
#464→ <<cset ab2dfc54e9c3>>