Tweak logging for anonymisation (#528)

orthanc/orthanc-anon/plugin/pixl.py

@@ -265,6 +265,8 @@ def _import_study_from_raw(study_resource_id: str, study_uid: str) -> None:
                 zipped_study=zipped_study,
                 study_uid=study_uid,
             )
+        except PixlDiscardError as discard:
+            logger.warning("Failed to anonymize study {}: {}", study_uid, discard)
         except Exception:  # noqa: BLE001
             logger.exception("Failed to anonymize study: {} ", study_uid)
             return
@@ -357,6 +359,7 @@ def _anonymise_study_instances(zipped_study: ZipFile, study_uid: str) -> tuple[l
         message = f"All instances have been skipped for study {study_uid}"
         raise PixlDiscardError(message)
 
+    logger.success("Finished anonymising file: {} for study: {}", file, study_uid)
     return anonymised_instances_bytes, anonymised_study_uid
 
 

pixl_dcmd/src/pixl_dcmd/_database.py

@@ -125,7 +125,7 @@ def get_unexported_image(
     """
     Get an existing, non-exported (for this project) image record from the database
     identified by the study UID. If no result is found, retry with querying on
-    MRN + accession number. If this fails as well, raise a PixlDiscardError.
+    MRN + accession number. If this fails as well, raise a NoResultFound.
     """
     try:
         existing_image: Image = (

pixl_dcmd/src/pixl_dcmd/main.py

@@ -147,7 +147,7 @@ def anonymise_dicom(
         msg = f"Dropping DICOM Modality: {dataset.Modality}"
         raise PixlSkipInstanceError(msg)
 
-    logger.info("Anonymising received instance: {}", study_info)
+    logger.debug("Anonymising instance for: {}", study_info)
 
     # Merge tag schemes
     tag_operations = load_tag_operations(project_config)