nwb-video-widgets 0.1.2__tar.gz → 0.1.4__tar.gz

nwb_video_widgets-0.1.4/CHANGELOG.md

@@ -0,0 +1,22 @@
+# v0.1.4 (Upcoming)
+
+## Removals, Deprecations and Changes
+
+## Bug Fixes
+
+## Features
+
+## Improvements
+
+# v0.1.3 (2025-02-2)
+
+## Removals, Deprecations and Changes
+
+## Bug Fixes
+
+* Fixed DANDI widget path construction on Windows. On Windows, `pathlib.Path` produces backslashes when converting to string, but DANDI asset paths always use forward slashes. Switched to `PurePosixPath` for DANDI path construction. [PR #2](https://github.com/catalystneuro/nwb-video-widgets/pull/2)
+
+## Features
+
+## Improvements
+

{nwb_video_widgets-0.1.2 → nwb_video_widgets-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nwb-video-widgets
-Version: 0.1.2
+Version: 0.1.4
 Summary: Interactive Jupyter widgets for NWB video and pose visualization
 Project-URL: Homepage, https://github.com/catalystneuro/nwb-video-widgets
 Project-URL: Repository, https://github.com/catalystneuro/nwb-video-widgets

{nwb_video_widgets-0.1.2 → nwb_video_widgets-0.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "nwb-video-widgets"
-version = "0.1.2"
+version = "0.1.4"
 description = "Interactive Jupyter widgets for NWB video and pose visualization"
 readme = "README.md"
 license = "MIT"

{nwb_video_widgets-0.1.2 → nwb_video_widgets-0.1.4}/src/nwb_video_widgets/dandi_pose_widget.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 import pathlib
-from pathlib import Path
+from pathlib import PurePosixPath
 from typing import TYPE_CHECKING, Optional
 
 import anywidget
@@ -283,7 +283,8 @@ class NWBDANDIPoseEstimationWidget(anywidget.AnyWidget):
         client = DandiAPIClient()
         dandiset = client.get_dandiset(asset.dandiset_id, asset.version_id)
 
-        nwb_parent = Path(asset.path).parent
+        # Use PurePosixPath because DANDI paths always use forward slashes
+        nwb_parent = PurePosixPath(asset.path).parent
         video_series = discover_video_series(nwbfile)
         video_urls = {}
 

{nwb_video_widgets-0.1.2 → nwb_video_widgets-0.1.4}/src/nwb_video_widgets/dandi_video_widget.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 import pathlib
-from pathlib import Path
+from pathlib import PurePosixPath
 from typing import TYPE_CHECKING, Optional
 
 import anywidget
@@ -195,7 +195,8 @@ class NWBDANDIVideoPlayer(anywidget.AnyWidget):
         client = DandiAPIClient()
         dandiset = client.get_dandiset(asset.dandiset_id, asset.version_id)
 
-        nwb_parent = Path(asset.path).parent
+        # Use PurePosixPath because DANDI paths always use forward slashes
+        nwb_parent = PurePosixPath(asset.path).parent
         video_series = discover_video_series(nwbfile)
         video_urls = {}
 

{nwb_video_widgets-0.1.2 → nwb_video_widgets-0.1.4}/src/nwb_video_widgets/video_widget.py

@@ -108,14 +108,15 @@ class NWBFileVideoPlayer(anywidget.AnyWidget):
         ...     nwbfile_raw, dandi_asset
         ... )
         """
-        from pathlib import Path
+        from pathlib import PurePosixPath
 
         from pynwb.image import ImageSeries
 
         # Derive dandiset from dandi_asset
         dandiset = dandi_asset.client.get_dandiset(dandi_asset.dandiset_id)
 
-        nwb_parent = Path(dandi_asset.path).parent
+        # Use PurePosixPath because DANDI paths always use forward slashes
+        nwb_parent = PurePosixPath(dandi_asset.path).parent
         video_urls = {}
 
         for name, obj in nwbfile_raw.acquisition.items():

nwb_video_widgets-0.1.4/tests/integration/test_dandi_path_construction.py

@@ -0,0 +1,38 @@
+"""Integration test for DANDI path construction on Windows.
+
+Regression test for: https://github.com/CodyCBakerPhD/dandi-open-data-tutorial/pull/5
+
+On Windows, pathlib.Path produces backslashes when converting to string,
+but DANDI asset paths always use forward slashes. This test verifies that
+video asset paths are constructed correctly on all platforms.
+"""
+
+import pytest
+
+from nwb_video_widgets import NWBDANDIPoseEstimationWidget
+
+
+@pytest.mark.integration
+def test_dandi_video_path_construction():
+    """Test that DANDI video paths are constructed with forward slashes on any OS."""
+    from dandi.dandiapi import DandiAPIClient
+
+    client = DandiAPIClient()
+    dandiset = client.get_dandiset("000409", "draft")
+
+    session_eid = "64e3fb86-928c-4079-865c-b364205b502e"
+    session_assets = [asset for asset in dandiset.get_assets() if session_eid in asset.path]
+    raw_asset = next((asset for asset in session_assets if "desc-raw" in asset.path), None)
+    processed_asset = next((asset for asset in session_assets if "desc-processed" in asset.path), None)
+
+    # This would fail on Windows before the fix with:
+    # NotFoundError: No asset at path 'sub-NYU-46\\sub-NYU-46_ses-...\\video.mp4'
+    widget = NWBDANDIPoseEstimationWidget(
+        asset=processed_asset,
+        video_asset=raw_asset,
+    )
+
+    # Verify video URLs were resolved (paths constructed correctly)
+    assert len(widget.video_name_to_url) > 0
+    for url in widget.video_name_to_url.values():
+        assert url.startswith("https://")