Canon (Toshiba) fluoro RDSR with mGy RP dose units

Issue #808 resolved
David Platten created an issue

A newly-installed Canon Ultimax-i creates DICOM RDSR objects which store dose at reference point values as mGy. These should be stored as Gy. As a result the dose at reference point values in OpenREM are a factor of 1000 too high. Need to implement a check for RP dose units in a similar way to what was used for DAP units in issue #704.

Comments (15)

  1. Ed McDonagh

    Have you got the right field? Haven’t looked through properly yet, but row 21 of TID 10003 is for Mammo and should be in mGy (which is what is currently failing in pipeline)

  2. David Platten reporter

    Removed reference point dose units check in the TID 10003 section as this should be in mGy. Left the check in-place for reference point doses in TID 10003B (, 10004 ( and 10007 ( References issue #808

    → <<cset c966a042feb2>>

  3. Ed McDonagh

    Amen to that!

    While you’re in there, can you update the tags you’ve changed as per this example to use code values rather than code meanings?

    At some point I’ll be changing them all (possibly making use of the new SR capabilities of pydicom, currently write only), but I’m changing as I touch things anyway.

    We’ve had trouble before with manufacturers misspelling the meanings, or just changing the text. And David Clunie says one must never rely on the meanings!

  4. David Platten reporter

    Added test to check that reference point dose is correctly imported from a Canon Ultimax-i where the values are stored in the RDSR in mGy. I can't run tests locally, so not sure if I have the correct event index at the moment. References issue #808

    → <<cset 4d6462985165>>

  5. David Platten reporter

    Now checking against CodeValue instead of CodeMeaning for the dose at reference point data. Will avoid problems where manufacturers sometimes misspell CodeMeaning. References issue #808

    → <<cset a38e6cd7722f>>

  6. Log in to comment