Skin map with head doesn't render correctly in 2D view
In 2D view, with the head on the phantom, the lower part of the torso is rendered below the bottom of the canvas.
In 3D view, it all renders as expected.
Comments (13)
-
reporter -
I’ll have a look
-
Hi @Ed McDonagh what portion of the lower part of the phantom? Literally on the edge?
could you provide a screen shot?
What would also be an improvement in the case presented above, is to shift the first X rows of the skin dose map to make the head part align with the body
-
reporter Ooh, what is going on in your screenshot? Where is the dose on the head coming from? Is that wrapped around from the fields at the lower end of the torso or something?
I can’t remember which study I saw this on first when I was testing something else, but if you have a look at the studies in https://dev.openrem.org you will see the same thing (which isn’t like what you have above). For example, https://dev.openrem.org/openrem/rf/19/ has exposure at the bottom of the 2D canvas but in the middle of the 3D torso. In the example I first saw this on there was a sliver of exposure showing on the 2D canvas at the lower edge, but it was nicely on the torso in the 3D.
-
Hi Ed, that’s interesting indeed… I will have a look into it. It’s odd that my version appears to render the lower part of the torso fine in 2D.
As for the phantom head part in 2D that is not aligned with body: It’s def. phantom head radiation showing on top but it has to do with the phantom map class. I simply expanded the phantommap array to add it’s cylindrical head but used the same “resolution” so to say. Head and phantombody dose data is contained within a single array, this array has a sparse part that does not correspond to a physical location on the computational phantom. It looks like this and this also explains why dose data corresponding to the phantom’s head appears on the far left in my previous 2D skin dose image.
-
@Ed McDonagh Hi ed I’ve found the cause of the problem you detected, I’ve adjusted the canvas height. Will push it tomorrow
-
Fixed 2D JS canvas size [skip ci]. Ref issue
#910→ <<cset e27afd2def3a>>
-
I think that the head height needs to be added to the height in the Ajax function, not to the width. I've also added a comment to the user if a skin dose map fails to say that it may be due to skin dose maps being disabled for that system type. Refs issue
#910→ <<cset eb2bc6673350>>
-
2D map using commit e27afd2def3a (Wens, adding head height to phantom width):
2D map using commit eb2bc6673350 (DP, adding head height to phantom height):
-
Yes it should be added to its height indeed. I admit I was in a hurry when I copied it from my institution’s laptop
-
Updated changes file [skip ci]. Refs issue
#910→ <<cset 145d4747896f>>
-
Resolved merge conflict in changes file. Refs issue
#910→ <<cset 57aac73bf97f>>
-
reporter - changed status to resolved
Merged in issue910render2D (pull request #468)
Fixes
#910Approved-by: Ed McDonagh
→ <<cset e86a53e6555c>>
- Log in to comment
@David Platten @wens have you seen this too?