-
Notifications
You must be signed in to change notification settings - Fork 211
Fix testing loop in Active Learning #879
Fix testing loop in Active Learning #879
Conversation
for more information, see https://pre-commit.ci
Codecov Report
@@ Coverage Diff @@
## master #879 +/- ##
==========================================
- Coverage 89.08% 89.01% -0.07%
==========================================
Files 230 230
Lines 12708 12719 +11
==========================================
+ Hits 11321 11322 +1
- Misses 1387 1397 +10
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM !
…ash into Dref360/test_metric
for more information, see https://pre-commit.ci
…ash into Dref360/test_metric
Hello, anything missing on this? Once this is merged, I'll update our example on BaaL and finalize the blog post. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, LGTM 😃
What does this PR do?
Test metrics were not logged correctly.
Fixes # (issue)
Before submitting
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃