xarray icon indicating copy to clipboard operation
xarray copied to clipboard

Clarify pydata#8728 in docs

Open ks905383 opened this issue 1 year ago • 2 comments

  • Add reference to numpy docs on view / copies in the corresponding section of the xarray docs, to help clarify pydata#8728 .
  • Add note that other xarray operations also return views rather than copies in the Copies vs. Views section of the docs
  • Add note that da.values() returns a view in the header for da.values().

ks905383 avatar Feb 13 '24 16:02 ks905383

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 Feb 13 '24 16:02 welcome[bot]

I think the failing check is something that came from the latest main branch update, so once that gets resolved, I'll merge this barring any other comments?

ks905383 avatar Mar 11 '24 17:03 ks905383

does anything else need to happen before it's cleared for merging btw?

ks905383 avatar Mar 25 '24 20:03 ks905383

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 Mar 25 '24 20:03 welcome[bot]