1. OpenREM Avatar OpenREM
  2. OpenREM
  3. OpenREM
  4. Issues

Plot of mean CTDIvol for each CT acquisition type would be useful

Issue #202 resolved
David Platten created an issue

It may be possible to incorporate this as a second series on the existing plot of mean DLP for each CT acquisition type.

Comments (33)

  4. David Platten reporter

    Added HTML and JavaScript for the new plot of mean CTDIvol for each acquisition protocol. It is plotted as a seperate plot to the DLP one for the time being. May be good to have it combined. References issue 202.

    → <<cset 29ec220b40ff>>

  5. Ed McDonagh

    May be a silly question; I haven't run your code against real data yet: how do you decide which series to use for the CTDIvol of a procedure?

  6. David Platten reporter

    Added combined DLP and CTDI plot - just testing at the moment. I've had to remove the drill-down to the histogram data at the moment, and the formatting of the values in the tooltips has gone back to ugly defaults. If I can get the drill-down and tooltips working properly then I'd like to use this plot rather than the two seperate ones if the user has selected to look at both DLP and CTDI plots. References issue 202.

    → <<cset 405dd02473ad>>

  7. David Platten reporter

    Hi @edmcdonagh. The plot is just for the CTDIvol of acquisitions, not for studies, so I don't think there is a choice to make - or am I being dim or missing something?

  9. Ed McDonagh

    You're at acquisition level so it's me being dim.

    Really glad you're doing this - we use CTDIvol much more than DLP for these things; now just need to add patient size... 😃

  11. David Platten reporter

    Drilldown now working for both DLP and CTDI series. Need to sort out axes labels and tooltips for the drilldown and drillup. References issue 202.

    → <<cset 7bbfc4f6b197>>

  13. David Platten reporter

    Improved behaviour of the chart. Title of drilldown now changes appropriately. Changes back on drillup. Tooltip information is correct, and continues to work once drilled back up (didn't before). Need to supress the display of the secondary y-axis on drilldown. References issue 202.

    → <<cset 996aae6e2e5a>>

  16. David Platten reporter

    CTDI plot now working properly. References issue 202. I can't get the combined DLP and CTDI plot to work so have commented this out for the time being.

    → <<cset 5bcba0d255c9>>

  17. David Platten reporter

    The combined DLP and CTDI plot is now working. Fixes issue 202. If the user has selected both DLP and CTDI acquisition plots then a combined chart is shown. If they have just selected one or the other then a single plot is shown.

    → <<cset 364bbf1897ae>>

  18. David Platten reporter

    @edmcdonagh, I think you'll find the CTDI chart useful. Are you happy for this branch to be merged back into the main develop one?

  20. David Platten reporter

    Very close. I don't have any plans for more charts at the moment. I should probably write some documentation - I did put some info in the release notes. Do you think there should be some more documentation somewhere?

  21. Ed McDonagh

    We need to do install, release and interface docs. I'm thinking we probably need to remove numpy from the install requires, and include installing it in the install and release notes. Or does that work on both Linux and Windows?

    I need to do similar things for the pynetdicom stuff.

  25. David Platten reporter

    Added HTML and JavaScript for the new plot of mean CTDIvol for each acquisition protocol. It is plotted as a seperate plot to the DLP one for the time being. May be good to have it combined. References issue 202.

    → <<cset 29ec220b40ff>>

  26. David Platten reporter

    Added combined DLP and CTDI plot - just testing at the moment. I've had to remove the drill-down to the histogram data at the moment, and the formatting of the values in the tooltips has gone back to ugly defaults. If I can get the drill-down and tooltips working properly then I'd like to use this plot rather than the two seperate ones if the user has selected to look at both DLP and CTDI plots. References issue 202.

    → <<cset 405dd02473ad>>

  28. David Platten reporter

    Drilldown now working for both DLP and CTDI series. Need to sort out axes labels and tooltips for the drilldown and drillup. References issue 202.

    → <<cset 7bbfc4f6b197>>

  30. David Platten reporter

    Improved behaviour of the chart. Title of drilldown now changes appropriately. Changes back on drillup. Tooltip information is correct, and continues to work once drilled back up (didn't before). Need to supress the display of the secondary y-axis on drilldown. References issue 202.

    → <<cset 996aae6e2e5a>>

  32. David Platten reporter

    CTDI plot now working properly. References issue 202. I can't get the combined DLP and CTDI plot to work so have commented this out for the time being.

    → <<cset 5bcba0d255c9>>

  33. David Platten reporter

    The combined DLP and CTDI plot is now working. Fixes issue 202. If the user has selected both DLP and CTDI acquisition plots then a combined chart is shown. If they have just selected one or the other then a single plot is shown.

    → <<cset 364bbf1897ae>>

  34. Log in to comment
Assignee
Type
enhancement
Priority
minor
Status
resolved
Component
Interface: Data visualisation
Milestone
0.6.0
Votes
0
Watchers
2
Jira: the preferred issue tracker for Bitbucket. Join the team!