Restrict Plexon2RecordingInterface to Windows#1001
Conversation
|
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. |
|
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. |
|
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. |
for more information, see https://pre-commit.ci
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The Plexon2 SDK only supports Windows, so exclude the interface on other platforms in both the backend API and frontend UI. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
for more information, see https://pre-commit.ci
rly
changed the title
Resolve conflict: remove autogenerated SourceData.stories.js (no longer stored in repo)
This directory is already in .gitignore — autogenerated schemas should not be stored in the repo. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Instead of excluding Plexon2RecordingInterface from the backend on non-Windows, let it appear in the search list as disabled with a "Windows only" label. This informs users about the platform requirement rather than silently hiding the option. - Remove backend sys.platform check in get_all_interface_info() - Export windowsOnlyInterfaces from globals - Pass disabledReason to Search options for per-item disabled labels - Support per-item --disabled-label override in Search component Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
for more information, see https://pre-commit.ci
Working for me |
2 participants
Fix #915
TODO: