Skin maps fail with existing fluoro studies
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)
-
reporter -
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
-
reporter Thanks @dplatten. I'd like to be able to see whether it is assumed or extracted from source. Is that your intention?
-
Yes - I'll do that. Something like "Assumed" or "Extracted" as the prefix.
-
reporter Excellent, thanks. I haven't checked the code - does the same thing happen for patient size attributes?
-
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.
-
reporter Thanks
-
Skin dose maps now display either
Assumed
orExtracted
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 asHead-first supine
, rather thanHFS
, but can't work out how to do this at the moment (ideas welcome...). References issue#438and#61→ <<cset 80713e1992b4>>
-
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#438and#61→ <<cset 83b26f71bb8d>>
-
- changed status to resolved
@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.
- Log in to comment
This makes existing studies work, assuming HFS. Would be good to record assumptions for display with skin map... Refs
#438→ <<cset a442b3a7efc9>>