Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More explicit consistency checks for Channel aggregation #3151

Conversation

h-mayorquin
Copy link
Collaborator

@h-mayorquin h-mayorquin commented Jul 4, 2024

Following up a @zm711 request in #2797:

#2797 (comment)

The errors now tell the user explictly what went wrong.

@h-mayorquin h-mayorquin self-assigned this Jul 4, 2024
@h-mayorquin h-mayorquin changed the title More explicit concistency checks for Channel aggregation More explicit consistency checks for Channel aggregation Jul 4, 2024
@h-mayorquin h-mayorquin marked this pull request as ready for review July 4, 2024 22:18
@alejoe91 alejoe91 added the core Changes to core module label Jul 5, 2024
@alejoe91 alejoe91 added this to the 0.101.0 milestone Jul 5, 2024
@zm711
Copy link
Collaborator

zm711 commented Jul 5, 2024

Love it! Thanks!

@samuelgarcia samuelgarcia merged commit 90386a6 into SpikeInterface:main Jul 5, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Changes to core module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants