Skip to content

Regularised the API from parent_recording to recording in zero_channel_pad#2923

Merged
alejoe91 merged 3 commits intoSpikeInterface:mainfrom
AnujVader:fix_padded
Jun 1, 2024
Merged

Regularised the API from parent_recording to recording in zero_channel_pad#2923
alejoe91 merged 3 commits intoSpikeInterface:mainfrom
AnujVader:fix_padded

Conversation

@AnujVader
Copy link
Copy Markdown

SpikeInterface/SpikeInterface-Hackathon-Edinburgh-May24#6

Changed variable name from parent_recording to recording

@alejoe91 alejoe91 added refactor Refactor of code, with no change to functionality packaging Related to packaging/style labels May 30, 2024
@zm711
Copy link
Copy Markdown
Member

zm711 commented May 31, 2024

Sam and I discussed about making a breaking change by also switching the self._kwargs so we need to think about that. I'll need to look through this and see if you've made that change.

@alejoe91 alejoe91 added the hackathon-24 Contributions during the SpikeInterface Hackathon May 24 label Jun 1, 2024
@alejoe91
Copy link
Copy Markdown
Member

alejoe91 commented Jun 1, 2024

Thanks @AnujVader !

@alejoe91 alejoe91 merged commit cc52eb4 into SpikeInterface:main Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hackathon-24 Contributions during the SpikeInterface Hackathon May 24 packaging Related to packaging/style refactor Refactor of code, with no change to functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants