Fix dev-deps CI failures: detect pynwb num_samples support via __docval__

[CodyCBakerPhD]

Summary

Fix failing dev-deps CI jobs (ubuntu-latest, Python 3.10 and 3.14) caused by pynwb dev (post-3.1.3) adding num_samples as a required parameter for ImageSeries when format='external' and rate-based timing is used.

Root cause

The test fixture in dandi/tests/fixtures.py creates ImageSeries objects without num_samples, which pynwb dev now requires. A simple version check via inspect.signature() doesn't work because hdmf's @docval decorator wraps __init__ and hides parameters from Python's inspection machinery. Instead, the parameter list is stored in ImageSeries.__init__.__docval__["args"].

Changes

• dandi/tests/fixtures.py: Detect num_samples support by inspecting __docval__["args"] and conditionally pass num_samples=4 when creating ImageSeries fixtures. Includes a TODO to simplify once the minimum supported pynwb is bumped above 3.1.3.

[CodyCBakerPhD]

(certain parts of the daily tests have been failing for a while; trying to resolve)

[codecov]

Codecov Report

❌ Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 76.78%. Comparing base (65c79c9) to head (a4ffe2f).

Files with missing lines	Patch %	Lines
dandi/tests/fixtures.py	80.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1882      +/-   ##
==========================================
- Coverage   76.79%   76.78%   -0.01%     
==========================================
  Files          87       87              
  Lines       12805    12809       +4     
==========================================
+ Hits         9833     9836       +3     
- Misses       2972     2973       +1     

Flag	Coverage Δ
unittests	76.78% <80.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[rly]

This looks good to me. Alternatively, the test fixture could provide timestamps, e.g., timestamps=[0, 1/150, 2/150, 3/150] instead of rate=150.0 and starting_time=0.0.

In PyNWB 4.0.0, a new error is raised when creating a new ImageSeries (not read from file) where external_file is not None, rate is not None, and num_samples is None.

[yarikoptic]

THANKS!

[github-actions]

🚀 PR was released in 0.76.5 🚀