From 67a84e91976c2caa2cf4cad30936db2e16aab085 Mon Sep 17 00:00:00 2001 From: Kai Schlamp Date: Mon, 14 Aug 2023 06:18:54 +0000 Subject: [PATCH] Remove manual query for ModalitiesInStudy for now --- TODO.md | 2 ++ adit/core/utils/dicom_operator.py | 8 ++++---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/TODO.md b/TODO.md index 03826867..449b0769 100644 --- a/TODO.md +++ b/TODO.md @@ -17,6 +17,8 @@ - Split messages into toasts (client only) and messages from server (using Bootstrap alerts) - Rewrite Celery unit tests using the official test helpers - Check why this dicomweb test takes so long +- Use pynetdicom SCPs instead of Orthancs in integrations tests + -- Find those tests by looking for "setup_orthancs" fixture ## High Priority diff --git a/adit/core/utils/dicom_operator.py b/adit/core/utils/dicom_operator.py index 6b495f4d..c92f642f 100644 --- a/adit/core/utils/dicom_operator.py +++ b/adit/core/utils/dicom_operator.py @@ -195,10 +195,10 @@ def _handle_found_studies( # TODO: I guess this won't work as we are in the middle of a C-FIND request (we # are using generators now) and we can't do another C-FIND request here. # But we could use a new Connector instance for this. Fix this later. - if "ModalitiesInStudy" not in result: - result.ModalitiesInStudy = self._fetch_study_modalities( - result.PatientID, result.StudyInstanceUID - ) + # if "ModalitiesInStudy" not in result: + # result.ModalitiesInStudy = self._fetch_study_modalities( + # result.PatientID, result.StudyInstanceUID + # ) modality_query = query.ModalitiesInStudy if modality_query: