Skip to content

Dev - #508

Merged
rcpeene merged 14 commits into
mainfrom
dev
Oct 19, 2026
Merged

Dev#508
rcpeene merged 14 commits into
mainfrom
dev

Conversation

@rcpeene

@rcpeene rcpeene commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

camila-maura and others added 13 commits August 10, 2026 16:18
pynapple 0.11.0 unified the peri-event helpers: compute_perievent was
resignatured to compute_perievent(data, events, window), and
compute_perievent_continuous was removed entirely, with the same
function now handling continuous data.

The notebook still used the pre-0.11 signatures (timestamps=/tref=/
minmax=, plus compute_perievent_continuous), so it fails against
current pynapple.

- Update the five affected code cells to the new API
- Update four markdown cells that still named and linked the removed
  compute_perievent_continuous
- Require pynapple>=0.11 in pyproject.toml, and refresh uv.lock
  (0.10.3 -> 0.11.3) and requirements-ci.txt to match

The lockfile previously pinned 0.10.3, which still has the old API, so
the notebook and the dependency bump have to land together. This
notebook is the only pynapple user in the repo, and nemos only requires
pynapple>=0.6.6, so the bump is contained.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
fix(GLM_pynapple_nemos): update to pynapple 0.11 perievent API
Remove DANDI Hub references following service sunset
Add `.DS_Store` to `.gitignore`
# Conflicts:
#	docs/higher-order/GLM_pynapple_nemos.ipynb
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@rcpeene
rcpeene merged commit 9e594b9 into main Oct 19, 2026
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants