ENH: Streamline ICA reporting
Before merging …
- [x] Changelog has been updated (
docs/source/changes.md)
-
Moved all ICA reports to the main report:
-
Added docs suggesting manual intervention is required (maybe it should be suggested instead?) when using ICA to take care of the doc part of #881
-
Add just one subject's data for
ERP_CORE(plus the average) to the website to cut size down. -
Add script to remove examples from old versions so we don't have to remove their docs anymore (they go down to like 6MB or something!).
Closes #880
@SophieHerbst can you look to see if the unified report titles and contents seem reasonable to you? Here is an example
https://output.circle-artifacts.com/output/job/3ed5bcd9-e31f-48c1-96c5-b582bd7bccfc/artifacts/0/site/examples/ERP_CORE/sub-015_ses-N170_task-N170_report.html#ICA__epochs_for_fitting
@larsoner the report looks good to me! Do I understand correctly that there is only one single report now?
Yep!
Is this good to go? are you waiting for my review?
Yeah this one is ready to go!
If it's not super urgent, please allow me to review first before merging
I won't have time before later tonight or maybe even tomorrow, though
@hoechenberger just a friendly reminder about this one when you get a chance!
I won't have time to review it before the end of this week. If this is rather urgent, go ahead and merge and we'll do a follow-up PR if need be
I'll fix the conflict here and merge, happy to do a follow-up PR if needed though @hoechenberger !
ok with me, sorry for the delay here
Added a workaround for https://github.com/joblib/joblib/issues/1572