Skip to content

Conversation

@Moritz-Alexander-Kern
Copy link
Member

Enhanced description for the spike_interval parameter to clarify the bahavior and effects on the resulting spike train aswell as the choice of the chosen default values.

Added notes section to clarify the behavior when time_stamps is set to None and what the expected results are.

This PR adresses #415 .

@Moritz-Alexander-Kern Moritz-Alexander-Kern added the documentation Indicates a need for improvements or additions to documentation label Sep 23, 2024
@Moritz-Alexander-Kern Moritz-Alexander-Kern linked an issue Sep 23, 2024 that may be closed by this pull request
@coveralls
Copy link
Collaborator

coveralls commented Sep 23, 2024

Coverage Status

coverage: 87.92% (-0.3%) from 88.262%
when pulling 2ded06b on INM-6:enh/spike_extraction_documentation
into 3d54e99 on NeuralEnsemble:master.

Copy link
Collaborator

@kohlerca kohlerca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made some suggestions for clarity in the text.

Copy link
Member Author

@Moritz-Alexander-Kern Moritz-Alexander-Kern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

build wheels

@Moritz-Alexander-Kern Moritz-Alexander-Kern merged commit bb0a394 into NeuralEnsemble:master Jan 14, 2025
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Indicates a need for improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Doc] Spike Extraction parameters

3 participants