PyQG Ross2022

[cmdupuis3]

Re: #231

[cisaacstern]

pre-commit.ci autofix

[cisaacstern]

@cmdupuis3 the CI failure is because your PR adds a subdirectory to the top-level of this repo.

As noted here, the contributed directory should be a subdirectory of the recipes directory:

staged-recipes/recipes/
                └──{dataset-name}/
                        ├──meta.yaml
                        └──{recipe-module-name}.py

Could you push a commit moving PyQG_subgrid_forcing_Ross2022 into recipes/?

[cmdupuis3]

pre-commit.ci autofix

[cisaacstern]

@cmdupuis3, two edits here:

1. You're defining a dict of recipes, rather than a single recipe object, so we'll just the dict_object syntax in meta.yaml. Apologies that this is very poorly documented and I'm not sure there is an official source to point to for it at the moment. So no way you could've known!
2. We don't need the list of compiled recipes at the end of the recipe module, so I've suggested to remove that. The backend service will do this compilation automatically (and in a different way) from the dict of recipes.

[cmdupuis3]

pre-commit.ci autofix

[cisaacstern]

/run eddy/low_res.zarr

[pangeo-forge]

The test failed, but I'm sure we can find out why!

Pangeo Forge maintainers are working diligently to provide public logs for contributors.
That feature is not quite ready yet, however, so please reach out on this thread to a
maintainer, and they'll help you diagnose the problem.

[cisaacstern]

/run eddy/low_res.zarr

[pangeo-forge]

The test failed, but I'm sure we can find out why!

Pangeo Forge maintainers are working diligently to provide public logs for contributors.
That feature is not quite ready yet, however, so please reach out on this thread to a
maintainer, and they'll help you diagnose the problem.

[rabernat]

This is now blocked on the issue reported in m2lines/pyqg_parameterization_benchmarks#6

[cmdupuis3]

/run eddy/low_res.zarr

[cmdupuis3]

Hey @cisaacstern, the issue blocking this was resolved a couple of weeks ago, with no changes needed on our end, as I understand it. Do you think we can get this moving again?

On the other hand, my testbed notebook is now throwing this error 😬

ValueError: Starting with Zarr 2.11.0, stores must be subclasses of BaseStore, if your store exposes the MutableMapping interface wrap it in Zarr.storage.KVStore. Got <File-like object HTTPFileSystem, https://g-402b74.00888.8540.data.globus.org/eddy/high_res.zarr>