Chuck Daniels

Allow `help` to exit with status other than 1

2

I would like to be able to exit with status 0 when specifying `-h` or `--help`, but it appears that `1` is hard-coded as the exit status. Perhaps providing a...

It appears that although a `--log` option is available for commands, it is moot, as `logging` is not used, but rather all messages appear to be written via the `print`...

Please either remove the dependency on `click` (it appears not to be used), or upgrade to `click>=7`

Invalid [bad-unpacking] error when using "catch-all"

1

When using a "catch-all" with unpacking, pytype incorrectly reports a [bad-unpacking] error. As a simple case: ```python first, *rest = ["foo"] ``` This is valid, where `first` is assigned `"foo"`...

aiohttp.ClientSession read_timeout is deprecated

3

When using `pfun.http` (pfun 0.13.1), I get the following deprecation warning: ``` .../pfun/http.py:104: DeprecationWarning: read_timeout is deprecated, use timeout argument instead ```

The SandboxTransformer does not support "tuple" assignment ([object destructuring with multiple assignment](http://groovy-lang.org/semantics.html#_object_destructuring_with_multiple_assignment)), as per the following: https://github.com/jenkinsci/groovy-sandbox/blob/01744ccb8f4e5fefe1ff48b3258fa6905583a46c/src/main/java/org/kohsuke/groovy/sandbox/SandboxTransformer.java#L729 Is there any plan to implement this?

Passing preprocessor to MultiZarrToZarr raises PicklingError

11

To possibly bypass the memory issue I reported in #614, I've attempted to use kerchunk instead, based upon the [HDF Reference Recipe for CMIP6](https://pangeo-forge.readthedocs.io/en/latest/pangeo_forge_recipes/tutorials/hdf_reference/reference_cmip6.html?highlight=HDF5#hdf-reference-recipe-for-cmip6), but also to no avail: ```python...

Xarray-to-Zarr recipe runs out of memory

15

I'm following the PangeoForge recipe tutorial [Xarray-to-Zarr Sequential Recipe: NOAA OISST](https://pangeo-forge.readthedocs.io/en/latest/pangeo_forge_recipes/tutorials/xarray_zarr/netcdf_zarr_sequential.html) to create a recipe for [CEDA monthly daytime land surface temperature data](https://data.ceda.ac.uk/neodc/esacci/land_surface_temperature/data/MULTISENSOR_IRCDR/L3S/0.01/v2.00/monthly), but I'm running into issues using pangeo-forge-recipes...

Fix CMR-related type hints

3

There were a number of type hints in `search.py` and `api.py` related to CMR queries that were incorrect. These were fixed. In addition, there were a number of other static...

Along the lines of #508, we should add type stubs for `tinynetrc` since it is a very small API, and doing so will allow more thorough type checking.