openradx / adit

ADIT (Automated DICOM Transfer) is a swiss army knife to exchange DICOM data between various systems by using a convenient web frontend.
GNU Affero General Public License v3.0
7 stars 3 forks source link

Series level transfer pseudonymization #15

Closed mdebic closed 3 years ago

mdebic commented 3 years ago

Current pseudonymization runs on transfer task level. If a transfer task level is executed on only one series, different series from the same study are pseudonymized independently from each other, with a different anonymizer object. Series from the same study transferred on series level lose their connection to each other. At the destination they will be treated as if they come from different studies.

Move anonymizer object to job level?

medihack commented 3 years ago

Sounds absolutely reasonable. Give it a shot 😀

mdebic commented 3 years ago

Du sollst das in deinem Urlaub nicht kommentieren!

mdebic commented 3 years ago

@medihack geht nicht, zumindest so wie ich es probiert habe. Ich ruf dich die nächste Woche mal wieder dazu an

medihack commented 3 years ago

Man könnte weiter über die Studie reingehen. Nur dass die Funktion noch eine optionale Liste von Serien bekommt. Dann könnte man weiter über die Studie pseudonymisieren und alles ist gut.

mdebic commented 3 years ago

Mal schauen. Einfachste ohne große Code Änderung wäre das Anomyizer Object einfach in die batch_transfer/tasks.py oben reinzuschreiben und des objelt in doe handle_dicom_task methode weiterzugeben in das excute_transfer. Aber damit hast halt immer das selbe Anomyizer object. weiß nicht ob das von nachteil ist