Skip to content

Conversation

@rly
Copy link
Collaborator

@rly rly commented Jul 2, 2025

Fix #915

TODO:

  • Restrict usage to Windows distributions

@rly
Copy link
Collaborator Author

rly commented Jul 2, 2025

I'm getting an error from neuroconv when getting the extractor -> spikeinterface/extractors/neoextractors/plexon2.py -> spikeinterface/extractors/neoextractors/neobaseextractor.py line 218 that "stream_id 3 is not in []". Investigating.

@rly
Copy link
Collaborator Author

rly commented Jul 2, 2025

This seems to have been resolved in neuroconv 0.6.2 catalystneuro/neuroconv#1058.

This could also be resolved by limiting neo version to <=0.13.3 but then spikeinterface needs to be downgraded which lowers the zarr version, which lowers the hdmf-zarr version, and so on. Might be better to bump to just support neuroconv 0.6.2 with the fix.

@rly
Copy link
Collaborator Author

rly commented Jul 3, 2025

Installing neuroconv 0.6.2 from pypi does not install its extras (see catalystneuro/neuroconv#1116). This is fixed in neuroconv 0.6.5. So I think it would be best to punt PL2 support to when neuroconv 0.6.5 support is added.

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.

[Feature]: Support reading pl2

2 participants