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

DataFrame.asfreq(MonthEnd(0)) currently goes into infinite loop #683

Closed
james18 opened this issue Jan 25, 2012 · 3 comments
Closed

DataFrame.asfreq(MonthEnd(0)) currently goes into infinite loop #683

james18 opened this issue Jan 25, 2012 · 3 comments
Labels
Bug
Milestone
0.7.0

Comments

@james18
Copy link

james18 commented Jan 25, 2012

I understand that 0 should not be passed into any DateOffset subclasses, but:

Currently:
DataFrame.asfreq(MonthEnd(0)) goes into infinite loop. I believe this is the case for all the Date *End subclasses

Desired:
Exception should be raised
wesm added a commit that referenced this issue Jan 26, 2012
@wesm
BUG: raise exception in DateRange with MonthEnd(0) instead of infinit…
daf0c67 
…e loop, GH #683
@wesm
Copy link
Member

wesm commented Jan 26, 2012

Thanks for catching this. Fixed in the above commit, now raises an exception

@wesm wesm closed this as completed Jan 26, 2012
@james18
Copy link
Author

james18 commented Jan 26, 2012

Hey Wes,

I'm using pandas more and more at work. Read your docs and Craig helped me
with some initial questions I had. I like it, keep up the good work!

R/
Jim

On Thu, Jan 26, 2012 at 11:59 AM, Wes McKinney <
reply@reply.github.com

wrote:

Thanks for catching this. Fixed in the above commit, now raises an
exception

Reply to this email directly or view it on GitHub:
https://github.com/wesm/pandas/issues/683#issuecomment-3672200

@wesm
Copy link
Member

wesm commented Jan 26, 2012

Cool. Putting in a lot of work on revamping the time series functionality right now so will be valuable to get feedback from users like yourself

yarikoptic added a commit to neurodebian/pandas that referenced this issue Feb 10, 2012
@yarikoptic
Merge commit 'v0.7.0rc1-73-g69d5bd8' into debian
b4ff285 
* commit 'v0.7.0rc1-73-g69d5bd8': (44 commits)
  BUG: integer slices should never access label-indexing, GH pandas-dev#700
  BUG: pandas-dev#680 clean up with check for py3compat
  BUG: pandas-dev#680 rears again. cut off another hydra head
  ENH: change to tree-like MultiIndex output with > 2 levels, GH pandas-dev#689
  TST: added a test related to pandas-dev#680
  BUG: related to closes pandas-dev#691, removed cruft
  BUG: closes pandas-dev#691, assignment with ix and mixed dtypes
  BUG: handle incomparable values when creating Factor, caused bug in py3
  TST: Fixes for tests on Python 3.
  BUG: pandas-dev#680, print consistently when dataframe is empty
  TST: unit test for PR pandas-dev#684
  ENH: Allow Series.to_csv to ignore the index.
  BUG: raise exception in DateRange with MonthEnd(0) instead of infinite loop, GH pandas-dev#683
  BUG: unbox 0-dimensional arrays in map_infer, GH pandas-dev#690
  updated license and credits for overview
  ENH: cythonize timestamp conversion in HDFStore
  TST: ok, this appears to work GH pandas-dev#680
  TST: even more woes GH pandas-dev#680
  TST: unicode woes on windoze GH pandas-dev#680
  TST: unicode codec test issue, GH pandas-dev#680
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
0.7.0
Development

No branches or pull requests
2 participants
@wesm @james18