Skip to content

Add a section about tree based indexing#350

Merged
ianhi merged 12 commits intomainfrom
ian/trees
Feb 6, 2026
Merged

Add a section about tree based indexing#350
ianhi merged 12 commits intomainfrom
ian/trees

Conversation

@ianhi
Copy link
Contributor

@ianhi ianhi commented Feb 5, 2026

Add a section about tree based indexing. Extracted from xarray-contrib/xarray-indexes#39

@ianhi ianhi changed the title Ian/trees Add a section about tree based indexing Feb 5, 2026
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

🎊 PR Preview 8e26239 has been successfully built and deployed to https://xarray-contrib-xarray-tutorial-preview-pr-350.surge.sh

🕐 Build time: 0.014s

🤖 By surge-preview

@dcherian
Copy link
Contributor

dcherian commented Feb 5, 2026

I'd add it under an Indexing subheading just like the Intermediate section, and link to the NDPointIndex gallery thing right up top also perhaps in a See Also block

Copy link
Contributor

@dcherian dcherian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge away!

PS: if you want to clean up that sidebar, that would be very welcome too

@ianhi
Copy link
Contributor Author

ianhi commented Feb 6, 2026

image idk why this plot is gettign cut off. didn't happen to me locally.

PS: if you want to clean up that sidebar, that would be very welcome too

in this notebook?

@dcherian
Copy link
Contributor

dcherian commented Feb 6, 2026

in this notebook?

nah the left sidebar; it's a mess after last scipy

@ianhi ianhi merged commit 93d41e2 into main Feb 6, 2026
3 checks passed
@ianhi ianhi deleted the ian/trees branch February 6, 2026 16:35
@ianhi
Copy link
Contributor Author

ianhi commented Feb 6, 2026

in this notebook?

nah the left sidebar; it's a mess after last scipy

will do in another pr.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants