pull_requests: 608110624
This data as json
id | node_id | number | state | locked | title | user | body | created_at | updated_at | closed_at | merged_at | merge_commit_sha | assignee | milestone | draft | head | base | author_association | auto_merge | repo | url | merged_by |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
608110624 | MDExOlB1bGxSZXF1ZXN0NjA4MTEwNjI0 | 5102 | closed | 0 | Flexible indexes: add Index base class and xindexes properties | 4160723 | This PR clears up the path for flexible indexes: - it adds a new ~~`IndexAdapter`~~ `Index` base class that is meant to be inherited by all xarray-compatible indexes (built-in or 3rd-party) - `PandasIndexAdapter` now inherits from ~~`IndexAdapter`~~ `Index` - the `xarray_obj.xindexes` properties return `Index` (`PandasIndexAdapter`) instances. `xarray_obj.indexes` properties still return `pandas.Index` instances. ~~The latter is a breaking change, although I'm not sure if the `indexes` property has been made public yet.~~ This is still work in progress, there are many broken tests that are not fixed yet. (EDIT: all tests should be fixed now). There's a lot of dirty fixes to avoid circular dependencies and in the many places where we still need direct access to the `pandas.Index` objects, but I'd expect that these will be cleaned-up further in the refactoring. | 2021-04-02T16:18:07Z | 2022-03-29T07:10:07Z | 2021-05-11T08:21:26Z | 2021-05-11T08:21:26Z | 6e14df62f0b01d8ca5b04bd0ed2b5ee45444265d | 0 | ce59dece723ca49eaae69779dee5da2aa30d0286 | 234b40a37e484a795e6b12916315c80d70570b27 | MEMBER | 13221727 | https://github.com/pydata/xarray/pull/5102 |
Links from other tables
- 1 row from pull_requests_id in labels_pull_requests