fli-iam / shanoir-ng

Shanoir (SHAring iN vivO Imaging Resources)
GNU General Public License v3.0
21 stars 21 forks source link

check patient exist during import #859

Open julien-louis opened 3 years ago

julien-louis commented 3 years ago

In the clinical context page, it would be very useful if there was a warning about a patient mismatch. Aslo, if a subject matches the dicom data it should be pre-selected.

* if hash exists in db
* else if patient name exists in db
---> then pre-select this patient

* if birth date/year don't match
* else if hash exists in db
* else if patient name exists in db
---> then display a warning msg
michaelkain commented 3 years ago

Hi Julien, I think this is a good point and please let's discuss this in the daily. ShUp and the frontend should do the same. Thank you and cheers, Michael

ebannier2018 commented 3 years ago

Great. 4 comments that come to mind :

  1. Shanoir only keeps birth month and year, not day, so only these two can be compared when importing a new exam.
  2. Any idea how many users import already pre-anonymized data (patient name is the subject code for the study), ie. how many have a hash based on the real name (import non pre-anonymized data)?
  3. As for checking if patient name exist, do you mean subject code in the study ? Do you check in the whole db (as written below) on specifically in the study (preferable)?
  4. Refering to an old discussion, somes studies use just number (ie 01001 is common for center - subject id in center and cannot be used twice prospectively but what about already imported data ? We add the study name in front of the subject code to avoid this situation but I do not know how other Shanoir users deal with this. Let me know what comes out of your meeting.
michaelkain commented 3 years ago

I am taking over this issue from Julien.

michaelkain commented 3 years ago

https://github.com/fli-iam/shanoir-ng/wiki/DICOM-Import-Process-Steps#import-step-a6-calculate-subjectidentifier