dbdicom 0.3.7__py3-none-any.whl → 0.3.8__py3-none-any.whl

dbdicom/api.py

@@ -198,19 +198,20 @@ def move(from_entity:list, to_entity:list):
     dbd.delete(from_entity)
     dbd.close()
 
-def split_series(series:list, attr:Union[str, tuple])->dict:
+def split_series(series:list, attr:Union[str, tuple], key=None)->list:
     """
     Split a series into multiple series
     
     Args:
         series (list): series to split.
-        attr (str or tuple): dicom attribute to split the series by.  
+        attr (str or tuple): dicom attribute to split the series by. 
+        key (function): split by by key(attr) 
     Returns:
-        dict: dictionary with keys the unique values found (ascending) 
-        and as values the series corresponding to that value.     
+        list: list of two-element tuples, where the first element is
+        is the value and the second element is the series corresponding to that value.      
     """
     dbd = open(series[0])
-    split_series = dbd.split_series(series, attr)
+    split_series = dbd.split_series(series, attr, key)
     dbd.close()
     return split_series
 

dbdicom/dataset.py

@@ -344,10 +344,24 @@ def format_value(value, VR=None, tag=None):
             #return value[:64]
     if VR == 'TM':
         return variables.seconds_to_str(value)
+    if VR == 'DA':
+        if not is_valid_dicom_date(value):
+            return '99991231'
     
     return value
 
 
+
+def is_valid_dicom_date(da_str: str) -> bool:
+    if not isinstance(da_str, str) or len(da_str) != 8 or not da_str.isdigit():
+        return False
+    try:
+        datetime.strptime(da_str, "%Y%m%d")
+        return True
+    except ValueError:
+        return False
+
+
 def check_value(value, tag):
 
     # If the change below is made (TM, DA, DT) then this needs to 

dbdicom/dbd.py

@@ -3,6 +3,7 @@ from datetime import datetime
 import json
 from typing import Union
 import zipfile
+import re
 
 from tqdm import tqdm
 import numpy as np
@@ -651,37 +652,43 @@ class DataBaseDicom():
         self.delete(from_entity)
         return self
     
-    def split_series(self, series:list, attr:Union[str, tuple]) -> dict:
+    def split_series(self, series:list, attr:Union[str, tuple], key=None) -> list:
         """
         Split a series into multiple series
         
         Args:
             series (list): series to split.
             attr (str or tuple): dicom attribute to split the series by. 
+            key (function): split by by key(attr)
         Returns:
-            dict: dictionary with keys the unique values found (ascending) 
-            and as values the series corresponding to that value.         
+            list: list of two-element tuples, where the first element is
+            is the value and the second element is the series corresponding to that value.         
         """
 
         # Find all values of the attr and list files per value
         all_files = register.files(self.register, series)
-        files = {}
+        files = []
+        values = []
         for f in tqdm(all_files, desc=f'Reading {attr}'):
             ds = dbdataset.read_dataset(f)
             v = dbdataset.get_values(ds, attr)
-            if v in files:
-                files[v].append(f)
+            if key is not None:
+                v = key(v)
+            if v in values:
+                index = values.index(v)
+                files[index].append(f)
             else:
-                files[v] = [f]
+                values.append(v)
+                files.append([f])
 
         # Copy the files for each value (sorted) to new series
-        values = sorted(list(files.keys()))
-        split_series = {}
-        for v in tqdm(values, desc='Writing new series'):
+        split_series = []
+        for index, v in tqdm(enumerate(values), desc='Writing new series'):
             series_desc = series[-1] if isinstance(series, str) else series[-1][0]
-            series_v = series[:3] + [f'{series_desc}_{attr}_{v}']
-            self._files_to_series(files[v], series_v)
-            split_series[v] = series_v
+            series_desc = clean_folder_name(f'{series_desc}_{attr}_{v}')
+            series_v = series[:3] + [(series_desc, 0)]
+            self._files_to_series(files[index], series_v)
+            split_series.append((v, series_v))
         return split_series
 
 
@@ -910,6 +917,29 @@ class DataBaseDicom():
 
 
 
+def clean_folder_name(name, replacement="", max_length=255):
+    # Strip leading/trailing whitespace
+    name = name.strip()
+
+    # Replace invalid characters (Windows, macOS, Linux-safe)
+    illegal_chars = r'[<>:"/\\|?*\[\]\x00-\x1F\x7F]'
+    name = re.sub(illegal_chars, replacement, name)
+
+    # Replace reserved Windows names
+    reserved = {
+        "CON", "PRN", "AUX", "NUL",
+        *(f"COM{i}" for i in range(1, 10)),
+        *(f"LPT{i}" for i in range(1, 10))
+    }
+    name_upper = name.upper().split(".")[0]  # Just base name
+    if name_upper in reserved:
+        name = f"{name}_folder"
+
+    # Truncate to max length (common max: 255 bytes)
+    return name[:max_length] or "folder"
+
+
+
 def infer_slice_spacing(vols):
     # In case spacing between slices is not (correctly) encoded in 
     # DICOM it can be inferred from the slice locations.

{dbdicom-0.3.7.dist-info → dbdicom-0.3.8.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dbdicom
-Version: 0.3.7
+Version: 0.3.8
 Summary: A pythonic interface for reading and writing DICOM databases
 Author-email: Steven Sourbron <s.sourbron@sheffield.ac.uk>, Ebony Gunwhy <e.gunwhy@sheffield.ac.uk>
 Project-URL: Homepage, https://openmiblab.github.io/dbdicom/

{dbdicom-0.3.7.dist-info → dbdicom-0.3.8.dist-info}/RECORD

@@ -1,9 +1,9 @@
 dbdicom/__init__.py,sha256=dW5aezonmMc_41Dp1PuYmXQlr307RkyJxsJuetkpWso,87
-dbdicom/api.py,sha256=lUNLQo3aAAIFVF7IMccNhXAlbf1hMNTU9gHibdfbKcI,14135
+dbdicom/api.py,sha256=VA0rXXax30v_ZZ2M-TM_6w4SaV0zqc--5J4w9kUqSyc,14220
 dbdicom/const.py,sha256=BqBiRRjeiSqDr1W6YvaayD8WKCjG4Cny2NT0GeLM6bI,4269
 dbdicom/database.py,sha256=_LUbH7gc9l7j_63AC71DjwxgTUwbEjHSy5kuvRw75Hw,4764
-dbdicom/dataset.py,sha256=kbswoVE3-Wtu0uwcSW319KzpdWx5fZphfKxizxJePzc,22773
-dbdicom/dbd.py,sha256=yvj4PRj4gP1Qmh0VhibOrZ1oxy9v9qM9iOV0L6C_Ryw,38261
+dbdicom/dataset.py,sha256=hDw6aZ3PF-k5wdR2CBBL_ZtvtDBWWrjKvaKKPfs-yYg,23146
+dbdicom/dbd.py,sha256=6F2uC00KzweRCUQnCVBm-Co-QF-KqxfZUFjQxTnUEi0,39291
 dbdicom/register.py,sha256=_NyNbOEAN_hkwjxupNpr9F5DWUwARCsci8knK41-EsA,13931
 dbdicom/external/__init__.py,sha256=XNQqfspyf6vFGedXlRKZsUB8k8E-0W19Uamwn8Aioxo,316
 dbdicom/external/__pycache__/__init__.cpython-311.pyc,sha256=pXAQ35ixd92fm6YcuHgzR1t6RcASQ-cHhU1wOA5b8sw,542
@@ -47,8 +47,8 @@ dbdicom/utils/dcm4che.py,sha256=Vxq8NYWWK3BuqJkzhBQ89oMqzJlnxqTxgsgTo_Frznc,2317
 dbdicom/utils/files.py,sha256=qhWNJqeWnRjDNbERpC6Mz962_TW9mFdvd2lnBbK3xt4,2259
 dbdicom/utils/image.py,sha256=qsU_wOdleZCqU5g7LSp8OXKmoG109NauYx7wad_ZQ7Q,3839
 dbdicom/utils/variables.py,sha256=vUh5cDnmCft5hoXDYXUvfkg5Cy5WlgMAogU38Y_BKRo,5753
-dbdicom-0.3.7.dist-info/licenses/LICENSE,sha256=HrhfyXIkWY2tGFK11kg7vPCqhgh5DcxleloqdhrpyMY,11558
-dbdicom-0.3.7.dist-info/METADATA,sha256=kSsHB7e2v_JdenBx6j9hewbYIOsPucwANgvL_BZEjXw,1030
-dbdicom-0.3.7.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-dbdicom-0.3.7.dist-info/top_level.txt,sha256=nJWxXg4YjD6QblfmhrzTMXcr8FSKNc0Yk-CAIDUsYkQ,8
-dbdicom-0.3.7.dist-info/RECORD,,
+dbdicom-0.3.8.dist-info/licenses/LICENSE,sha256=HrhfyXIkWY2tGFK11kg7vPCqhgh5DcxleloqdhrpyMY,11558
+dbdicom-0.3.8.dist-info/METADATA,sha256=QQLXmSX2KJ5-on3wW3ORtelkKawriWolIzntBzb8Y5k,1030
+dbdicom-0.3.8.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+dbdicom-0.3.8.dist-info/top_level.txt,sha256=nJWxXg4YjD6QblfmhrzTMXcr8FSKNc0Yk-CAIDUsYkQ,8
+dbdicom-0.3.8.dist-info/RECORD,,