Skip to content

TST: Add test for clip-na #16369

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

Merged
merged 1 commit into from
May 16, 2017

Conversation

TomAugspurger
Copy link
Contributor

Additional test cases for #16364
when upper and / or lower is nan.

cc @jreback this line was missing tests I think.

Additional test cases for pandas-dev#16364
when upper and / or lower is nan.
@TomAugspurger TomAugspurger added Performance Memory or execution speed performance Testing pandas testing functions or related to the test suite labels May 16, 2017
@TomAugspurger TomAugspurger added this to the 0.21.0 milestone May 16, 2017
@codecov
Copy link

codecov bot commented May 16, 2017

Codecov Report

Merging #16369 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #16369      +/-   ##
==========================================
- Coverage    90.4%   90.37%   -0.04%     
==========================================
  Files         161      161              
  Lines       50933    50933              
==========================================
- Hits        46048    46031      -17     
- Misses       4885     4902      +17
Flag Coverage Δ
#multiple 88.15% <ø> (-0.02%) ⬇️
#single 40.21% <ø> (-0.11%) ⬇️
Impacted Files Coverage Δ
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/plotting/_converter.py 63.23% <0%> (-1.82%) ⬇️
pandas/core/frame.py 97.68% <0%> (-0.1%) ⬇️
pandas/core/generic.py 92.12% <0%> (+0.16%) ⬆️
pandas/core/common.py 91.03% <0%> (+0.34%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 42e2a87...b546752. Read the comment docs.

@jreback
Copy link
Contributor

jreback commented May 16, 2017

thanks @TomAugspurger

I think ok to merge this. I will push another PR for clip (to handle int/float things).

going to mark 0.20.2 for consistency.

@jreback jreback modified the milestones: 0.20.2, 0.21.0 May 16, 2017
@jreback jreback merged commit 9c8337a into pandas-dev:master May 16, 2017
pawroman added a commit to pawroman/pandas that referenced this pull request May 18, 2017
* upstream/master: (48 commits)
  BUG: Categorical comparison with unordered (pandas-dev#16339)
  ENH: Adding 'protocol' parameter to 'to_pickle'.
  PERF: improve MultiIndex get_loc performance (pandas-dev#16346)
  TST: remove pandas-datareader xfail as 0.4.0 works (pandas-dev#16374)
  TST: followup to pandas-dev#16364, catch errstate warnings (pandas-dev#16373)
  DOC: new oauth token
  TST: Add test for clip-na (pandas-dev#16369)
  ENH: Draft metadata specification doc for Apache Parquet (pandas-dev#16315)
  MAINT: Add .iml to .gitignore (pandas-dev#16368)
  BUG/API: Categorical constructor scalar categories (pandas-dev#16340)
  ENH: Provide dict object for to_dict() pandas-dev#16122 (pandas-dev#16220)
  PERF: improved clip performance (pandas-dev#16364)
  DOC: try new token for docs
  DOC: try with new secure token
  DOC: add developer section to the docs
  DEPS: Drop Python 3.4 support (pandas-dev#16303)
  DOC: remove credential helper
  DOC: force fetch on build docs
  DOC: redo dev docs access token
  DOC: add dataframe construction in merge_asof example (pandas-dev#16348)
  ...
pcluo pushed a commit to pcluo/pandas that referenced this pull request May 22, 2017
Additional test cases for pandas-dev#16364
when upper and / or lower is nan.
@TomAugspurger TomAugspurger deleted the test-clip-nan-raises branch May 27, 2017 16:36
stangirala pushed a commit to stangirala/pandas that referenced this pull request Jun 11, 2017
Additional test cases for pandas-dev#16364
when upper and / or lower is nan.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Performance Memory or execution speed performance Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants