Skip to content
This repository was archived by the owner on Sep 11, 2023. It is now read-only.

Update NWPDataSource with new NWP variable names #371

Closed
Tracked by #393
JackKelly opened this issue Nov 12, 2021 · 1 comment · Fixed by #387
Closed
Tracked by #393

Update NWPDataSource with new NWP variable names #371

JackKelly opened this issue Nov 12, 2021 · 1 comment · Fixed by #387
Assignees
Labels
enhancement New feature or request

Comments

@JackKelly
Copy link
Member

JackKelly commented Nov 12, 2021

The new NWP to Zarr conversion script brings in 17 NWP variables (up from 10 in the old Zarr):

['cdcb', 'lcc', 'mcc', 'hcc', 'sde', 'hcct', 'dswrf', 'dlwrf', 'h', 't', 'r', 'dpt', 'vis', 'si10', 'wdir10', 'prmsl', 'prate']

Although, we should do some exploration to make sure we actually want all of these! See the Met Office's documentation: we're consuming and merging "Un-split File 1" and "Un-split File 2" (although the variables available in the grib files doesn't exactly line up with the docs!)

Also change on_premises.yaml to point to the NWP zarr in /mnt/storage_ssd

@JackKelly JackKelly added the enhancement New feature or request label Nov 12, 2021
@JackKelly JackKelly moved this to Todo in Nowcasting Nov 12, 2021
@JackKelly JackKelly self-assigned this Nov 15, 2021
@JackKelly
Copy link
Member Author

JackKelly commented Nov 15, 2021

I think I've finished the code changes in PR #387

Still TODO:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant