1. OpenREM
  2. OpenREM
  3. OpenREM
  4. Issues

Issues

Issue #137 resolved

Multiple values in filter thickness causes ValueError

Ed McDonagh
created an issue

Carestream have two values in the filter thickness fields when both Copper and Aluminium is in use.

The field is a Decimal String value representation, so two numbers with a comma in the middle is not permitted, and causes pydicom to bomb out with a ValueError as it can't convert the string to a float.

Comments (12)

  1. Ed McDonagh reporter

    Added try/except ValueError to filter thickness to gracefully handle Carestream use of two comma separated decimals in a Decimal String field. Commented out code that was to deal with the string that I can't get hold of. Refs #137.

    → <<cset 6e0cc2cc0d91>>

  2. Ed McDonagh reporter

    Tested successfully but discovered that the modal only allows for two decimal places. Created issue #154 to address.

    Issue with max and min in the wrong fields is directly from the source data.

  3. Log in to comment