openrem_mg.py required dob for age
openrem_mg.py uses the date of birth to set the patient age, even if a patient age is also present. This creates problems for anonymised data where the DoB has been removed but age left.
Comments (8)
-
-
Merged in jacole/openrem-1/jacole/added-an-if-to-catch-patientage-and-cast-1395161457493 (pull request #4)
Refs
#63- needs further testing and the same approach taken with RDSR and Philips CT. Need to make sure pat and patatt are being read in the right order as it seems to be the opposite way around between mam and rdsr. Re:#63Added an if to catch 'PatientAge' and cast from a string to the decimal field, removing the 'Y' for years.→ <<cset a00ebca49ec2>>
-
Modified JAC code to use Decimals instead of floats and to quantize the age to 1 decimal place to reduce accuracy of reverse date of birth calculation. Refs
#63.→ <<cset 7a3d77c38d5e>>
-
Moved patient_age_decimal routine to patientmoduleattributes so that patient_birth_date does not need to be stored in the database. This is already the case for rdsr - mistake has been made in mammo version. Need to check Philips CT. Refs
#63.→ <<cset 3cba739cc792>>
-
Now tested with mammo images with DOB and with no DOB but with age. Restricted Decimal import as per best practice and to avoid errors. Need to duplicate in other DICOM extractors. Refs
#63.→ <<cset 934251881e7d>>
-
Modified rdsr and ct_philips as per mam. Refs
#63.→ <<cset 89e2ac5e943d>>
-
- changed status to resolved
New code implemented and tested in mam.py and rdsr.py; same code exists in ct_philips.py.
-
- changed milestone to 0.4.0
- changed component to Import: All
- Log in to comment
Merged in jacole/openrem-1/jacole/added-an-if-to-catch-patientage-and-cast-1395161457493 (pull request #4)
Refs
#63- needs further testing and the same approach taken with RDSR and Philips CT. Need to make sure pat and patatt are being read in the right order as it seems to be the opposite way around between mam and rdsr. Re:#63Added an if to catch 'PatientAge' and cast from a string to the decimal field, removing the 'Y' for years.→ <<cset a00ebca49ec2>>