Skip to content

Absolute tolerance for assert_bin_values#272

Merged
lwasser merged 7 commits intoearthlab:masterfrom
nkorinek:absolute_tolerance
May 7, 2020
Merged

Absolute tolerance for assert_bin_values#272
lwasser merged 7 commits intoearthlab:masterfrom
nkorinek:absolute_tolerance

Conversation

@nkorinek
Copy link
Contributor

@nkorinek nkorinek commented May 7, 2020

Hey @lwasser I added in changes to make the docs run on @ryla5068 's PR. Hopefully this is good to merge after this!

addresses #157

@nkorinek
Copy link
Contributor Author

nkorinek commented May 7, 2020

Sorry to open up a new PR. Made it easier for me to add the changes in.

@codecov
Copy link

codecov bot commented May 7, 2020

Codecov Report

Merging #272 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #272      +/-   ##
==========================================
+ Coverage   79.95%   79.96%   +0.01%     
==========================================
  Files          19       19              
  Lines        1831     1832       +1     
==========================================
+ Hits         1464     1465       +1     
  Misses        367      367              
Impacted Files Coverage Δ
matplotcheck/base.py 87.70% <100.00%> (ø)
matplotcheck/tests/test_base_data.py 100.00% <100.00%> (ø)

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 eab6e72...3d3fb65. Read the comment docs.

@lwasser
Copy link

lwasser commented May 7, 2020

i've made a few spelling corrections @nkorinek this can be merged once CI is happy again. THANK YOU!!

@nkorinek
Copy link
Contributor Author

nkorinek commented May 7, 2020

@lwasser passed!

@lwasser
Copy link

lwasser commented May 7, 2020

yay merging. thanks @nkorinek !!

@lwasser lwasser merged commit 5fb148c into earthlab:master May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants