google / google/weather-tools

PyPi package issues with regrid

Open
#226 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
252
Forks
60
Avg merge
8d 4h
Merged PRs (30d)
3

Description

PyPi distribution seems to not understand `output_path` in regrid and the extra is not provided. The package requires `output_table` for `rg`, which shouldn't be the case (`output_path` is enough).

## `output_path` Error

**Environment**
```
$ python3 -m venv env
$ . env/bin/activate
$ pip install google-weather-tools[regrid]
```

**Command**
```bash
weather-mv rg --uris "gs://$BUCKET/download/*.nc" --output_path "gs://$BUCKET/download-rg-$(date +%s)/" --dry-run
```

**Error**
```
weather-mv: error: the following arguments are required: -o/--output_table
```

**manual**
```
$ weather-mv
usage: weather-mv [-h] -i URIS -o OUTPUT_TABLE [-v variables [variables ...]] [-a area [area ...]] [--topic TOPIC] [--window_size WINDOW_SIZE] [--num_shards NUM_SHARDS]
[--import_time IMPORT_TIME] [--infer_schema] [--xarray_open_dataset_kwargs XARRAY_OPEN_DATASET_KWARGS] [--coordinate_chunk_size COORDINATE_CHUNK_SIZE]
[--tif_metadata_for_datetime TIF_METADATA_FOR_DATETIME] [-d] [--disable_in_memory_copy] [-s]
weather-mv: error: the following arguments are required: -i/--uris, -o/--output_table
```

## Extra
The regrid extra [mentioned](https://github.com/google/weather-tools/blob/bf6fe55226ed1ca13c46434ca2bdbbba0dd45984/weather_mv/README.md?plain=1#L193) in README instructions doesn't seem to exist (tried all available versions):

```
WARNING: google-weather-tools 0.3.2 does not provide the extra 'regrid’
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.