8000 Backport PR: Series rolling count ignores min_periods by fujiaxiang · Pull Request #31320 · pandas-dev/pandas · GitHub
[go: up one dir, main page]
Skip to content

Backport PR: Series rolling count ignores min_periods #31320

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

Merged
merged 12 commits into from
Jan 26, 2020
Merged

Backport PR: Series rolling count ignores min_periods #31320

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
74cba7d
BUG: Series rolling count ignores min_periods (GH26996)
fujiaxiang Jan 11, 2020
bd0fb50
ENH: handles min_periods argument in rolling.count (GH26996)
fujiaxiang Jan 17, 2020
61226b5
DOC: moved whatsnew to V1.1.0
fujiaxiang Jan 18, 2020
4a93069
TST: added more tests (GH26996)
fujiaxiang Jan 21, 2020
14a4fcb
BUG: updated rolling and expanding count for consistency (GH26996)
fujiaxiang Jan 21, 2020
4988b4f
CLN: further cleaned code (GH26996)
fujiaxiang Jan 22, 2020
1e48ac3
TST: cleaned up tests (GH26996)
fujiaxiang Jan 24, 2020
c091787
BUG: changed min_periods default to 0 for rolling and expanding (GH26…
fujiaxiang Jan 24, 2020
f905cbb
BUG: reverted non-relevant changes accidentally added (GH26996)
fujiaxiang Jan 24, 2020
0a27a21
BUG: small change in whatsnew (GH26996)
fujiaxiang Jan 25, 2020
337f1ae
CLN: slight cleanup (GH26996)
fujiaxiang Jan 25, 2020
12e1f09
removed redundant file
fujiaxiang Jan 26, 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
fujiaxiang committed Jan 26, 2020
commit f905cbbd5452af6c34cbffa52668713bf1177e4e
3 changes: 0 additions & 3 deletions pandas/_libs/tslibs/timestamps.pyx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -374,13 +374,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(f'tzinfo must be a datetime.tzinfo object, '
Expand All @@ -392,7 +390,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
0