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

Issues

Issue #202 resolved

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

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)

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

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

  3. 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... 😃

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

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

  6. 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?

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

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

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

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

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

  12. Log in to comment