Skip to content

Conversation

@bparks13
Copy link
Member

This PR is the last PR aimed at fixing #366, specifically focusing on NeuropixelsV2e devices in this branch. Similarly to the other PRs (#549, #522), all probe configuration properties have been consolidated into a single class, and there is additionally a new property in the ProbeConfiguration class that holds a filename for where the ProbeInterface file will be saved/loaded.

The ProbeInterface file is only touched in one of two scenarios: 1) The Bonsai file is saved (the ProbeInterface file is also saved), and 2) The Bonsai file is loaded (the ProbeInterface file is also loaded). At all other times, any operation on changing the electrode configuration is operating on the object in memory; this includes importing/exporting files via the GUI File menu items, or using the GUI to change the electrode configuration.

Fixes #546

@bparks13 bparks13 added this to the 0.7.0 milestone Nov 14, 2025
@bparks13 bparks13 requested review from aacuevas and jonnew November 14, 2025 19:28
@bparks13 bparks13 self-assigned this Nov 14, 2025
Base automatically changed from refactor-npx2-config to main November 14, 2025 19:31
@bparks13 bparks13 marked this pull request as ready for review November 14, 2025 20:01
@bparks13 bparks13 linked an issue Nov 18, 2025 that may be closed by this pull request
@bparks13 bparks13 merged commit e4e546b into main Nov 18, 2025
8 checks passed
@bparks13 bparks13 deleted the issue-546 branch November 18, 2025 18:57
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.

Save ProbeInterface files externally for Neuropixels V2 ProbeConfiguration is deserializing ChannelConfiguration to XML

3 participants