Skip to content

BUG: Series rolling count ignores min_periods #30923

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.

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

Merged
Merged
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
398bf7d
BUG: Series rolling count ignores min_periods (GH26996)
fujiaxiang Jan 11, 2020
5d471c9
Merge remote-tracking branch 'upstream/master' into series_rolling_co…
fujiaxiang Jan 17, 2020
47f4e61
ENH: handles min_periods argument in rolling.count (GH26996)
fujiaxiang Jan 17, 2020
df2a3e9
DOC: moved whatsnew to V1.1.0
fujiaxiang Jan 18, 2020
554abfa
Merge remote-tracking branch 'upstream/master' into series_rolling_co…
fujiaxiang Jan 18, 2020
c4878b0
Merge remote-tracking branch 'upstream/master' into series_rolling_co…
fujiaxiang Jan 18, 2020
b0f5baa
Merge remote-tracking branch 'upstream/master' into series_rolling_co…
fujiaxiang Jan 19, 2020
bfe10f0
TST: added more tests (GH26996)
fujiaxiang Jan 21, 2020
1a90629
Merge branch 'master' into series_rolling_count_ignores_min_periods
fujiaxiang Jan 21, 2020
be0926b
Merge remote-tracking branch 'upstream/master' into series_rolling_co…
fujiaxiang Jan 21, 2020
c0046b6
BUG: updated rolling and expanding count for consistency (GH26996)
fujiaxiang Jan 21, 2020
8807ba1
CLN: further cleaned code (GH26996)
fujiaxiang Jan 22, 2020
7aa1582
Merge remote-tracking branch 'upstream/master' into series_rolling_co…
fujiaxiang Jan 23, 2020
bcce129
TST: cleaned up tests (GH26996)
fujiaxiang Jan 24, 2020
1a4352b
BUG: changed min_periods default to 0 for rolling and expanding (GH26…
fujiaxiang Jan 24, 2020
9fa4531
BUG: reverted non-relevant changes accidentally added (GH26996)
fujiaxiang Jan 24, 2020
987c033
BUG: small change in whatsnew (GH26996)
fujiaxiang Jan 25, 2020
a00532a
CLN: slight cleanup (GH26996)
fujiaxiang Jan 25, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
BUG: reverted non-relevant changes accidentally added (GH26996)
  • Loading branch information
fujiaxiang committed Jan 24, 2020
commit 9fa453174bcd03f7a57d39b406cdef674a8269bb
3 changes: 0 additions & 3 deletions pandas/_libs/tslibs/timestamps.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -375,13 +375,11 @@ class Timestamp(_Timestamp):
# Mixing pydatetime positional and keyword arguments is forbidden!

cdef _TSObject ts
print('haha')

_date_attributes = [year, month, day, hour, minute, second,
microsecond, nanosecond]

if tzinfo is not None:
print('tzinfo not None')
if not PyTZInfo_Check(tzinfo):
# tzinfo must be a datetime.tzinfo object, GH#17690
raise TypeError(
Expand All @@ -394,7 +392,6 @@ class Timestamp(_Timestamp):
tz, tzinfo = tzinfo, None

if isinstance(ts_input, str):
print('ts_input is str')
# User passed a date string to parse.
# Check that the user didn't also pass a date attribute kwarg.
if any(arg is not None for arg in _date_attributes):
Expand Down