Skip to content

Flexible indexing blog post #795

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 32 commits into from
Sep 2, 2025
Merged

Flexible indexing blog post #795

merged 32 commits into from
Sep 2, 2025

Conversation

scottyhq
Copy link
Contributor

Supercedes #597

@vercel Vercel
Copy link

vercel bot commented Aug 11, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
xarray-dev Ready Ready Preview Comment Sep 2, 2025 8:32pm

@netlify Netlify
Copy link

netlify bot commented Aug 11, 2025 •
edited
Loading

✅ Deploy Preview for xarraydev ready!

Name Link
🔨 Latest commit bdef441
🔍 Latest deploy log https://app.netlify.com/projects/xarraydev/deploys/68b753ea548f880008d12be1
😎 Deploy Preview https://deploy-preview-795--xarraydev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

keewis
keewis reviewed Aug 13, 2025
View reviewed changes
Copy link
Member

@keewis keewis left a comment

Choose a reason for hiding this comment

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

thanks, Scott, that was a nice read.

I didn't check how this looks, but from the markdown at least this looks good to me in general. I do have a few comments on the details, though.

keewis
keewis reviewed Aug 13, 2025
View reviewed changes
@scottyhq scottyhq mentioned this pull request Aug 13, 2025
Open
dcherian
dcherian reviewed Aug 23, 2025
View reviewed changes
dcherian
dcherian reviewed Aug 23, 2025
View reviewed changes
dcherian
dcherian reviewed Aug 23, 2025
View reviewed changes
dcherian
dcherian reviewed Aug 23, 2025
View reviewed changes
scottyhq and others added 2 commits August 25, 2025 13:51
@scottyhq @dcherian
Nod to geopandas
b8ce048 
Co-authored-by: Deepak Cherian <[email protected]>
@scottyhq @dcherian
link to index gallery in tldr
f6b0289 
Co-authored-by: Deepak Cherian <[email protected]>
@scottyhq
Copy link
Contributor Author

I left a few pretty minor suggestions. Let's get this out next week!

Thanks, publishing this week sounds good to me! I added your suggestions and went over one more time with a few tweaks. @benbovy do you have time this week to have a look?

benbovy
benbovy approved these changes Sep 1, 2025
View reviewed changes
Copy link
Member

@benbovy benbovy left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thank you so much @scottyhq for putting this together! And sorry for the late review.

Overall this looks great! I added a few comments, mostly nit picking.

src/posts/flexible-indexing/index.md Outdated
Comment on lines 5 to 8
- name: Benoît Bovy
github: benbovy
- name: Scott Henderson
github: scottyhq
Copy link
Member

Choose a reason for hiding this comment

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

@scottyhq since you authored this blog post I think it would make sense to add you as 1st author here, or at least add something that gives you credit for authoring this post.

I'm thinking of authoring a follow-up technical blog post on how to customize Xarray's behavior via indexes for common operations such as assigning, indexing and alignment (with flowcharts illustrating the logic of Xarray internals and how the index API is called).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, either way is fine by me. want to make sure you're recognized as leading the effort overall!

@scottyhq
Copy link
Contributor Author

scottyhq commented Sep 2, 2025

Thanks for the review @benbovy I've incorporated all your feedback. I'm going to go ahead and merge it in, thanks again everyone for helping with this!

@scottyhq scottyhq merged commit f384aac into main Sep 2, 2025
7 checks passed
@scottyhq scottyhq deleted the flexible-indexes2 branch September 2, 2025 20:35
@keewis
Copy link
Member

keewis commented Sep 2, 2025 •
edited
Loading

the only thing we forgot to do is update the blog post release date: right now it says "23 days ago"

@scottyhq scottyhq mentioned this pull request Sep 2, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@keewis keewis keewis left review comments

@dcherian dcherian dcherian approved these changes

@benbovy benbovy benbovy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@scottyhq @keewis @dcherian @benbovy