aces / Loris-MRI

The set of scripts that preprocess and insert MRI data into the database.
10 stars 50 forks source link

`run_dicom_archive_validation.py` not returning error when providing wrong uploadID/tarchive path combination #956

Closed cmadjar closed 1 year ago

cmadjar commented 1 year ago
$ run_dicom_archive_validation.py -p database_config_25.py -t /data/loris/data/tarchive/2023/DCM_2023-03-06_DICOM_DIR_2023-03-25_04h20m25s_4y8oen3t.tar -u 147

tarchive path above correspond to uploadID 148 not 147, an error should have been returned and the IsCandidateInfoValidated and IsTarchiveValidated should not have been updated to 1 for that upload ID.

cmadjar commented 1 year ago

Fix sent to 24.1-release and will be released with 24.1.11. Once 24.1.11 is released, code will be updated in main and fix 25.0-release