DX procedure_code_meaning not being sensibly populated

Issue #268 resolved
David Platten created an issue

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.

Comments (5)

  1. David Platten reporter

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

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

  3. David Platten reporter

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

  4. Log in to comment