Add extraction of the dose check details for CT

[Luuk]

No description provided.

[Ed McDonagh]

Updating metadata. We should bring this into 0.8.0 as the work has already been done!

[Ed McDonagh]

Refs #491 Add extraction of the dose check details for CT

Implemented extraction of dose check details. Partly tested as no RDSR available with 'reason for proceeding' and/or 'forward estimates'

→ <<cset 6ed0fba59e3e>>

[Ed McDonagh]

Refs #491

Commit to fix error with Dose Check Notification Details.

→ <<cset a48cadd145d8>>

[Ed McDonagh]

Merged in LuukO/openrem/issue491ExtractDoseCheckDetails (pull request #73) to enable testing and review

Refs #491 Add extraction of the dose check details for CT

→ <<cset 14ca8d4feeef>>

[Ed McDonagh]

Added presentation of any dose check data in the CT detail page along with the series level comments. Refs #491

→ <<cset 95bd86d906a8>>

[Ed McDonagh]

Added _person_participant function to extract person information, applied it to _ct_dose_check_details. Added display of person name in web interface. Refs #491

→ <<cset 718451537a27>>

[Ed McDonagh]

Adjusting line break in code to ensure it doesn't create an empty line when there is no dose check data. Refs #491

→ <<cset 7173ba6dd1bd>>

[Ed McDonagh]

Beginning to replicate dose check information in CT export. Refs #491

→ <<cset 07bf00af495f>>

[Ed McDonagh]

Alerts data now exporting as per web interface. Now to add notifications. @LuukO, @dplatten: I'm putting all this data in prose in one cell for each series. Individually each element could take up a dozen cells for each series. Do you agree with my approach? Refs #491

→ <<cset 35357000311b>>

[Ed McDonagh]

Added notifications to export. Corrected a few references to alert in the notifications section of web interface text. Now need to add tests. Refs #491

→ <<cset 5774dc05234a>>

[David Platten]

@edmcdonagh, I don't think that taking up a dozen cells for this data is a good idea, so I think what you've done is fine.

[Ed McDonagh]

Thanks David. My concern is that for those who want to care about this technique (I think it is very ill-judged myself!), they won't be able to sort or colour-code based on alert or notification values being exceeded.

Once I've de-identified the example @LuukO gave me I'll be able to add it into the repository and you can see what it looks like.

[Luuk]

I think you're right to fit all data in one cell per series. And you don't need to de-indentify my example. That was already done. Or do you want to de-identify also the hospital and manufacturer / model (by the way, you might have seen an unknown model, but Toshiba just officially launched it at RSNA)

[Ed McDonagh]

I tend to de-identify the hospital and serial numbers etc as well as the patient details. But I like to keep the manufacturer and model as it is useful to keep that information.

[Ed McDonagh]

Adding test file. Refs #491

→ <<cset cf4a5b2d8f66>>

[Ed McDonagh]

Adding test function Refs #491

→ <<cset c1e433b7d7b3>>

[Ed McDonagh]

Removing unused import Refs #491

→ <<cset 0b5665afc974>>

[Ed McDonagh]

Added new Toshiba file to export test, checks for dose check strings. Refs #491

→ <<cset 619ee453fb6a>>

[Ed McDonagh]

Adding ref #491 to changes

→ <<cset f8a906d19b6b>>

[Ed McDonagh]

Merged in issue491ExtractDoseCheckDetails (pull request #137)

Issue491ExtractDoseCheckDetails

Fixes #491

→ <<cset 5bfe8f63aa74>>