Compression force in mammography makes GE based assumptions

Issue #135 resolved
Ed McDonagh created an issue

On reading image header for compression force, the extractor code assumes the unit is dN as it is for the GE Senographe DS (though the DICOM conformance statement says N).

This is likely to not be true for other manufacturers - and even possibly later GE models.

@dplatten, @jacole - do you know what unit is used for Compression Force on other mammo units?

Comments (12)

  1. Ed McDonagh reporter

    Added in if statement to check if modality is Senograph DS, on the assumption that all other units are correct. Refs #135. Won't affect existing imports, needs testing.

    → <<cset 582e7e8c95c6>>

  2. Ed McDonagh reporter

    Fixes #135. Now tests for model name containing 'senograph ds' and if so assumes that the compression force is in dN instead of N. Otherwise the value in the Compression Force field is taken as is. Will need to document as this will change behaviour from current for existing users.

    → <<cset a528a6b7b42a>>

  3. Ed McDonagh reporter
    • changed status to open

    Not sure if this was ever right... GE Senographe DS images today have compression force in N. Either way, the code treats the unit as though it were dN, when if it were anything it would be daN!

  4. Ed McDonagh reporter

    Removed compression conversion for GE Senographe DS that is not required (and incorrect). Documented in changes/CHANGES and release notes. Fixes #135, again!

    → <<cset e8f520f52659>>

  5. Log in to comment