Skip to content

DOC: resample warnings #13675

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
wants to merge 2 commits into from
Closed

Conversation

chris-b1
Copy link
Contributor

@chris-b1 chris-b1 commented Jul 16, 2016

Prevents warnings triggered by IPython by matching against '_repr' or '_ipython', and expands the resample warning message to indicate what caused the deferred object to be materialized.

@sinhrks sinhrks added Error Reporting Incorrect or improved errors from pandas Resample resample method labels Jul 16, 2016
@codecov-io
Copy link

codecov-io commented Jul 16, 2016

Current coverage is 84.38%

Merging #13675 into master will decrease coverage by 0.16%

@@             master     #13675   diff @@
==========================================
  Files           141        142     +1   
  Lines         51185      51225    +40   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          43275      43226    -49   
- Misses         7910       7999    +89   
  Partials          0          0          

Powered by Codecov. Last updated by 506520b...c1f7727

@jreback jreback added this to the 0.19.0 milestone Jul 18, 2016
@jreback
Copy link
Contributor

jreback commented Jul 18, 2016

lgtm. I would add at the very least a bug fix note. I don't think anything is warranted unless you think a user would be suprised by the change in deprecation warning.

@chris-b1
Copy link
Contributor Author

@jreback - added a doc note - not sure why travis didn't run, but that's the only change. I didn't think there needs to be any doc change on the deprecation warning, it's basically saying the same thing, just more verbose.

@jreback jreback closed this in 4c9ae94 Jul 19, 2016
@jreback
Copy link
Contributor

jreback commented Jul 19, 2016

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Error Reporting Incorrect or improved errors from pandas Resample resample method
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Resampler: IPython 5.0 triggers wrong FutureWarning DOC: better warning on chained resample(...)
4 participants