Skip to content

Notebooks - Add original authors to each#18

Merged
aaarendt merged 4 commits into
ProjectPythia:mainfrom
jomey:authors
Apr 17, 2026
Merged

Notebooks - Add original authors to each#18
aaarendt merged 4 commits into
ProjectPythia:mainfrom
jomey:authors

Conversation

@jomey
Copy link
Copy Markdown
Collaborator

@jomey jomey commented Apr 9, 2026

Adding the original authors to all notebooks to give credit.
I am not sure where the TLS or SNOTEL Data access came from and did not add one there.

@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
Contributor

github-actions Bot commented Apr 9, 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: bc9be9e
✅ Deployment Preview URL: In Progress

@jomey
Copy link
Copy Markdown
Collaborator Author

jomey commented Apr 9, 2026

Just noting that one build failure is because it can not access a DOI. However, there is a "bot" checker and my guess is that it fails because of that. The URL is correct.

The error:

notebooks/timelapse-camera-tutorial.ipynb
  - ⛔ Could not find DOI "https://doi.org/10.1002/2013WR013958" from doi.org as doi:10.1002/2013WR013958

@aaarendt
Copy link
Copy Markdown
Collaborator

@jomey can you push an empty commit to trigger the CI again? I think my edits should have this fixed now

@jomey jomey marked this pull request as draft April 17, 2026 17:55
@jomey jomey marked this pull request as ready for review April 17, 2026 17:55
Take two
@jomey
Copy link
Copy Markdown
Collaborator Author

jomey commented Apr 17, 2026

@jomey can you push an empty commit to trigger the CI again? I think my edits should have this fixed now

Just tried a force push adding to your last commit without luck.

@aaarendt aaarendt merged commit 68c74a6 into ProjectPythia:main Apr 17, 2026
@jomey jomey deleted the authors branch April 17, 2026 18:40
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.

2 participants