Skip to content
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

(fix): extension array indexers #9671

Open
wants to merge 35 commits into
base: main
Choose a base branch
from
Open

(fix): extension array indexers #9671

wants to merge 35 commits into from

Conversation

ilan-gold
Copy link
Contributor

Identical to kmuehlbauer#1 - probably not very helpful in terms of changes since https://github.com/kmuehlbauer/xarray/tree/any-time-resolution-2 contains most of it....

kmuehlbauer and others added 30 commits October 18, 2024 07:31
@kmuehlbauer
implement default_precision_timestamp, refactor coding/times.py and c…
7b5f323 
…ore/variable.py to use any-precision datetime/timedelta with autmatic inferring of resolution
@kmuehlbauer
align tests with new time resolution behaviour
8784f33
@kmuehlbauer
timedelta decoding, fsspec handling
b45ab23
@kmuehlbauer
fixes in coding/times.py
39086ef
@kmuehlbauer
add docs on time coding
df49a40
@kmuehlbauer
attempt fixing doc tests
adb8ca3
@kmuehlbauer
fix issue where out-of-bounds floating point values slipped in the pr…
266b1ed 
…ocessing, raise now early
@kmuehlbauer
convert to UTC first before stripping of tz in _unpack_time_units_and…
6d5f13b 
…_ref_date
@kmuehlbauer
reorganize pandas compatibility code, remove unneeded code, attempt t…
5d68bfe 
…o fix mypy
@kmuehlbauer
another attempt to finally fix mypy
07bba69
@kmuehlbauer
refactor out _check_date_is_after_shift
6e7f0bb
@kmuehlbauer
refactor out _maybe_strip_tz_from_timestamp
b4a49bb
@kmuehlbauer
more refactoring in coding.times.py
2e1ff4f
@kmuehlbauer
more refactoring in coding.times.py
d5a7da0
@kmuehlbauer
minor fix in time-coding.rst
821b68d
@kmuehlbauer
set default resolution to "s", which actually means, use pandas lowes…
d066edf 
…t resolution, fix code and tests to allow this
@kmuehlbauer
Add section for default units, fix options
ed22da1
@kmuehlbauer
attempt to fix typing
8bf23f4
@kmuehlbauer
attempt to fix typing
c3a2b39
@kmuehlbauer
fix scalar datetime/timedelta
3c44aed
@kmuehlbauer
fix user docs
48be73a
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
7ac9983 
for more information, see https://pre-commit.ci
@kmuehlbauer
Fix variable tests, mostly datetime/timedelta is inittialized with us…
d86ad04 
…-resolution
@kmuehlbauer
revert changes in _possible_convert_objects, this needs to be checked…
b5d0795 
… more carefully, for now using pd.Series to covert `OMm` type datetimes/timedeltas (will result in ns precision)
@kmuehlbauer
fix doc link
60324f0
@ilan-gold
(fix): allow all extension array data types in pandas adapters
c2bc4df
@ilan-gold
(fix): dataframes have no array attr
84569bc
@ilan-gold
(fix): allow chunked numpy extension arrays because of `test_pandas_a…
90e390d 
…rray` series creating an extension array when `.array` is accessed
@ilan-gold
(fix): dtypes for PandasIndex
7c32bd0
@ilan-gold
(chore): remove test for unnecessary conversion
795ecf6
@ilan-gold ilan-gold changed the title Ig/fix extension indexer (fix): extension array indexers Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
topic-indexing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ilan-gold @TomNicholas @kmuehlbauer