Skip to content

Update Numpy Basics notebook#600

Merged
brian-rose merged 2 commits intoProjectPythia:mainfrom
brian-rose:trapz
Feb 17, 2026
Merged

Update Numpy Basics notebook#600
brian-rose merged 2 commits intoProjectPythia:mainfrom
brian-rose:trapz

Conversation

@brian-rose
Copy link
Copy Markdown
Member

Closes #599

Replaces np.trapz with np.trapezoid (consistent with numpy version 2 and above)

Also updates the admonitions in this chapter to nicer MyST formats.

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 16, 2026

👋 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: 12c8f36
✅ Deployment Preview URL: https://ProjectPythia.github.io/pythia-foundations/_preview/600

@brian-rose brian-rose marked this pull request as ready for review February 16, 2026 20:35
@brian-rose brian-rose requested a review from a team as a code owner February 16, 2026 20:35
@brian-rose brian-rose requested review from dcamron and mgrover1 and removed request for a team February 16, 2026 20:35
Copy link
Copy Markdown
Contributor

@kafitzgerald kafitzgerald left a comment

Choose a reason for hiding this comment

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

Looks good (double checked the preview as well)!

@brian-rose brian-rose merged commit 999853d into ProjectPythia:main Feb 17, 2026
3 checks passed
@brian-rose brian-rose deleted the trapz branch February 17, 2026 16:50
github-actions bot pushed a commit that referenced this pull request Feb 17, 2026
brian-rose added a commit that referenced this pull request Feb 17, 2026
* Update call to numpy.trapezoid and fix admonitions

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

trapz has been removed from numpy 2 and above

2 participants