Bug Fixes and Unit Tests for BLANK & REPLACE Actions & Sequences

.github/workflows/main.yaml

@@ -40,5 +40,5 @@ jobs:
         run: |
           export PATH="/usr/share/miniconda/bin:$PATH"
           source activate testing
-          pip install deid-data==0.0.17
+          pip install deid-data==0.0.19
           python -m unittest discover -s deid/tests/ -p '[t|T]est*.py'

CHANGELOG.md

@@ -14,7 +14,7 @@ and **Merged pull requests**. Critical items to know are:
 Referenced versions in headers are tagged on Github, in parentheses are for pypi.
 
 ## [vxx](https://github.com/pydicom/deid/tree/master) (master)
-
+- Expand BLANK Action to additional VRs [#241](https://github.com/pydicom/deid/issues/241) (0.3.2)
 - pre-commit for linting and formatting (0.3.1)
 - Add `ctpcoordinates` and `ctpkeepcoordinates` to handle different formats (0.3.0)
   - Minimum Python required is 3.7, numpy 1.20

deid/dicom/parser.py

@@ -186,16 +186,19 @@ def blank_field(self, field):
         """
         Blank a field
         """
-        element = self.get_nested_field(field)
+        tag = self.get_nested_field(field)
+        if tag in self.dicom:
+            element = self.dicom[tag]
+        else:
+            element = None
+            bot.warning(f"Field {field} not found in DICOM header.")
 
         # Assert we have a data element, and can blank a string
         if element:
             if not isinstance(element, DataElement):
-                bot.warning("Issue parsing %s as a DataElement, not blanked." % field)
-            elif element.VR in ["US", "SS"]:
-                element.value = ""
+                bot.warning(f"Issue parsing {field} as a DataElement, not blanked.")
             else:
-                bot.warning("Unrecognized VR for %s, skipping blank." % field)
+                element.value = None
 
     def replace_field(self, field, value):
         """