Skip to content

Docs: r2 is NSE, not cc²; wind_rose example labels - #735

Open
ecomodeller wants to merge 2 commits into
mainfrom
r2-docstring
Open

ecomodeller wants to merge 2 commits into
mainfrom
r2-docstring

Conversation

@ecomodeller

Copy link
Copy Markdown
Member
  • The r2 docstring called it the proportion of explained variance, which is ev. It now says r2 equals NSE and scikit-learn's r2_score, is not cc**2, and goes negative when the model is worse than the observed mean.
  • The wind_rose example in the plotting guide passed the model columns first, so the legend labelled the model "Measurement".

🤖 Generated with Claude Code

ecomodeller and others added 2 commits October 22, 2026 08:06
The docstring called r2 the proportion of explained variance, which is ev.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The example passed the model first, so the legend labelled it Measurement.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant