Rewrite dated intro commentary; fill matplotlib and seaborn coverage - #30
Merged
Conversation
Matplotlib covered 2 of the 13 plots and classic seaborn 5. Add cells so matplotlib covers 12 and seaborn all 13, matching each reference plot's titles and labels. Where matplotlib has no primitive (faceting, stacking, dodging, KDE, loess) the cell does the work by hand and says so in a note. Matplotlib skips scatter-with-regression: an OLS fit with a 95% confidence band is a dozen lines of statsmodels and two of matplotlib, so it would demonstrate statsmodels rather than matplotlib.
The library notes still described a 2017 site: plotnine as a young product, plotly as an online publishing service, and an apology for having no interactive examples that have since been added. Update the quotes against each project's current docs (matplotlib, seaborn, plotnine, Vega-Altair, plotly, Lets-Plot, Bokeh), point plotnine at plotnine.org and plotly at plotly.com, drop the bqplot mention since the site has no bqplot column, and say what each library actually covers in the grid below.
Fix two subject-verb errors, drop puffery and needless words in the opener and closing sections. Library commentary untouched.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
TL;DR
The intro's library commentary was written around 2017 — plotnine "a young product," plotly described via its retired online-publishing model, an apology for having "little experience" with interactive libraries that now have full columns. Rewritten for 2026 with every tagline re-verified against current docs. Coverage gaps also filled: matplotlib goes from 2 to 12 of 13 plots, classic seaborn from 5 to 13 of 13, bringing the site to 119 example cells.
Files to review (4):
INTRO.md(start here)Examples.ipynbtests/test_plots.pyReviewer notes
sm.nonparametric.lowessis two lines.Verified: notebook executes clean, 119/119 cells emit PNGs, render + pytest pass.