Add radiographic scatter plot of DDI vs. DAP for each acquisition protocol
I would find it useful to have a scatter plot of DDI vs. DAP for each radiographic acquisition protocol.
Comments (18)
-
-
reporter Added scatter plots for DAP vs DDI for radiography acquisitions. There's a chart for both IEC and vendor DDI. I've altered the scatter plot code so that a series is produced for each acquisition protocol. If the user asks to plot a series per system then you get a series per system per acquisition protocol. This can take a long time to calculate, or even hang... It does do what I'd like it to do, though. References issue #574
→ <<cset 84b1ef20264e>>
-
reporter I should have added that these changes have not affected the mammography scatter plots. They still function as they did. I could now alter the mammography scatter charts so that they also have a series per acquisition protocol.
-
reporter Due to the performance of these charts it may be prudent to wait for issue
#477, which may include an overhaul of how data for charts, and their plotting, is carried out. -
I haven't looked at the database queries for the charts, but I have learned a lot in optimising the interface display and exports, so there might be big improvements to be made to charts if they are operating in a similar fashion?
-
reporter @edmcdonagh, it would be good if you could take a look at the chart queries - perhaps we could put our heads together at some point?
-
reporter Fixed conflict. Merged develop into DDIvsDAP branch. References issue #574
→ <<cset 229696be2115>>
-
reporter Fixed JavaScript error that happens when there's no data points in a scatter plot series. References issue #574
→ <<cset 88ddafdda4ee>>
-
reporter Darkened colours of scatter plot points to ensure series are visible. Changed mammo scatter plot colour scheme to match other modalities now that I've darkened the colours for the scatter plots. References issue #574 and issue
#390→ <<cset bd5b396b137a>>
-
reporter @edmcdonagh, I'm wondering if we should include these DDI vs DAP charts in 0.8. Any thoughts?
-
I haven't looked at them... can you chuck it onto testing?
-
reporter Done
-
reporter The DAP vs vendor DDI on testing.openrem.org shows that I randomised the DAP in the fake DICOM objects, but not the DDI...
-
Oops!
Have you noticed on the vs IEC chart that the 'OpenREM CAREDRXEVO right thigh' data points are half off the edge of the chart?
There is a similar thing on the vendor DDI chart - does the chart need to be a little wider?
-
@David Platten I think this is not included in the current charts? Was that a conscious decision?
-
reporter @Ed McDonagh no, it wasn’t a conscious decision. Not convinced that I would find them useful as the DAP is (obviously) highly dependent on field size, whereas the DDI is not.
-
Leave open as a maybe for the future? Or close as wontfix?
-
reporter I think leave open for now.
- Log in to comment
I forgot to ask earlier: which DDI are you planning to use?