Skip to content

Fix the IBL Cbin extractor so that it works with 4 split shanks as well#4567

Open
rai-pranav wants to merge 1 commit intoSpikeInterface:mainfrom
int-brain-lab:fix/cbin-ibl-4-split-shanks
Open

Fix the IBL Cbin extractor so that it works with 4 split shanks as well#4567
rai-pranav wants to merge 1 commit intoSpikeInterface:mainfrom
int-brain-lab:fix/cbin-ibl-4-split-shanks

Conversation

@rai-pranav
Copy link
Copy Markdown

The .meta file that is produced when converting the Np2 probes to 4 individual shanks does not contain the ~ sign in front of the fileds which are lists, and hence the read_meta_file function from neo.rawio.spikeglxrawio fails. So I have written some functions which can handle this version of the meta file.

I am taking the latest way of getting the sample shifts during the merge conflit.
@rai-pranav
Copy link
Copy Markdown
Author

Hi @alejoe91
I have created this PR to fix some IBL specific loading of the Np2 4 shanks dataset.
Some of the windows OS tests are failing which I don't think are related to the my changes. Any hints on how to resolve them?

@chrishalcrow
Copy link
Copy Markdown
Member

Hello, I think the test failure was related to a downstream cache test failure here: https://github.com/SpikeInterface/spikeinterface/actions/runs/24997078165
I reran the caching successfully here: https://github.com/SpikeInterface/spikeinterface/actions/runs/25054593823
Which seems to have fixed the git-annex install here.
I'm not sure how to make this more robust, but at least the tests are passing now!

@rai-pranav
Copy link
Copy Markdown
Author

I cannot directly assign reviewers to this PR.
Can someone please review this PR ? @chrishalcrow @alejoe91 🙏

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