-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Regression test for ground-truth jobs from video #9150
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
Regression test for ground-truth jobs from video #9150
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #9150 +/- ##
========================================
Coverage 73.32% 73.32%
========================================
Files 448 448
Lines 45862 45862
Branches 3914 3914
========================================
+ Hits 33627 33628 +1
+ Misses 12235 12234 -1
|
|
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
This provides coverage for fix #9095
Motivation and context
How has this been tested?
frameCount
)feature test
ground_truth_jobs.js
already contains regression tests, so this test updates that. A separatedescribe
block is added with separate hooks to generate a gt-task from videoChecklist
develop
branchLicense
Feel free to contact the maintainers if that's a concern.