Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix error in day and lat indices #141

Merged
merged 2 commits into from
Aug 29, 2024
Merged

Fix error in day and lat indices #141

merged 2 commits into from
Aug 29, 2024

Conversation

brian-rose
Copy link
Owner

There's currently an error in the summary plot in the last section of the Orbital notes. The plot is using the wrong index for latitude so is not plotting for 70N despite what the caption says.

This PR fixes that. Thanks to a reader for privately pointing out the error.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

github-actions bot commented Aug 29, 2024

👋 Thanks for opening this PR! The book will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: 7701247
✅ Deployment Preview URL: https://brian-rose.github.io/ClimateLaboratoryBook/_preview/141

@brian-rose brian-rose merged commit bb51898 into main Aug 29, 2024
2 checks passed
@brian-rose brian-rose deleted the fix-insolation-plots branch August 29, 2024 15:32
github-actions bot pushed a commit that referenced this pull request Aug 29, 2024
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