search

ProjectPythia / cmip6-cookbook

Examples of analysis of Google Cloud CMIP6 data using Pangeo tools
https://projectpythia.org/cmip6-cookbook/
Apache License 2.0
11 stars 9 forks source link

Try to fix build #36

Closed r-ford closed 1 year ago

r-ford commented 1 year ago

Same as https://github.com/ProjectPythia/cmip6-cookbook/pull/32

github-actions[bot] commented 1 year ago

👋 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: 7649a4717b909a85de5f5a90d14bf69b0c65a7ac ✅ Deployment Preview URL: https://ProjectPythia.github.io/cmip6-cookbook/_preview/36

r-ford commented 1 year ago

Based on the KeyError here, it looks like it is trying to find imbalance in the coordinates.

r-ford commented 1 year ago

@brian-rose, I'm trying to use a previous Xarray version, but it doesn't look like the execution environment gets updated during the build. Is there a way to specify that I want to update it?

brian-rose commented 1 year ago

@brian-rose, I'm trying to use a previous Xarray version, but it doesn't look like the execution environment gets updated during the build. Is there a way to specify that I want to update it?

Since you're using execute_notebooks: binder, the execution environment uses the image from the main branch of the repo. This is a limitation in how we are doing things right now. Changes to the environment.yml file in the PR will not have any effect on the Binder environment.

brian-rose commented 1 year ago

A simple workaround is just to merge your environment changes into main, and then test your new code. Sort of a hack, but that will accomplish what you want.

brian-rose commented 1 year ago

This line of code here in the cookbook actions repo is where the call to binderbot is made, and we are currently hard-coding to the main branch.

r-ford commented 1 year ago

Ok, that seemed to work. I'll just merge to see if it fixes the main branch.