Skin maps fail with existing fluoro studies

[Ed McDonagh]

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.

[Ed McDonagh]

This makes existing studies work, assuming HFS. Would be good to record assumptions for display with skin map... Refs #438

→ <<cset a442b3a7efc9>>

[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

[Ed McDonagh]

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

[David Platten]

Yes - I'll do that. Something like "Assumed" or "Extracted" as the prefix.

[Ed McDonagh]

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

[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.

[Ed McDonagh]

Thanks

[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>>

[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>>

[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.