Skip to content

Conversation

@duodenum96
Copy link
Contributor

This PR fixes #375.

@TomDonoghue
Copy link
Member

Thanks for finding & fixing this @duodenum96! This fix makes sense to me based on the discussion in #375 - unless @rdgao has any additional comments / thoughts on this I think it can be merged straight in!

@voytek voytek merged commit 24b74ca into fooof-tools:main Nov 20, 2025
10 checks passed
@duodenum96
Copy link
Contributor Author

Thanks!

@voytek
Copy link
Contributor

voytek commented Nov 20, 2025

Thank you for catching this!

@fooof-tools fooof-tools deleted a comment from codecov bot Nov 21, 2025
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.

Potential mistake in the documentation of calculating time constant based on knee frequency

3 participants