Closed terrytangyuan closed 8 years ago
94.84%
Merging #62 into master will decrease coverage by -0.14% as of
b258164
@@ master #62 diff @@
======================================
Files 31 32 +1
Stmts 957 990 +33
Branches 0 0
Methods 0 0
======================================
+ Hit 909 939 +30
Partial 0 0
- Missed 48 51 +3
Review entire Coverage Diff as of
b258164
Powered by Codecov. Updated on successful CI builds.
Thanks for doing this! I was thinking of something with moving window average/std dev, but this is simpler and will work pretty well!
No problem. One simple test was added to assert whether there's improvement using early stopping. Example was also added.
Fixes #61
Realized that in the future we may need to re-factor doc-strings for those subclasses of TensorFlowEstimator or it's going to be a lot of adding and deleting.