search

xarray-contrib / xdggs

Xarray extension for DGGS
Apache License 2.0
54 stars 9 forks source link

Design docs #20

Closed benbovy closed 9 months ago

benbovy commented 10 months ago

I've tried to put together thoughts and ideas found in the issues of this repository and in the sprint doc (https://hackmd.io/UBM5L6YNRlG73e3eVo6vOg) in a design document.

This is still work in progress (although already quite detailed). Feedback and suggestions are very welcome!

keewis commented 9 months ago

it looks like the remaining points will take a while to discuss, so I don't mind merging this now and continuing the discussion elsewhere (your choice).

benbovy commented 9 months ago

Many thanks @keewis for the feedback. I addressed all your comments that were easily fixable.

I'm +1 for merging this now and continue the discussion in the Discussions section of this repository. Follow-up PRs are also welcome to improve the document!

keewis commented 9 months ago

great, this looks good to me, and let's proceed as you suggested.

Edit: though you'll have to merge yourself, as I don't have the permissions to do that

benbovy commented 9 months ago

Discussion opened there: https://github.com/benbovy/xdggs/discussions/22