Skip to content

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented Jun 21, 2018

Note the hard-coded switch between absolute and relative tolerance during checking.

Closes #21528.

Note the hard-coded switch between absolute
and relative tolerance during checking.

Closes pandas-devgh-21528.
@gfyoung gfyoung added Testing pandas testing functions or related to the test suite Docs labels Jun 21, 2018
@gfyoung gfyoung added this to the 0.24.0 milestone Jun 21, 2018
@codecov
Copy link

codecov bot commented Jun 22, 2018

Codecov Report

Merging #21580 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #21580   +/-   ##
=======================================
  Coverage    91.9%    91.9%           
=======================================
  Files         153      153           
  Lines       49549    49549           
=======================================
  Hits        45537    45537           
  Misses       4012     4012
Flag Coverage Δ
#multiple 90.3% <ø> (ø) ⬆️
#single 41.78% <ø> (ø) ⬆️
Impacted Files Coverage Δ
pandas/util/testing.py 85.06% <ø> (ø) ⬆️

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 f1ffc5f...0f4d201. Read the comment docs.

@jreback jreback merged commit ac56856 into pandas-dev:master Jun 22, 2018
@jreback
Copy link
Contributor

jreback commented Jun 22, 2018

thanks!

@gfyoung gfyoung deleted the testing-almost-equal-docs branch June 22, 2018 16:52
Sup3rGeo pushed a commit to Sup3rGeo/pandas that referenced this pull request Oct 1, 2018
Note the hard-coded switch between absolute
and relative tolerance during checking.

Closes pandas-devgh-21528.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Docs Testing pandas testing functions or related to the test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants