search

ImagingDataCommons / ETL

(CORE REPO)
Apache License 2.0
0 stars 1 forks source link

Values for `dicom_patient_id` are invalid for the `acrin_6698` collection #43

Closed fedorov closed 1 year ago

fedorov commented 2 years ago

It appears that the clinical table is referring to the patients by suffixes of DICOM PatientID. To get the actual patient ID we need to prepend that suffix with ACRIN-6698-.

Going forward, when populating dicom_patient_id column, there should be a check that all IDs included in that column for the specific clinical data table are all present among the values of PatientID selected from dicom_all for that specific table.

G-White-ISB commented 2 years ago

Will add a check of all IDs by v12.

G-White-ISB commented 2 years ago

The ACRIN 6698 dicom_patient_id values now have the correct form. The clinical data exporter now checks these values against the PatientID value in dicom_all when the clinical dataset is created