[Example] update Anscombe's Quartet #18814
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Summary
Hi!
This PR includes a tiny change to the Anscombe's Quartet example:
min(x)
tomax(x)
per plot.Reason for this update:
When I looked at the IV quadrant, I thought that its linear regression line was different from the rest in terms of its intercept, slope seemed the same, but even though I squinted hard, I still doubted my prior assumption that all 4 datasets should result in the same summary statistics (and therefore also regression slopes & intercepts). I visited Wikipedia and confirmed that indeed the 4 lines should be (almost) exactly the same.
Maybe it's just me but I couldn't easily see the similarity of the regression lines between the IV and the rest (and even the other 3 among themselves feel a bit iffy). I think making the regression lines go through the entire plots would drive home the idea of Anscombe's Quartet much more decisively.
Hope this helps and doesn't take too much of your time reviewing.
Thanks!
PR Checklist
(I took the liberty to remove the checklist items that are not so relevant for this tiny update to an example)
pytest
passes).flake8
on changed files to check).flake8-docstrings
and runflake8 --docstring-convention=all
).