Merge pull request #98 from blairconrad/anonymize-station-no-modality

Support anonymization of StationName when dataset has no Modality

Author: blairconrad

src/dicognito/anonymizer.py

@@ -96,6 +96,7 @@ def __init__(self, id_prefix="", id_suffix="", seed=None):
                 "PlacerOrderNumberProcedure",
                 "RequestedProcedureID",
                 "ScheduledProcedureStepID",
+                "StationName",
                 "StudyID",
             ),
             address_anonymizer,

src/dicognito/equipmentanonymizer.py

@@ -17,7 +17,6 @@ def __init__(self, address_anonymizer):
             pydicom.datadict.tag_for_keyword("InstitutionName"): self.anonymize_institution_name,
             pydicom.datadict.tag_for_keyword("InstitutionAddress"): self.anonymize_institution_address,
             pydicom.datadict.tag_for_keyword("InstitutionalDepartmentName"): self.anonymize_department_name,
-            pydicom.datadict.tag_for_keyword("StationName"): self.anonymize_station_name,
         }
 
     def __call__(self, dataset, data_element):
@@ -33,8 +32,8 @@ def __call__(self, dataset, data_element):
         data_element : pydicom.dataset.DataElement
             The current element. Will be anonymized if it has a value
             and if its keyword is one of InstitutionName,
-            InstitutionAddress, InstitutionalDepartmentName, or
-            StationName. Additionally, if its keyword is InstitutionName,
+            InstitutionAddress, or InstitutionalDepartmentName.
+            Additionally, if its keyword is InstitutionName,
             then InstitutionAddress will also be anonymized.
 
         Returns
@@ -62,6 +61,3 @@ def anonymize_institution_address(self, dataset, data_element):
 
     def anonymize_department_name(self, dataset, data_element):
         data_element.value = "RADIOLOGY"
-
-    def anonymize_station_name(self, dataset, data_element):
-        data_element.value = dataset.Modality + "01"

src/dicognito/release_notes.md

@@ -9,8 +9,8 @@
 ### Fixed
 
 - Deflated files are corrupt when anonymized from the command line ([#80](https://github.com/blairconrad/dicognito/issues/80))
-
 - Date/time offset is not always the same for a given seed ([#86](https://github.com/blairconrad/dicognito/issues/86))
+- Anonymizing dataset with StationName but no Modality fails ([#97](https://github.com/blairconrad/dicognito/issues/97))
 
 
 ## 0.10.0

tests/test_anonymize_single_instance.py

@@ -313,6 +313,19 @@ def test_equipment_gets_anonymized():
         assert new_station_name != original_station_name
 
 
+def test_station_gets_anonymized_when_no_modality():
+    with load_test_instance() as dataset:
+        original_station_name = dataset.StationName
+        del dataset.Modality
+
+        anonymizer = Anonymizer()
+        anonymizer.anonymize(dataset)
+
+        new_station_name = dataset.StationName
+
+        assert new_station_name != original_station_name
+
+
 def test_requesting_service_gets_anonymized():
     with load_test_instance() as dataset:
         original = dataset.RequestingService