SciTools / iris-esmf-regrid

A collection of structured and unstructured ESMF regridding schemes for Iris.
https://iris-esmf-regrid.readthedocs.io/en/latest
BSD 3-Clause "New" or "Revised" License
20 stars 17 forks source link

Review the API docstrings #158

Closed pp-mo closed 1 year ago

pp-mo commented 2 years ago

📰 Custom Issue

Great to have the built API docs up. But just roughly scanning, I reckon there is quite a bit in there that needs some attention / improvement. For instance (in my personal views)..

So, I'd suggest this could do with thorough general review, when we can find time.

I know from bitter experience (1) how very hard is to precisely + clearly describe these different types of coordinate mappings, given that nearly all the recognised technical terms are used ambiguously in different contexts, and (2) how confused users are about these distinctions in the first place So, it's always an uphill struggle to define terms clearly.
But we can at least aim to be complete + consistent in our usage -- which would at least be definite progress, since the existing Iris docs have made rather a mess of it !!

github-actions[bot] commented 2 years ago

✨ Congratulations! ✨ Thanks for submitting your first issue to iris-esmf-regrid. We really appreciate it and will get back to you as soon as possible. Awesome job 👍

github-actions[bot] commented 2 years ago

@SciTools-incubator/esmf-regrid-devs This issue is stale due to a lack of activity in the last 90 days. Remove stale label or comment, otherwise this issue will close automatically in 7 days time.

trexfeathers commented 2 years ago

We did make a lot of progress in #155, but not focused on the things mentioned here. Should this be kept open @pp-mo?

github-actions[bot] commented 1 year ago

@SciTools-incubator/esmf-regrid-devs This issue is stale due to a lack of activity in the last 90 days. Remove stale label or comment, otherwise this issue will close automatically in 7 days time.

github-actions[bot] commented 1 year ago

@SciTools-incubator/esmf-regrid-devs This stale issue has been automatically closed due to no community activity