Conversation
Add r-squared metric to FOOOF and FOOOFGroup objects, and FOOOFGroup report plot. R-squared is adjusted for the number of parameters in the model, facilitating model comparison.
|
Hey @mwprestonjr - thanks, this looks great! Is it okay with you if I target this for the 2.0 release, which will have the breaking change of name and so on? Since this updates the set of attributes on our core objects, I think it makes sense to put this update there. If this sounds good to you, I'll review this PR here, then redirect it to the 2.0 branch, and merge it there to be part of that release. |
|
Yea that sounds great, thanks. |
|
Hey @mwprestonjr - I'm keeping this in mind for integrating into 2.0, which I plan to include a somewhat broader framework for specifying and calculating goodness of fit measures. I left a quick comment above to make sure I understand. I'm still working on some re-organizations for 2.0 that will add the new fit measures, and this will need a bit of refactoring then - do you want me to tag you back in when this PR could be refactored into 2.0, or is it easier if I go ahead and adapt this PR into some of the updates directly? |
parameter count has been updated to take into account the aperiodic mode.
|
Hey @TomDonoghue, sorry for the late reply. It's likely easier for you to adapt this PR into some of your updates directly at this point. But let me know what you think. Happy to help |
|
I have now adapted this and integrated having additional metrics, including adjusted R^2, into specparam 2.0, including in #298, which replaces this PR. Thanks for your work on this @mwprestonjr - it was super helpful! |
R-squared metric added to FOOOF and FOOOFGroup objects, and FOOOFGroup report plot.
R-squared is adjusted for the number of parameters in the model, facilitating model comparison (#251).