Skip to content

Pin pandas < 3.0 to fix E2E test failure#1055

Merged
bendichter merged 5 commits intomainfrom
fix-e2e-main
Feb 13, 2026
Merged

Pin pandas < 3.0 to fix E2E test failure#1055
bendichter merged 5 commits intomainfrom
fix-e2e-main

Conversation

@bendichter
Copy link
Copy Markdown
Collaborator

pandas 3.0 uses Arrow backend by default, returning read-only arrays that break spikeinterface's PhyKilosortExtractor:

ValueError: assignment destination is read-only

at spikeinterface/extractors/phykilosortextractors.py:138

This was causing E2E tests to fail at 'Specify source data information' on main.

pandas 3.0 uses Arrow backend by default, returning read-only arrays
that break spikeinterface's PhyKilosortExtractor with:
ValueError: assignment destination is read-only
Kyle added 3 commits February 12, 2026 18:22
neuroconv 0.6.1 requires tables<3.9.2, but pytables was pinned to
3.10.2 which is too new. pip then tries to build tables from source
and fails because HDF5 is not available.
dandi 0.74.0 renamed the 'dandi-staging' instance to 'dandi-sandbox'.
neuroconv 0.6.1's automatic_dandi_upload uses 'dandi-staging' internally,
causing a KeyError when uploading to the sandbox with dandi >= 0.74.0.
Instead of pinning dandi < 0.74.0, register 'dandi-staging' as an alias
for 'dandi-sandbox' in dandi's known_instances dict. This allows
neuroconv 0.6.1 (which uses 'dandi-staging' internally) to work with
both old and new dandi versions.
Comment thread src/pyflask/manageNeuroconv/manage_neuroconv.py Outdated
Comment thread src/pyflask/manageNeuroconv/manage_neuroconv.py Outdated
Comment thread src/pyflask/manageNeuroconv/manage_neuroconv.py Outdated
Per Ben's direction: focus on bumping neuroconv rather than finding
workarounds for the current main branch's DANDI compat issues.
@bendichter bendichter merged commit c12d968 into main Feb 13, 2026
12 of 21 checks passed
@bendichter bendichter deleted the fix-e2e-main branch February 13, 2026 00:10
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.

2 participants