Skin maps fail with existing fluoro studies

Issue #438 resolved
Ed McDonagh created an issue

make_skin_map sets patient position to None if it isn't recorded in the database. I think it should be set to HFS instead.

As it is set to None, there are subsequent routines that either don't set to HFS because patPos does have a value (ie None), or the routine simply quits passing an error message.

Comments (10)

  1. David Platten

    It will be easy to add the display of patient orientation. I can add it below the four items already there (I would prefer the wordy description, rather than the acronym):

    • Calculated peak skin dose: 0.0018 Gy
    • Phantom dimensions: 70x34x20 cm (HxWxD)
    • Assumed patient height: 1.79 m
    • Assumed patient mass: 73.2
    • Assumed patient orientation: head-first supine
  2. Ed McDonagh reporter

    Thanks @dplatten. I'd like to be able to see whether it is assumed or extracted from source. Is that your intention?

  3. Ed McDonagh reporter

    Excellent, thanks. I haven't checked the code - does the same thing happen for patient size attributes?

  4. David Platten

    The "Assumed" wording is hard-written in for the patient size attributes. I'll make it depend on whether defaults have been used. I'm doing these modifications in the issue438Noneposition branch. Will make some commits this evening.

  5. David Platten

    Skin dose maps now display either Assumed or Extracted for patient height, mass and orientation information depending on whether the corresponding information is available in the database. I think I'd prefer verbose orientations to be displayed, such as Head-first supine, rather than HFS, but can't work out how to do this at the moment (ideas welcome...). References issue #438 and #61

    → <<cset 80713e1992b4>>

  6. David Platten

    phantom width and height were being calculated as float values, causing an error with the creation of Numpy arrays. Forced width and height to be int. References issue #438 and #61

    → <<cset 83b26f71bb8d>>

  7. David Platten

    @edmcdonagh, I'd like this to be merged back in to develop, as it is causing me some problems. The only thing that's not completely sorted is that the full text of the orientation is not currently being displayed on the skin dose map.

  8. Log in to comment