Skip to content

Add snow-observations-cookbook to gallery#29

Open
aaarendt wants to merge 1 commit intoProjectPythia:mainfrom
aaarendt:add-snow-cookbook
Open

Add snow-observations-cookbook to gallery#29
aaarendt wants to merge 1 commit intoProjectPythia:mainfrom
aaarendt:add-snow-cookbook

Conversation

@aaarendt
Copy link
Copy Markdown

This PR is to request the inclusion of the snow observations cookbook to the Project Pythia gallery.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 13, 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: 0fa1d67
✅ Deployment Preview URL: https://ProjectPythia.github.io/cookbook-gallery/_preview/29

@brian-rose
Copy link
Copy Markdown
Member

brian-rose commented Mar 13, 2026

The build seems to be choking on the thumbnail image. I'm not immediately sure why.

Reference: this build log which is producing

⛔️ 940b2fbbda490bffbf834be770b29aa6 Error saving image "https://raw.githubusercontent.com/projectpythia/snow-observations-cookbook/main/thumbnail.png": https://raw.githubusercontent.com/projectpythia/snow-observations-cookbook/main/thumbnail.png content-type "text/plain; charset=utf-8" is not an image
⛔️ root.md Unsupported image extension "<no extension>" may not correctly render.

@brian-rose
Copy link
Copy Markdown
Member

brian-rose commented Mar 13, 2026

The build seems to be looking for the thumbnail image at https://raw.githubusercontent.com/projectpythia/snow-observations-cookbook/main/thumbnail.png instead of https://raw.githubusercontent.com/projectpythia/snow-observations-cookbook/main/thumbnails/thumbnail.png. The path is wrong. Will see if there's an obvious reason.

@brian-rose
Copy link
Copy Markdown
Member

@aaarendt the build failure is because the thumbnail image is in a subdirectory thumbnails in your source repo, but this is inconsistent with what is specified in your _gallery_info.yml.

The reason for this inconsistency seems to originate in our Cookbook Template, so this needs to be fixed upstream.

But the solution here is to do either:

  1. Move your thumbnail.png to the root of the source repo, or
  2. Edit this line of your _gallery_info.yml to have
thumbnail: thumbnails/thumbnail.png

Either one should work fine, and it's just a matter of taste for how you keep your repo organized.

@aaarendt
Copy link
Copy Markdown
Author

Thanks @brian-rose, I moved the thumbnail to the root directory as suggested.

@brian-rose brian-rose closed this Mar 14, 2026
@brian-rose brian-rose reopened this Mar 14, 2026
@brian-rose
Copy link
Copy Markdown
Member

Gallery build is passing now. I will make some suggestions over at https://github.com/ProjectPythia/snow-observations-cookbook to resolve some current build issues for the cookbook.

@aaarendt
Copy link
Copy Markdown
Author

Thanks @brian-rose. Is this to do with the nightly builds failing? I've tried to fix but couldn't figure it out. Let me know if I can assist.

@brian-rose
Copy link
Copy Markdown
Member

Thanks @brian-rose. Is this to do with the nightly builds failing? I've tried to fix but couldn't figure it out. Let me know if I can assist.

On closer inspection, the nightly builds are passing, it's just that the badge on the README page is still pointing to the Cookbook Template (which is currently broken). I will open a PR to fix that.

@brian-rose
Copy link
Copy Markdown
Member

@aaarendt
Copy link
Copy Markdown
Author

Thanks @brian-rose ! I approved and merged ProjectPythia/snow-observations-cookbook#14

@aaarendt
Copy link
Copy Markdown
Author

@brian-rose I think this is ready to merge?

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