Skip to content

Inferred freq bug #1669

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
changhiskhan opened this issue Jul 24, 2012 · 4 comments
Closed

Inferred freq bug #1669

changhiskhan opened this issue Jul 24, 2012 · 4 comments
Assignees
Labels
Bug Datetime Datetime data dtype
Milestone

Comments

@changhiskhan
Copy link
Contributor

In [13]: tsWithGaps.index[3]
Out[13]: <Timestamp: 2001-01-06 00:00:00>

In [14]: tsWithGaps.index[3].dayofweek
Out[14]: 5

In [15]: tsWithGaps.index.inferred_freq
Out[15]: 'B'

@lesteve
Copy link
Contributor

lesteve commented Jul 24, 2012

Not sure what happened to the return value of tsWithGaps.index[3] in your snippet but I get this:

In [15]: tsWithGaps.index[3]
Out[15]: <Timestamp: 2001-01-06 00:00:00>

which is what you expect. Not sure how the freq='B' comes into play since I never mentioned anything about a business day frequency but I am guessing that's the bug.

@changhiskhan
Copy link
Contributor Author

In order to plot with the nice scikits.timeseries style tick labeling, the index is converted from Timestamp to Periods prior to calling Matplotlib

On Jul 24, 2012, at 11:15 AM, lesteve wrote:

Not sure what happened to the return value of tsWithGaps.index[3] in your snippet but I get this:

In [15]: tsWithGaps.index[3]
Out[15]: <Timestamp: 2001-01-06 00:00:00>

which is what you expect. Not sure how the freq='B' comes into play since I never mentioned anything about a business day frequency but I am guessing that's the bug.


Reply to this email directly or view it on GitHub:
#1669 (comment)

@changhiskhan
Copy link
Contributor Author

Sorry, I'm out of it today. This is a false alarm. The issue is not Period constructor. It's in the inferred frequency.
I'll put in a fix

@changhiskhan
Copy link
Contributor Author

Too much of a performance hit to check in infer_freq
Just check when plotting

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Datetime Datetime data dtype
Projects
None yet
Development

No branches or pull requests

2 participants