DX procedure_code_meaning not being sensibly populated

[David Platten]

The _generalstudymoduleattributes part of the DX extractor routine tries to obtain the procedure_code_meaning:

    g.procedure_code_meaning = get_value_kw('ProtocolName',dataset) # Being used to summarise protocol for study
    if not g.procedure_code_meaning: g.procedure_code_meaning = get_value_kw('SeriesDescription',dataset)

If ProtocolName isn't present then the extractor uses the SeriesDescription instead.

I think it would be more sensible for this to use StudyDescription rather than SeriesDescription, as this is study-level data.

This has come about because I've noticed that some of my radiographic data could be labelled better when ProtocolName is missing.

[David Platten]

Updated dx extractor to use StudyDescription if ProtocolName is not available. If StudyDescription is not available then it falls back to SeriesDescription. References issue #268

→ <<cset a446db82a57b>>

[Ed McDonagh]

Is this change compatible with the changes made for #194? I think it probably is - just want to make sure we aren't undoing the changes made previously!

[David Platten]

This shouldn't affect #194 - it will just populate procedure_code_meaning with a more appropriate value if one is available.

[Ed McDonagh]

Thanks

[David Platten]

I've been using this on my live system and it appears to work.