Skip to content

Resample BM adds extra index point #9756

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

Closed
bjonen opened this issue Mar 31, 2015 · 6 comments
Closed

Resample BM adds extra index point #9756

bjonen opened this issue Mar 31, 2015 · 6 comments
Labels
Bug Resample resample method
Milestone

Comments

@bjonen
Copy link
Contributor

bjonen commented Mar 31, 2015

In [1]: index = pd.DatetimeIndex(start='20150101', end='20150331', freq='B')

In [2]: df = pd.DataFrame(index=index, data=len(index)*[0])

In [3]: df.resample('BM', how='last')
Out[3]:
             0
2015-01-30   0
2015-02-27   0
2015-03-31   0
2015-04-30 NaN

This is both in 0.14.1 and 0.15.2

@rockg
Copy link
Contributor

rockg commented Mar 31, 2015

This actually might be fixed on master related to a bunch of resample fixes.

@rockg
Copy link
Contributor

rockg commented Mar 31, 2015

Confirmed that it is not fixed on master.

@jreback jreback added Bug Resample resample method labels Apr 12, 2015
@jreback
Copy link
Contributor

jreback commented Apr 12, 2015

couple of resample bins issues still lurking; #9623 fixed a lot of them

@jreback jreback added this to the Next Major Release milestone Apr 12, 2015
@jreback jreback changed the title Resample BM adds nan entry Resample BM adds extra index point Apr 12, 2015
@scari
Copy link
Contributor

scari commented Apr 13, 2015

Working on this.

@scari
Copy link
Contributor

scari commented Apr 14, 2015

Also reproducible /w resample BQ

scari added a commit to scari/pandas that referenced this issue May 1, 2015
@jreback jreback modified the milestones: 0.16.1, Next Major Release May 4, 2015
scari added a commit to scari/pandas that referenced this issue May 4, 2015
@jreback
Copy link
Contributor

jreback commented May 4, 2015

closed by #10044

@jreback jreback closed this as completed May 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Resample resample method
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants