From d775763eb791fd37949675ade10df7cf176d0a75 Mon Sep 17 00:00:00 2001 From: Heberto Mayorquin Date: Fri, 15 Nov 2024 21:29:42 -0600 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Paul Adkisson --- src/neuroconv/datainterfaces/behavior/audio/audiointerface.py | 3 ++- .../behavior/lightningpose/lightningposedatainterface.py | 1 + .../datainterfaces/behavior/medpc/medpcdatainterface.py | 1 + 3 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/neuroconv/datainterfaces/behavior/audio/audiointerface.py b/src/neuroconv/datainterfaces/behavior/audio/audiointerface.py index 1389dd5f8..6561096b5 100644 --- a/src/neuroconv/datainterfaces/behavior/audio/audiointerface.py +++ b/src/neuroconv/datainterfaces/behavior/audio/audiointerface.py @@ -46,7 +46,8 @@ def __init__(self, file_paths: list[FilePath], verbose: bool = False): verbose : bool, default: False """ - # This import is to assure that the ndx_pose is in the global namespace when an pynwb.io object is created + # This import is to assure that ndx_sound is in the global namespace when an pynwb.io object is created. + # For more detail, see https://github.com/rly/ndx-pose/issues/36 import ndx_sound # noqa: F401 suffixes = [suffix for file_path in file_paths for suffix in Path(file_path).suffixes] diff --git a/src/neuroconv/datainterfaces/behavior/lightningpose/lightningposedatainterface.py b/src/neuroconv/datainterfaces/behavior/lightningpose/lightningposedatainterface.py index 451eb24b4..dbd425b5b 100644 --- a/src/neuroconv/datainterfaces/behavior/lightningpose/lightningposedatainterface.py +++ b/src/neuroconv/datainterfaces/behavior/lightningpose/lightningposedatainterface.py @@ -81,6 +81,7 @@ def __init__( controls verbosity. ``True`` by default. """ # This import is to assure that the ndx_pose is in the global namespace when an pynwb.io object is created + # For more detail, see https://github.com/rly/ndx-pose/issues/36 import ndx_pose # noqa: F401 from neuroconv.datainterfaces.behavior.video.video_utils import ( diff --git a/src/neuroconv/datainterfaces/behavior/medpc/medpcdatainterface.py b/src/neuroconv/datainterfaces/behavior/medpc/medpcdatainterface.py index 6ff723ba3..09f9111d7 100644 --- a/src/neuroconv/datainterfaces/behavior/medpc/medpcdatainterface.py +++ b/src/neuroconv/datainterfaces/behavior/medpc/medpcdatainterface.py @@ -74,6 +74,7 @@ def __init__( Whether to print verbose output, by default True """ # This import is to assure that the ndx_events is in the global namespace when an pynwb.io object is created + # For more detail, see https://github.com/rly/ndx-pose/issues/36 import ndx_events # noqa: F401 if aligned_timestamp_names is None: