-
Notifications
You must be signed in to change notification settings - Fork 23
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
DOC: Remove psd_welch usage and replace with Spectrum #45
Conversation
The docs now build on CircleCI but other warning are causing the tick to go ❌ Could we take a baby step here @larsoner @agramfort and merge this fix and then iterate? |
Actually, a larger issue, I see that the pytest workflow has been disabled. @agramfort @larsoner @drammock could someone please enable it here: https://github.com/mne-tools/mne-realtime/actions/workflows/linux.yml thanks |
done
… Message ID: ***@***.***>
|
Thanks for the fix @rob-luke , I snuck in a number of other maintenance stuff :) |
Thanks @larsoner. And thanks for sneaking those other fixes in too, that's much quicker than me stringing PRs over a week 🚀 |
I noticed in #44 that this usage of psd_welch was causing the documentation build to fail, so have updated it to use the newer spectrum object.