Add autograde module with wrapper functions for test results#93
Add autograde module with wrapper functions for test results#93lwasser merged 12 commits intoearthlab:masterfrom
Conversation
Update jlpalomino fork from earthlab master
Codecov Report
@@ Coverage Diff @@
## master #93 +/- ##
==========================================
- Coverage 59.25% 58.04% -1.21%
==========================================
Files 13 14 +1
Lines 1205 1230 +25
==========================================
Hits 714 714
- Misses 491 516 +25
Continue to review full report at Codecov.
|
lwasser
left a comment
lwasser
left a comment
There was a problem hiding this comment.
@jlpalomino please remember how we crafted docstrings for earthpy. we will follow the same standards here. it all renders in the RTD build so i'm really surprised that your build actually ran locally with the missing - (unless my eye isn't catching things right?? ping me when this is fixed.
| def run_test( | ||
| func, | ||
| points, | ||
| *args, |
There was a problem hiding this comment.
@kcranston should args come after the correct_message and error message parameters in the function definition?
or is it better to break them up as they are here?
lwasser
changed the title
lwasser
changed the title
|
ok merging this and will revisit to add tests and documentation later. |
2 participants
@lwasser @kcranston This PR adds the new autograde module containing the wrapper functions that were previously including in the notebooks that are tested with matplotcheck functions.