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
Refactor #38
base: master
Are you sure you want to change the base?
Refactor #38
Conversation
Codecov Report
@@ Coverage Diff @@
## master #38 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 3 4 +1
Lines 552 506 -46
=====================================
- Hits 552 506 -46
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #38 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 3 4 +1
Lines 552 506 -46
=====================================
- Hits 552 506 -46
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #38 +/- ##
===========================================
- Coverage 100% 80.35% -19.65%
===========================================
Files 3 4 +1
Lines 552 667 +115
===========================================
- Hits 552 536 -16
- Misses 0 131 +131
Continue to review full report at Codecov.
|
@rafaelslins let us merge only when all changes and new plots are implemented, and everything has been thoroughly tested. For example, the current version has an error for the error bounds:
|
Ok, Carlos |
@rafaelslins for this bug to not happen again, you can save the bounds of the plot and return invisibly. Then you can write a simple test that checks whether the lower bounds of the plot have the correct values, are not NA etc. |
No description provided.