xarray icon indicating copy to clipboard operation
xarray copied to clipboard

Updates to Dask page in Xarray docs

Open scharlottej13 opened this issue 1 year ago • 3 comments

  • [ ] Closes #xxxx
  • [ ] Tests added
  • [ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
  • [ ] New functions/methods are listed in api.rst

Hi! It's been a while since the Dask docs page had a comprehensive update, so thought I'd open a PR with some suggestions.

I chatted a little bit about updating this page with @jhamman and @dcherian, some of the things I was trying to keep in mind:

  • Include more examples
  • Organize information around what the user is trying to do
  • Update outdated info

There are certainly a lot of things I wasn't sure about (I'm certainly not an xarray expert). I'll comment in-line with questions.

cc @phofl @jrbourbeau

scharlottej13 avatar Sep 14 '24 01:09 scharlottej13

Thank you for opening this pull request! It may take us a few days to respond here, so thank you for being patient. If you have questions, some answers may be found in our contributing guidelines.

welcome[bot] avatar Sep 14 '24 01:09 welcome[bot]

@shoyer @TomNicholas thank you both for the review!

I believe I've addressed all of your comments, but please let me know if I missed anything.

scharlottej13 avatar Sep 16 '24 23:09 scharlottej13

@shoyer @TomNicholas gentle ping here, thanks again for the thorough review!

scharlottej13 avatar Sep 26 '24 00:09 scharlottej13

@shoyer @TomNicholas another gentle ping here, thanks again!

scharlottej13 avatar Oct 28 '24 23:10 scharlottej13

Sorry for the really long delay here @scharlottej13

dcherian avatar Nov 09 '24 01:11 dcherian

Congratulations on completing your first pull request! Welcome to Xarray! We are proud of you, and hope to see you again! celebration gif

welcome[bot] avatar Nov 09 '24 01:11 welcome[bot]

Thank you @dcherian!!

scharlottej13 avatar Nov 09 '24 01:11 scharlottej13