linters

Author: Sun-flow

src/aind_data_schema/data_description.py

@@ -36,7 +36,7 @@ class DataRegex(Enum):
         f"{RegexParts.DATE.value})_(?P<c_time>{RegexParts.TIME.value})$"
     )
     NO_UNDERSCORES = "^[^_]+$"
-    NO_SPECIAL_CHARS = '^[^<>:;"/|? _\\]+$'
+    NO_SPECIAL_CHARS = "^[^<>:;\"/|? \\_]+$"
 
 
 class DataLevel(Enum):

src/aind_data_schema/imaging/mri_session.py

@@ -90,7 +90,7 @@ def validate_other(cls, v):
         if v.get("scan_sequence_type") == MriScanSequence.OTHER and not v.get("notes"):
             raise ValueError(
                 "Notes cannot be empty if scan_sequence_type is Other.",
-                "Describe the scan_sequence_type in the notes field."
+                "Describe the scan_sequence_type in the notes field.",
             )
         return v
 

tests/test_data_description.py

@@ -8,14 +8,14 @@
 from typing import List
 
 from aind_data_schema.data_description import (
+    AnalysisDescription,
     DataDescription,
     DerivedDataDescription,
     Funding,
     Institution,
     Modality,
     Platform,
     RawDataDescription,
-    AnalysisDescription
 )
 from aind_data_schema.schema_upgrade.data_description_upgrade import DataDescriptionUpgrade
 
@@ -123,8 +123,8 @@ def test_constructors(self):
             )
 
         AnalysisDescription(
-            analysis_name='analysis',
-            project_name='project',
+            analysis_name="analysis",
+            project_name="project",
             creation_time=dt,
             subject_id="1234",
             modality=[Modality.SPIM],
@@ -136,8 +136,8 @@ def test_constructors(self):
 
         with self.assertRaises(ValueError):
             AnalysisDescription(
-                analysis_name='ana lysis',
-                project_name='pro_ject',
+                analysis_name="ana lysis",
+                project_name="pro_ject",
                 subject_id="1234",
                 modality=[Modality.SPIM],
                 platform="exaspim",
@@ -149,8 +149,8 @@ def test_constructors(self):
 
         with self.assertRaises(ValueError):
             AnalysisDescription(
-                analysis_name='',
-                project_name='project',
+                analysis_name="",
+                project_name="project",
                 subject_id="1234",
                 modality=[Modality.SPIM],
                 platform="exaspim",
@@ -162,8 +162,8 @@ def test_constructors(self):
 
         with self.assertRaises(ValueError):
             AnalysisDescription(
-                analysis_name='analysis',
-                project_name='',
+                analysis_name="analysis",
+                project_name="",
                 subject_id="1234",
                 modality=[Modality.SPIM],
                 platform="exaspim",

tests/test_processing.py

@@ -22,10 +22,7 @@ def test_constructors(self):
         )
 
         with self.assertRaises(pydantic.ValidationError):
-            DataProcess(
-                name="Other",
-                notes=""
-            )
+            DataProcess(name="Other", notes="")
 
         assert p is not None