From 7ea96b4e476db4f1dced30fbefbfe0ab38ef247d Mon Sep 17 00:00:00 2001
From: Heberto Mayorquin <h.mayorquin@gmail.com>
Date: Tue, 27 Aug 2024 11:47:48 -0600
Subject: [PATCH 1/6] add ruff for classess

---
 pyproject.toml                                | 121 +++++++++---------
 .../ecephys/axona/axonadatainterface.py       |   3 +
 .../cellexplorer/cellexplorerdatainterface.py |   8 ++
 .../neuralynx/neuralynxdatainterface.py       |   5 +
 .../ecephys/spikeglx/spikeglxdatainterface.py |   4 +
 .../ophys/brukertiff/brukertiffconverter.py   |   8 ++
 src/neuroconv/tools/hdmf.py                   |   4 +-
 src/neuroconv/tools/path_expansion.py         |  17 ++-
 .../tools/testing/data_interface_mixins.py    |  30 +++++
 .../tools/testing/mock_interfaces.py          |  12 ++
 src/neuroconv/utils/json_schema.py            |  16 +++
 11 files changed, 165 insertions(+), 63 deletions(-)

diff --git a/pyproject.toml b/pyproject.toml
index 9b60372ab..d88bcb72a 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -8,71 +8,71 @@ version = "0.5.1"
 description = "Convert data from proprietary formats to NWB format."
 readme = "README.md"
 authors = [
-    {name = "Cody Baker"},
-    {name = "Szonja Weigl"},
-    {name = "Heberto Mayorquin"},
-    {name = "Luiz Tauffer"},
-    {name = "Ben Dichter",  email = "ben.dichter@catalystneuro.com"}
+  { name = "Cody Baker" },
+  { name = "Szonja Weigl" },
+  { name = "Heberto Mayorquin" },
+  { name = "Luiz Tauffer" },
+  { name = "Ben Dichter", email = "ben.dichter@catalystneuro.com" },
 ]
 urls = { "Homepage" = "https://github.com/catalystneuro/neuroconv" }
-license = {file = "license.txt"}
+license = { file = "license.txt" }
 keywords = ["nwb"]
 classifiers = [
-    "Intended Audience :: Science/Research",
-    "Programming Language :: Python :: 3.9",
-    "Programming Language :: Python :: 3.10",
-    "Programming Language :: Python :: 3.11",
-    "Programming Language :: Python :: 3.12",
-    "Operating System :: POSIX :: Linux",
-    "Operating System :: Microsoft :: Windows",
-    "Operating System :: MacOS",
-    "License ::  BSD-3-Clause ",
+  "Intended Audience :: Science/Research",
+  "Programming Language :: Python :: 3.9",
+  "Programming Language :: Python :: 3.10",
+  "Programming Language :: Python :: 3.11",
+  "Programming Language :: Python :: 3.12",
+  "Operating System :: POSIX :: Linux",
+  "Operating System :: Microsoft :: Windows",
+  "Operating System :: MacOS",
+  "License ::  BSD-3-Clause ",
 ]
 requires-python = ">=3.9"
 dependencies = [
-    "numpy>=1.22.0, <2.0.0; python_version <= '3.11'",
-    "numpy>=1.26.0, <2.0.0; python_version >= '3.12'",
-    "jsonschema>=3.2.0",
-    "PyYAML>=5.4",
-    "scipy>=1.4.1",
-    "h5py>=3.9.0",
-    "hdmf>=3.13.0",
-    "hdmf_zarr>=0.7.0",
-    "pynwb>=2.7.0",
-    "pydantic>=2.0.0",
-    "typing_extensions>=4.1.0",
-    "psutil>=5.8.0",
-    "tqdm>=4.60.0",
-    "pandas",
-    "parse>=1.20.0",
-    "click",
-    "docstring-parser",
-    "packaging"  # Issue 903
+  "numpy>=1.22.0, <2.0.0; python_version <= '3.11'",
+  "numpy>=1.26.0, <2.0.0; python_version >= '3.12'",
+  "jsonschema>=3.2.0",
+  "PyYAML>=5.4",
+  "scipy>=1.4.1",
+  "h5py>=3.9.0",
+  "hdmf>=3.13.0",
+  "hdmf_zarr>=0.7.0",
+  "pynwb>=2.7.0",
+  "pydantic>=2.0.0",
+  "typing_extensions>=4.1.0",
+  "psutil>=5.8.0",
+  "tqdm>=4.60.0",
+  "pandas",
+  "parse>=1.20.0",
+  "click",
+  "docstring-parser",
+  "packaging",                                       # Issue 903
 ]
 
 
 [project.optional-dependencies]
 test = [
-    "pytest",
-    "pytest-cov",
-    "ndx-events>=0.2.0",  # for special tests to ensure load_namespaces is set to allow NWBFile load at all times
-    "parameterized>=0.8.1",
-    "ndx-miniscope",
-    "spikeinterface[qualitymetrics]>=0.101.0",
-    "zarr<2.18.0",  # Error with Blosc (read-only during decode) in numcodecs on May 7; check later if resolved
-    "pytest-xdist"
+  "pytest",
+  "pytest-cov",
+  "ndx-events>=0.2.0",                       # for special tests to ensure load_namespaces is set to allow NWBFile load at all times
+  "parameterized>=0.8.1",
+  "ndx-miniscope",
+  "spikeinterface[qualitymetrics]>=0.101.0",
+  "zarr<2.18.0",                             # Error with Blosc (read-only during decode) in numcodecs on May 7; check later if resolved
+  "pytest-xdist",
 ]
 
 docs = [
-    "Jinja2<3.1",
-    "Sphinx==5.1.1",
-    "sphinx_rtd_theme==1.0.0",
-    "readthedocs-sphinx-search==0.1.2",
-    "sphinx-toggleprompt==0.2.0",
-    "sphinx-copybutton==0.5.0",
-    "roiextractors",  # Needed for the API documentation
-    "spikeinterface>=0.101.0",  # Needed for the API documentation
-    "pydata_sphinx_theme==0.12.0"
+  "Jinja2<3.1",
+  "Sphinx==5.1.1",
+  "sphinx_rtd_theme==1.0.0",
+  "readthedocs-sphinx-search==0.1.2",
+  "sphinx-toggleprompt==0.2.0",
+  "sphinx-copybutton==0.5.0",
+  "roiextractors",                    # Needed for the API documentation
+  "spikeinterface>=0.101.0",          # Needed for the API documentation
+  "pydata_sphinx_theme==0.12.0",
 ]
 dandi = ["dandi>=0.58.1"]
 compressors = ["hdf5plugin"]
@@ -89,14 +89,10 @@ neuroconv = "neuroconv.tools.yaml_conversion_specification._yaml_conversion_spec
 [tool.pytest.ini_options]
 minversion = "6.0"
 addopts = "-ra --doctest-glob='*.rst'"
-testpaths = [
-  "docs/conversion_examples_gallery/",
-  "tests"
-]
+testpaths = ["docs/conversion_examples_gallery/", "tests"]
 doctest_optionflags = "ELLIPSIS"
 
 
-
 [tool.black]
 line-length = 120
 target-version = ['py38', 'py39', 'py310']
@@ -119,21 +115,24 @@ extend-exclude = '''
 '''
 
 
-
 [tool.ruff]
-exclude = [
-  "*/__init__.py"
-]
 
 [tool.ruff.lint]
-select = ["F401", "I"]  # TODO: eventually, expand to other 'F' linting
+select = ["F401", "I", "D101"] # TODO: eventually, expand to other 'F' linting
+ignore = ["D100"]
 fixable = ["ALL"]
 
+[tool.ruff.lint.per-file-ignores]
+"**__init__.py" = ["F401", "I"]
+"tests/**" = ["D"]  # We are not enforcing docstrings in tests
+"src/neuroconv/tools/testing/data_interface_mixins.py" = ["D"]  # We are not enforcing docstrings in tests
+
 [tool.ruff.lint.isort]
 relative-imports-order = "closest-to-furthest"
 known-first-party = ["neuroconv"]
 
-
+[tool.ruff.lint.pydocstyle]
+convention = "numpy"
 
 [tool.codespell]
 skip = '.git*,*.pdf,*.css'
diff --git a/src/neuroconv/datainterfaces/ecephys/axona/axonadatainterface.py b/src/neuroconv/datainterfaces/ecephys/axona/axonadatainterface.py
index 69612c85c..731aec168 100644
--- a/src/neuroconv/datainterfaces/ecephys/axona/axonadatainterface.py
+++ b/src/neuroconv/datainterfaces/ecephys/axona/axonadatainterface.py
@@ -132,6 +132,9 @@ def __init__(self, file_path: FilePath, noise_std: float = 3.5):
 
 
 class AxonaLFPDataInterface(BaseLFPExtractorInterface):
+    """
+    Primary data interface class for converting Axona LFP data.
+    """
 
     display_name = "Axona LFP"
     associated_suffixes = (".bin", ".set")
diff --git a/src/neuroconv/datainterfaces/ecephys/cellexplorer/cellexplorerdatainterface.py b/src/neuroconv/datainterfaces/ecephys/cellexplorer/cellexplorerdatainterface.py
index 731ce981b..e388e5e44 100644
--- a/src/neuroconv/datainterfaces/ecephys/cellexplorer/cellexplorerdatainterface.py
+++ b/src/neuroconv/datainterfaces/ecephys/cellexplorer/cellexplorerdatainterface.py
@@ -361,6 +361,14 @@ def get_original_timestamps(self):
 
 
 class CellExplorerLFPInterface(CellExplorerRecordingInterface):
+    """
+    Adds lfp data from binary files with the new CellExplorer format:
+
+    https://cellexplorer.org/
+
+    See the `CellExplorerRecordingInterface` class for more information.
+    """
+
     display_name = "CellExplorer LFP"
     keywords = BaseRecordingExtractorInterface.keywords + (
         "extracellular electrophysiology",
diff --git a/src/neuroconv/datainterfaces/ecephys/neuralynx/neuralynxdatainterface.py b/src/neuroconv/datainterfaces/ecephys/neuralynx/neuralynxdatainterface.py
index da0573249..d4ceb6b38 100644
--- a/src/neuroconv/datainterfaces/ecephys/neuralynx/neuralynxdatainterface.py
+++ b/src/neuroconv/datainterfaces/ecephys/neuralynx/neuralynxdatainterface.py
@@ -94,6 +94,11 @@ def get_metadata(self) -> dict:
 
 
 class NeuralynxSortingInterface(BaseSortingExtractorInterface):
+    """
+    Primary data interface for converting Neuralynx sorting data. Uses
+    :py:class:`~spikeinterface.extractors.NeuralynxSortingExtractor`.
+    """
+
     display_name = "Neuralynx Sorting"
     associated_suffixes = (".nse", ".ntt", ".nse", ".nev")
     info = "Interface for Neuralynx sorting data."
diff --git a/src/neuroconv/datainterfaces/ecephys/spikeglx/spikeglxdatainterface.py b/src/neuroconv/datainterfaces/ecephys/spikeglx/spikeglxdatainterface.py
index 6e8c33586..7d97f7d25 100644
--- a/src/neuroconv/datainterfaces/ecephys/spikeglx/spikeglxdatainterface.py
+++ b/src/neuroconv/datainterfaces/ecephys/spikeglx/spikeglxdatainterface.py
@@ -17,6 +17,10 @@
 
 
 class SpikeGLXRecordingInterface(BaseRecordingExtractorInterface):
+    """
+    Primary SpikeGLX interface for converting raw SpikeGLX data using a :py:class:`~spikeinterface.extractors.SpikeGLXRecordingExtractor`.
+    """
+
     display_name = "SpikeGLX Recording"
     keywords = BaseRecordingExtractorInterface.keywords + ("Neuropixels",)
     associated_suffixes = (".imec{probe_index}", ".ap", ".lf", ".meta", ".bin")
diff --git a/src/neuroconv/datainterfaces/ophys/brukertiff/brukertiffconverter.py b/src/neuroconv/datainterfaces/ophys/brukertiff/brukertiffconverter.py
index 6f5daeb0c..86e8edc1f 100644
--- a/src/neuroconv/datainterfaces/ophys/brukertiff/brukertiffconverter.py
+++ b/src/neuroconv/datainterfaces/ophys/brukertiff/brukertiffconverter.py
@@ -13,6 +13,10 @@
 
 
 class BrukerTiffMultiPlaneConverter(NWBConverter):
+    """
+    Converter class for Bruker imaging data with multiple channels and multiple planes.
+    """
+
     display_name = "Bruker TIFF Imaging (multiple channels, multiple planes)"
     keywords = BrukerTiffMultiPlaneImagingInterface.keywords
     associated_suffixes = BrukerTiffMultiPlaneImagingInterface.associated_suffixes
@@ -123,6 +127,10 @@ def run_conversion(
 
 
 class BrukerTiffSinglePlaneConverter(NWBConverter):
+    """
+    Primary data interface class for converting Bruker imaging data with multiple channels and a single plane.
+    """
+
     display_name = "Bruker TIFF Imaging (multiple channels, single plane)"
     keywords = BrukerTiffMultiPlaneImagingInterface.keywords
     associated_suffixes = BrukerTiffMultiPlaneImagingInterface.associated_suffixes
diff --git a/src/neuroconv/tools/hdmf.py b/src/neuroconv/tools/hdmf.py
index e8dfff294..660971df5 100644
--- a/src/neuroconv/tools/hdmf.py
+++ b/src/neuroconv/tools/hdmf.py
@@ -7,7 +7,9 @@
 from hdmf.data_utils import GenericDataChunkIterator as HDMFGenericDataChunkIterator
 
 
-class GenericDataChunkIterator(HDMFGenericDataChunkIterator):
+class GenericDataChunkIterator(HDMFGenericDataChunkIterator):  # noqa: D101
+    # TODO Should this be added to the API?
+
     def _get_default_buffer_shape(self, buffer_gb: float = 1.0) -> tuple[int]:
         return self.estimate_default_buffer_shape(
             buffer_gb=buffer_gb, chunk_shape=self.chunk_shape, maxshape=self.maxshape, dtype=self.dtype
diff --git a/src/neuroconv/tools/path_expansion.py b/src/neuroconv/tools/path_expansion.py
index 427a33a9e..4ab839c0f 100644
--- a/src/neuroconv/tools/path_expansion.py
+++ b/src/neuroconv/tools/path_expansion.py
@@ -13,6 +13,15 @@
 
 
 class AbstractPathExpander(abc.ABC):
+    """
+    Abstract base class for expanding file paths and extracting metadata.
+
+    This class provides methods to extract metadata from file paths within a directory
+    and to expand paths based on a specified data specification. It is designed to be
+    subclassed, with the `list_directory` method needing to be implemented by any
+    subclass to provide the specific logic for listing files in a directory.
+    """
+
     def extract_metadata(self, base_directory: DirectoryPath, format_: str):
         """
         Uses the parse library to extract metadata from file paths in the base_directory.
@@ -128,7 +137,13 @@ def expand_paths(self, source_data_spec: dict[str, dict]) -> list[DeepDict]:
 
 
 class LocalPathExpander(AbstractPathExpander):
-    def list_directory(self, base_directory: DirectoryPath) -> Iterable[FilePath]:
+    """
+    Class for expanding file paths and extracting metadata on a local filesystem.
+
+    See https://neuroconv.readthedocs.io/en/main/user_guide/expand_path.html for more information.
+    """
+
+    def list_directory(self, base_directory: DirectoryPath) -> Iterable[FilePath]:  # noqa: D101
         base_directory = Path(base_directory)
         assert base_directory.is_dir(), f"The specified 'base_directory' ({base_directory}) is not a directory!"
         return (str(path.relative_to(base_directory)) for path in base_directory.rglob("*"))
diff --git a/src/neuroconv/tools/testing/data_interface_mixins.py b/src/neuroconv/tools/testing/data_interface_mixins.py
index 07e25bede..9f94091d4 100644
--- a/src/neuroconv/tools/testing/data_interface_mixins.py
+++ b/src/neuroconv/tools/testing/data_interface_mixins.py
@@ -767,6 +767,10 @@ def test_interface_alignment(self, setup_interface):
 
 
 class AudioInterfaceTestMixin(DataInterfaceTestMixin, TemporalAlignmentMixin):
+    """
+    A mixin for testing Audio interfaces.
+    """
+
     # Currently asserted in the downstream testing suite; could be refactored in future PR
     def check_read_nwb(self, nwbfile_path: str):
         pass
@@ -777,6 +781,10 @@ def test_interface_alignment(self):
 
 
 class DeepLabCutInterfaceMixin(DataInterfaceTestMixin, TemporalAlignmentMixin):
+    """
+    A mixin for testing DeepLabCut interfaces.
+    """
+
     def check_interface_get_original_timestamps(self):
         pass  # TODO in separate PR
 
@@ -797,6 +805,10 @@ def check_nwbfile_temporal_alignment(self):
 
 
 class VideoInterfaceMixin(DataInterfaceTestMixin, TemporalAlignmentMixin):
+    """
+    A mixin for testing Video interfaces.
+    """
+
     def check_read_nwb(self, nwbfile_path: str):
         with NWBHDF5IO(path=nwbfile_path, mode="r", load_namespaces=True) as io:
             nwbfile = io.read()
@@ -867,6 +879,10 @@ def check_interface_original_timestamps_inmutability(self):
 
 
 class MedPCInterfaceMixin(DataInterfaceTestMixin, TemporalAlignmentMixin):
+    """
+    A mixin for testing MedPC interfaces.
+    """
+
     def check_no_metadata_mutation(self, metadata: dict):
         """Ensure the metadata object was not altered by `add_to_nwbfile` method."""
 
@@ -1101,6 +1117,10 @@ def test_interface_alignment(self, medpc_name_to_info_dict: dict):
 
 
 class MiniscopeImagingInterfaceMixin(DataInterfaceTestMixin, TemporalAlignmentMixin):
+    """
+    A mixin for testing Miniscope Imaging interfaces.
+    """
+
     def check_read_nwb(self, nwbfile_path: str):
         from ndx_miniscope import Miniscope
 
@@ -1129,6 +1149,10 @@ def check_read_nwb(self, nwbfile_path: str):
 
 
 class ScanImageSinglePlaneImagingInterfaceMixin(DataInterfaceTestMixin, TemporalAlignmentMixin):
+    """
+    A mixing for testing ScanImage Single Plane Imaging interfaces.
+    """
+
     def check_read_nwb(self, nwbfile_path: str):
         with NWBHDF5IO(nwbfile_path, "r") as io:
             nwbfile = io.read()
@@ -1160,6 +1184,10 @@ def check_read_nwb(self, nwbfile_path: str):
 
 
 class ScanImageMultiPlaneImagingInterfaceMixin(DataInterfaceTestMixin, TemporalAlignmentMixin):
+    """
+    A mixin for testing ScanImage MultiPlane Imaging interfaces.
+    """
+
     def check_read_nwb(self, nwbfile_path: str):
         with NWBHDF5IO(nwbfile_path, "r") as io:
             nwbfile = io.read()
@@ -1190,6 +1218,8 @@ def check_read_nwb(self, nwbfile_path: str):
 
 
 class TDTFiberPhotometryInterfaceMixin(DataInterfaceTestMixin, TemporalAlignmentMixin):
+    """Mixin for testing TDT Fiber Photometry interfaces."""
+
     def check_no_metadata_mutation(self, metadata: dict):
         """Ensure the metadata object was not altered by `add_to_nwbfile` method."""
 
diff --git a/src/neuroconv/tools/testing/mock_interfaces.py b/src/neuroconv/tools/testing/mock_interfaces.py
index 6f91e775f..6e29c127a 100644
--- a/src/neuroconv/tools/testing/mock_interfaces.py
+++ b/src/neuroconv/tools/testing/mock_interfaces.py
@@ -18,6 +18,10 @@
 
 
 class MockBehaviorEventInterface(BaseTemporalAlignmentInterface):
+    """
+    A mock behavior event interface for testing purposes.
+    """
+
     @classmethod
     def get_source_schema(cls) -> dict:
         source_schema = get_schema_from_method_signature(method=cls.__init__, exclude=["event_times"])
@@ -56,6 +60,10 @@ def add_to_nwbfile(self, nwbfile: NWBFile, metadata: dict):
 
 
 class MockSpikeGLXNIDQInterface(SpikeGLXNIDQInterface):
+    """
+    A mock SpikeGLX interface for testing purposes.
+    """
+
     ExtractorName = "NumpyRecording"
 
     @classmethod
@@ -151,6 +159,10 @@ def get_metadata(self) -> dict:
 
 
 class MockImagingInterface(BaseImagingExtractorInterface):
+    """
+    A mock imaging interface for testing purposes.
+    """
+
     def __init__(
         self,
         num_frames: int = 30,
diff --git a/src/neuroconv/utils/json_schema.py b/src/neuroconv/utils/json_schema.py
index 73aa97bdf..cdccfe664 100644
--- a/src/neuroconv/utils/json_schema.py
+++ b/src/neuroconv/utils/json_schema.py
@@ -17,7 +17,17 @@
 
 
 class NWBMetaDataEncoder(json.JSONEncoder):
+    """
+    Custom JSON encoder for NWB metadata.
+
+    This encoder extends the default JSONEncoder class and provides custom serialization
+    for certain data types commonly used in NWB metadata.
+    """
+
     def default(self, obj):
+        """
+        Serialize custom data types to JSON. This overwrites the default method of the JSONEncoder class.
+        """
         # Over-write behaviors for datetime object
         if isinstance(obj, datetime):
             return obj.isoformat()
@@ -34,6 +44,12 @@ def default(self, obj):
 
 
 class NWBSourceDataEncoder(NWBMetaDataEncoder):
+    """
+    Custom JSON encoder for data interface source data (i.e. kwargs).
+
+    This encoder extends the default JSONEncoder class and provides custom serialization
+    for certain data types commonly used in interface source data.
+    """
 
     def default(self, obj):
 

From 79b88185693842a880336bebdc01d6949f9f4bb5 Mon Sep 17 00:00:00 2001
From: Heberto Mayorquin <h.mayorquin@gmail.com>
Date: Tue, 27 Aug 2024 11:57:14 -0600
Subject: [PATCH 2/6] restore pyproject format

---
 pyproject.toml | 107 ++++++++++++++++++++++++++-----------------------
 1 file changed, 56 insertions(+), 51 deletions(-)

diff --git a/pyproject.toml b/pyproject.toml
index d88bcb72a..a295eaeed 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -8,71 +8,71 @@ version = "0.5.1"
 description = "Convert data from proprietary formats to NWB format."
 readme = "README.md"
 authors = [
-  { name = "Cody Baker" },
-  { name = "Szonja Weigl" },
-  { name = "Heberto Mayorquin" },
-  { name = "Luiz Tauffer" },
-  { name = "Ben Dichter", email = "ben.dichter@catalystneuro.com" },
+    {name = "Cody Baker"},
+    {name = "Szonja Weigl"},
+    {name = "Heberto Mayorquin"},
+    {name = "Luiz Tauffer"},
+    {name = "Ben Dichter",  email = "ben.dichter@catalystneuro.com"}
 ]
 urls = { "Homepage" = "https://github.com/catalystneuro/neuroconv" }
-license = { file = "license.txt" }
+license = {file = "license.txt"}
 keywords = ["nwb"]
 classifiers = [
-  "Intended Audience :: Science/Research",
-  "Programming Language :: Python :: 3.9",
-  "Programming Language :: Python :: 3.10",
-  "Programming Language :: Python :: 3.11",
-  "Programming Language :: Python :: 3.12",
-  "Operating System :: POSIX :: Linux",
-  "Operating System :: Microsoft :: Windows",
-  "Operating System :: MacOS",
-  "License ::  BSD-3-Clause ",
+    "Intended Audience :: Science/Research",
+    "Programming Language :: Python :: 3.9",
+    "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Operating System :: POSIX :: Linux",
+    "Operating System :: Microsoft :: Windows",
+    "Operating System :: MacOS",
+    "License ::  BSD-3-Clause ",
 ]
 requires-python = ">=3.9"
 dependencies = [
-  "numpy>=1.22.0, <2.0.0; python_version <= '3.11'",
-  "numpy>=1.26.0, <2.0.0; python_version >= '3.12'",
-  "jsonschema>=3.2.0",
-  "PyYAML>=5.4",
-  "scipy>=1.4.1",
-  "h5py>=3.9.0",
-  "hdmf>=3.13.0",
-  "hdmf_zarr>=0.7.0",
-  "pynwb>=2.7.0",
-  "pydantic>=2.0.0",
-  "typing_extensions>=4.1.0",
-  "psutil>=5.8.0",
-  "tqdm>=4.60.0",
-  "pandas",
-  "parse>=1.20.0",
-  "click",
-  "docstring-parser",
-  "packaging",                                       # Issue 903
+    "numpy>=1.22.0, <2.0.0; python_version <= '3.11'",
+    "numpy>=1.26.0, <2.0.0; python_version >= '3.12'",
+    "jsonschema>=3.2.0",
+    "PyYAML>=5.4",
+    "scipy>=1.4.1",
+    "h5py>=3.9.0",
+    "hdmf>=3.13.0",
+    "hdmf_zarr>=0.7.0",
+    "pynwb>=2.7.0",
+    "pydantic>=2.0.0",
+    "typing_extensions>=4.1.0",
+    "psutil>=5.8.0",
+    "tqdm>=4.60.0",
+    "pandas",
+    "parse>=1.20.0",
+    "click",
+    "docstring-parser",
+    "packaging"  # Issue 903
 ]
 
 
 [project.optional-dependencies]
 test = [
-  "pytest",
-  "pytest-cov",
-  "ndx-events>=0.2.0",                       # for special tests to ensure load_namespaces is set to allow NWBFile load at all times
-  "parameterized>=0.8.1",
-  "ndx-miniscope",
-  "spikeinterface[qualitymetrics]>=0.101.0",
-  "zarr<2.18.0",                             # Error with Blosc (read-only during decode) in numcodecs on May 7; check later if resolved
-  "pytest-xdist",
+    "pytest",
+    "pytest-cov",
+    "ndx-events>=0.2.0",  # for special tests to ensure load_namespaces is set to allow NWBFile load at all times
+    "parameterized>=0.8.1",
+    "ndx-miniscope",
+    "spikeinterface[qualitymetrics]>=0.101.0",
+    "zarr<2.18.0",  # Error with Blosc (read-only during decode) in numcodecs on May 7; check later if resolved
+    "pytest-xdist"
 ]
 
 docs = [
-  "Jinja2<3.1",
-  "Sphinx==5.1.1",
-  "sphinx_rtd_theme==1.0.0",
-  "readthedocs-sphinx-search==0.1.2",
-  "sphinx-toggleprompt==0.2.0",
-  "sphinx-copybutton==0.5.0",
-  "roiextractors",                    # Needed for the API documentation
-  "spikeinterface>=0.101.0",          # Needed for the API documentation
-  "pydata_sphinx_theme==0.12.0",
+    "Jinja2<3.1",
+    "Sphinx==5.1.1",
+    "sphinx_rtd_theme==1.0.0",
+    "readthedocs-sphinx-search==0.1.2",
+    "sphinx-toggleprompt==0.2.0",
+    "sphinx-copybutton==0.5.0",
+    "roiextractors",  # Needed for the API documentation
+    "spikeinterface>=0.101.0",  # Needed for the API documentation
+    "pydata_sphinx_theme==0.12.0"
 ]
 dandi = ["dandi>=0.58.1"]
 compressors = ["hdf5plugin"]
@@ -89,10 +89,14 @@ neuroconv = "neuroconv.tools.yaml_conversion_specification._yaml_conversion_spec
 [tool.pytest.ini_options]
 minversion = "6.0"
 addopts = "-ra --doctest-glob='*.rst'"
-testpaths = ["docs/conversion_examples_gallery/", "tests"]
+testpaths = [
+  "docs/conversion_examples_gallery/",
+  "tests"
+]
 doctest_optionflags = "ELLIPSIS"
 
 
+
 [tool.black]
 line-length = 120
 target-version = ['py38', 'py39', 'py310']
@@ -115,6 +119,7 @@ extend-exclude = '''
 '''
 
 
+
 [tool.ruff]
 
 [tool.ruff.lint]

From 3687e1c61c14a717e9b6d7a5b0553a3b686dd5ac Mon Sep 17 00:00:00 2001
From: Heberto Mayorquin <h.mayorquin@gmail.com>
Date: Tue, 27 Aug 2024 11:58:03 -0600
Subject: [PATCH 3/6] changelog

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 50287da81..8006e71c5 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -36,6 +36,7 @@
 * Replaced all interface annotations with Pydantic types. [PR #1017](https://github.com/catalystneuro/neuroconv/pull/1017)
 * Changed typehint collections (e.g. `List`) to standard collections (e.g. `list`). [PR #1021](https://github.com/catalystneuro/neuroconv/pull/1021)
 * Testing now is only one dataset per test [PR #1026](https://github.com/catalystneuro/neuroconv/pull/1026)
+* Using ruff to enforce existence of public classes' docstrings [PR #1034](https://github.com/catalystneuro/neuroconv/pull/1034)
 
 
 

From da09524f516c98f69b3a891f7143bcf4bcdeccf1 Mon Sep 17 00:00:00 2001
From: Heberto Mayorquin <h.mayorquin@gmail.com>
Date: Wed, 28 Aug 2024 13:27:58 -0600
Subject: [PATCH 4/6] remove convention

---
 pyproject.toml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/pyproject.toml b/pyproject.toml
index 518c7cff7..b2e462f7a 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -138,8 +138,6 @@ fixable = ["ALL"]
 relative-imports-order = "closest-to-furthest"
 known-first-party = ["neuroconv"]
 
-[tool.ruff.lint.pydocstyle]
-convention = "numpy"
 
 [tool.codespell]
 skip = '.git*,*.pdf,*.css'

From d00eefdfcb656ad05ef075b55dece125cbd64d20 Mon Sep 17 00:00:00 2001
From: Heberto Mayorquin <h.mayorquin@gmail.com>
Date: Thu, 29 Aug 2024 08:15:25 -0600
Subject: [PATCH 5/6] remove unecessary ignore

---
 pyproject.toml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/pyproject.toml b/pyproject.toml
index b2e462f7a..dd8761178 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -126,13 +126,12 @@ extend-exclude = '''
 
 [tool.ruff.lint]
 select = ["F401", "I", "D101"] # TODO: eventually, expand to other 'F' linting
-ignore = ["D100"]
 fixable = ["ALL"]
 
 [tool.ruff.lint.per-file-ignores]
 "**__init__.py" = ["F401", "I"]
 "tests/**" = ["D"]  # We are not enforcing docstrings in tests
-"src/neuroconv/tools/testing/data_interface_mixins.py" = ["D"]  # We are not enforcing docstrings in tests
+"src/neuroconv/tools/testing/data_interface_mixins.py" = ["D"]  # We are not enforcing docstrings in the interface mixings
 
 [tool.ruff.lint.isort]
 relative-imports-order = "closest-to-furthest"

From 3a6ddb8ad03a73c56d7a33b8a6cc02206a4e0459 Mon Sep 17 00:00:00 2001
From: Heberto Mayorquin <h.mayorquin@gmail.com>
Date: Fri, 30 Aug 2024 09:16:54 -0600
Subject: [PATCH 6/6] changelog

---
 CHANGELOG.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 11f0a742d..16a8ef737 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,6 +7,7 @@
 ### Bug fixes
 
 ### Improvements
+* Using ruff to enforce existence of public classes' docstrings [PR #1034](https://github.com/catalystneuro/neuroconv/pull/1034)
 
 ## v0.6.1 (August 30, 2024)
 
@@ -51,7 +52,7 @@
 * Replaced all interface annotations with Pydantic types. [PR #1017](https://github.com/catalystneuro/neuroconv/pull/1017)
 * Changed typehint collections (e.g. `List`) to standard collections (e.g. `list`). [PR #1021](https://github.com/catalystneuro/neuroconv/pull/1021)
 * Testing now is only one dataset per test [PR #1026](https://github.com/catalystneuro/neuroconv/pull/1026)
-* Using ruff to enforce existence of public classes' docstrings [PR #1034](https://github.com/catalystneuro/neuroconv/pull/1034)
+