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

BUG: DatetimeIndex.is_year_start and DatetimeIndex.is_quarter_start always return False on double-digit frequencies #58549

Merged
merged 11 commits into from
May 10, 2024
Merged

BUG: DatetimeIndex.is_year_start and DatetimeIndex.is_quarter_start always return False on double-digit frequencies #58549

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
8c9aaa8
correct def get_start_end_field, add test, add a note to v3.0.0
natmokval May 3, 2024
21bd212
replace regex with to_offset
natmokval May 7, 2024
9f094d7
replace offset.freqstr.replace with offset.name
natmokval May 8, 2024
670a8db
move to_offset from get_start_end_field up
natmokval May 9, 2024
adaf974
fixup test_is_yqm_start_end
natmokval May 9, 2024
d2fab60
Merge branch 'main' into bug-DatetimeIndex-is_year_start-breaks-on-do…
natmokval May 9, 2024
57a834e
rename the argument freqstr in get_start_end_field
natmokval May 9, 2024
d77f248
rename the variable freqstr in _field_accessor
natmokval May 9, 2024
65e6301
simplify to_offset(freq.freqstr).name
natmokval May 9, 2024
e99a2c6
Merge branch 'main' into bug-DatetimeIndex-is_year_start-breaks-on-do…
natmokval May 10, 2024
45a2324
rename the variable freqstr
natmokval May 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions doc/source/whatsnew/v3.0.0.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -410,6 +410,7 @@ Interval
Indexing
^^^^^^^^
- Bug in :meth:`DataFrame.__getitem__` returning modified columns when called with ``slice`` in Python 3.12 (:issue:`57500`)
- Bug in :meth:`DatetimeIndex.is_year_start` and :meth:`DatetimeIndex.is_quarter_start` returning ``False`` on double-digit frequencies (:issue:`58523`)
Copy link
Member

Choose a reason for hiding this comment

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

this should have been in Datetimelike, "indexing" is more like .loc / .iloc / get/setitem stuff

but OK to address this as part of https://github.com/pandas-dev/pandas/pull/58665/files, as that one needs updating anyway

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks, I addressed this comment in the PR you suggested

-

Missing
Expand Down
4 changes: 2 additions & 2 deletions pandas/_libs/tslibs/fields.pyx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ from pandas._libs.tslibs.np_datetime cimport (
pandas_timedelta_to_timedeltastruct,
pandas_timedeltastruct,
)
from pandas._libs.tslibs.offsets cimport to_offset

import_pandas_datetime()

Expand Down Expand Up @@ -253,8 +254,7 @@ def get_start_end_field(
# month of year. Other offsets use month, startingMonth as ending
# month of year.

if (freqstr[0:2] in ["MS", "QS", "YS"]) or (
freqstr[1:3] in ["MS", "QS", "YS"]):
if to_offset(freqstr).name[0:2] in ["MS", "QS", "YS"]:
end_month = 12 if month_kw == 1 else month_kw - 1
start_month = month_kw
else:
Expand Down
8 changes: 8 additions & 0 deletions pandas/tests/indexes/datetimes/test_scalar_compat.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -328,3 +328,11 @@ def test_dti_is_month_start_custom(self):
msg = "Custom business days is not supported by is_month_start"
with pytest.raises(ValueError, match=msg):
dti.is_month_start

def test_dti_is_year_quarter_start_doubledigit_freq(self):
# GH#58523
dr = date_range("2017-01-01", periods=2, freq="10YS")
assert all(dr.is_year_start)

dr = date_range("2017-01-01", periods=2, freq="10QS")
assert all(dr.is_quarter_start)