Skip to content

More builtin indexes #20

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
benbovy merged 25 commits into from
Jul 9, 2025
Merged

More builtin indexes #20

benbovy merged 25 commits into from
Jul 9, 2025

Conversation

@benbovy
Copy link
Member

@benbovy benbovy commented Jul 8, 2025 •
edited
Loading

  • the default PandasIndex
  • stick coordinates together with PandasMultiIndex
  • large integer ranges with pandas.RangeIndex
  • float ranges with RangeIndex

benbovy
benbovy commented Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
scottyhq
scottyhq reviewed Jul 8, 2025
View reviewed changes
benbovy and others added 16 commits July 8, 2025 20:18
@benbovy @dcherian
Update docs/builtin/pdindex.md
7256fbc 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
7a1bb8b 
for more information, see https://pre-commit.ci
@benbovy @dcherian
Update docs/builtin/pdmultiindex.md
86bbb06 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@benbovy @dcherian
Update docs/builtin/pdmultiindex.md
43f0b2d 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@benbovy @dcherian
Update docs/builtin/pdmultiindex.md
9f59c22 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@benbovy @dcherian
Update docs/builtin/pdmultiindex.md
d5269c2 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@benbovy @dcherian
Update docs/builtin/pdmultiindex.md
0d02eeb 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@benbovy @dcherian
Update docs/builtin/pdmultiindex.md
abcd001 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@benbovy
re-arrange PandasMultiIndex
126d3cd
@benbovy
fix
2c87389
@benbovy
Merge branch 'main' into more-builtin-indexes
4da3f5d
@benbovy
change PandasMultiIndex title
38d8f82
@benbovy
fix PandasIndex and PandasMultiIndex cross-refs
7afb715
@benbovy
nit
b481922
@benbovy
temp: install Xarray main branch
b50b805 
This is needed for pd.RangeIndex with "lazy" coordinate variable. This
will be needed for NDPointIndex too.

TODO: remove when next version of Xarray is released.
@benbovy
add pandas.RangeIndex and RangeIndex
151e6f9
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
docs/builtin/pdindex.md
ds_air.sel(time="2013")
```

### Set indexes for non-dimension coordinates
Copy link
Contributor

@dcherian dcherian Jul 8, 2025

Choose a reason for hiding this comment

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

Suggested change
### Set indexes for non-dimension coordinates
### Set indexes for any coordinate variable

dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
dcherian
dcherian reviewed Jul 8, 2025
View reviewed changes
Copy link
Contributor

@dcherian dcherian 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.

Nice!

Would be good to link to RangeIndex and RasterIndex as examples in the CoordinateTransformIndex page. I think you have mentioned this already, but just writing it out as a reminder

benbovy and others added 3 commits July 9, 2025 09:35
@benbovy @dcherian
Update docs/builtin/pdrange.md
abd9a34 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@benbovy @dcherian
Update docs/builtin/range.md
7d4148a 
Co-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>
@benbovy
Revert "Update docs/builtin/pdrange.md"
83e436e 
This reverts commit abd9a34.
@benbovy benbovy closed this Jul 9, 2025
@benbovy benbovy reopened this Jul 9, 2025
@benbovy
Copy link
Member Author

benbovy commented Jul 9, 2025

Thanks for the review @dcherian and @scottyhq !

I'm merging this now.

Would be good to link to RangeIndex and RasterIndex as examples in the CoordinateTransformIndex page. I think you have mentioned this already, but just writing it out as a reminder

Will do in a follow-up PR.

@benbovy benbovy merged commit b63f78a into xarray-contrib:main Jul 9, 2025
2 checks passed
@benbovy benbovy deleted the more-builtin-indexes branch July 9, 2025 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@scottyhq scottyhq scottyhq left review comments

@dcherian dcherian dcherian 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.

3 participants

@benbovy @dcherian @scottyhq