New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

More changes to regridding section #11

Closed

6 of 8 tasks

pp-mo opened this issue Jan 25, 2023 · 1 comment

Contributor

pp-mo commented Jan 25, 2023 •

edited

Loading

As in this review comment

@pp-mo I have made further some comments + suggestions, but I think we could get on and merge this, and address those issues later.

Quick list -

align sample code with prompt
clearly state cost/accuracy tradeoff
small fixes to sample code here
... and to same here
review lengthy calc
?? convert all to C48 data
??convert all to use (mostly) user-provided code in examples
remove redundant imports from initial 'Setup' section

pp-mo mentioned this issue

Add regridding section #9

Merged

stephenworsley mentioned this issue

Address additional comments for regridding section #12

Merged

Contributor Author

pp-mo commented Feb 3, 2023

Essentially all done.
The remaining points (marked ??) are ongoing generally, TBD everywhere (to some extent, not yet agreed)

pp-mo closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment