Skip to content

Better overviews #650

Merged
brian-rose merged 27 commits into
ProjectPythia:mainfrom
brian-rose:overviews
Jun 16, 2026
Merged

Better overviews #650
brian-rose merged 27 commits into
ProjectPythia:mainfrom
brian-rose:overviews

Conversation

@brian-rose

@brian-rose brian-rose commented Jun 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Closes #634

This PR improves the "Overview" section of many chapters, moving away from just numbered lists of topics and giving a more narrative description of the chapter content.

This may be the last edit needed for our JOSE revisions.

@review-notebook-app

review-notebook-app Bot commented Jun 12, 2026

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@brian-rose brian-rose added content Content related issue JOSE revision labels Jun 12, 2026
@github-actions

github-actions Bot commented Jun 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: ca566f8
✅ Deployment Preview URL: https://ProjectPythia.github.io/pythia-foundations/_preview/650

@brian-rose brian-rose marked this pull request as ready for review June 14, 2026 22:57
@brian-rose brian-rose requested a review from a team as a code owner June 14, 2026 22:57
@brian-rose brian-rose requested review from dopplershift, kafitzgerald and r-ford and removed request for a team June 14, 2026 22:57
kafitzgerald
kafitzgerald reviewed Jun 15, 2026
View reviewed changes

@kafitzgerald kafitzgerald left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few minor suggestions (take them or leave them), but this looks great overall. A nice improvement!

Comment thread core/matplotlib/histograms-piecharts-animation.ipynb Outdated
Comment thread core/numpy/intermediate-numpy.ipynb Outdated
Comment thread core/pandas/pandas.ipynb Outdated
Comment thread core/matplotlib/annotations-colorbars-layouts.ipynb Outdated
Comment thread core/matplotlib/matplotlib-basics.ipynb Outdated
Comment thread foundations/github/basic-git.md Outdated
Comment thread foundations/github/git-workflows.md Outdated
Comment thread foundations/github/github-pull-request.md Outdated
Comment thread foundations/github/github-repos.md Outdated
Comment thread foundations/github/what-is-github.md Outdated
brian-rose and others added 6 commits June 14, 2026 21:13
@brian-rose @kafitzgerald
Update core/matplotlib/histograms-piecharts-animation.ipynb
b2b72d2 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update core/numpy/intermediate-numpy.ipynb
ca258c8 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update core/pandas/pandas.ipynb
2b187a6 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update core/matplotlib/annotations-colorbars-layouts.ipynb
7c0cd10 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update core/matplotlib/matplotlib-basics.ipynb
1f83c8d 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update foundations/github/basic-git.md
1ea3070 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
brian-rose and others added 4 commits June 14, 2026 21:15
@brian-rose @kafitzgerald
Update foundations/github/what-is-github.md
717a360 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update foundations/github/github-pull-request.md
2dddf19 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update foundations/github/github-repos.md
0a890f2 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update foundations/github/git-workflows.md
152a8cf 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose

brian-rose commented Jun 15, 2026

Copy link
Copy Markdown
Member Author

A few minor suggestions (take them or leave them), but this looks great overall. A nice improvement!

Thanks for these super helpful edits!

@brian-rose brian-rose requested a review from kafitzgerald June 15, 2026 03:29
kafitzgerald
kafitzgerald previously approved these changes Jun 15, 2026
View reviewed changes

@kafitzgerald kafitzgerald left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a few (3) more suggestions remaining in the matplotlib-basics, dask-arrays-xarray, and xarray-intro notebooks. No worries, if you'd rather dismiss them, but I know sometimes they're hard to see so figured I would point those out.

Either way is fine with me and this looks great IMO!

@brian-rose @kafitzgerald
Update core/matplotlib/matplotlib-basics.ipynb
487fac9 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
brian-rose and others added 2 commits June 15, 2026 17:05
@brian-rose @kafitzgerald
Update core/xarray/dask-arrays-xarray.ipynb
c29c2dc 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose @kafitzgerald
Update core/xarray/xarray-intro.ipynb
a42a45e 
Co-authored-by: Katelyn FitzGerald <7872563+kafitzgerald@users.noreply.github.com>
@brian-rose

brian-rose commented Jun 15, 2026

Copy link
Copy Markdown
Member Author

There are a few (3) more suggestions remaining in the matplotlib-basics, dask-arrays-xarray, and xarray-intro notebooks. No worries, if you'd rather dismiss them, but I know sometimes they're hard to see so figured I would point those out.

Shoot, I think I missed those because they got "hidden". Should be all done now!

I think I'll need another "Approval" before I can merge.

@brian-rose brian-rose requested a review from kafitzgerald June 15, 2026 23:07
@brian-rose brian-rose merged commit 9a3dbc4 into ProjectPythia:main Jun 16, 2026
3 checks passed
@brian-rose brian-rose deleted the overviews branch June 16, 2026 02:37
github-actions Bot pushed a commit that referenced this pull request Jun 16, 2026
@brian-rose @github-actions
Delete preview for pull request \#650
4a6acbe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kafitzgerald kafitzgerald kafitzgerald approved these changes
@dopplershift dopplershift Awaiting requested review from dopplershift dopplershift is a code owner automatically assigned from ProjectPythia/education
@r-ford r-ford Awaiting requested review from r-ford r-ford is a code owner automatically assigned from ProjectPythia/education

Assignees

No one assigned

Labels

content Content related issue JOSE revision

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Consistency of "Overview" sections

2 participants

@brian-rose @kafitzgerald