-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[Enhance] Support Webcam inference in topdown/bottomup demo. #2244
Conversation
Do you think it is necessary to ensure that the interfaces of |
That's necessary. But how about leaving this kind of stuff as the community assignment? |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## dev-1.x #2244 +/- ##
===========================================
+ Coverage 82.25% 82.29% +0.03%
===========================================
Files 228 230 +2
Lines 13387 13552 +165
Branches 2268 2301 +33
===========================================
+ Hits 11011 11152 +141
- Misses 1862 1871 +9
- Partials 514 529 +15
Flags with carried forward coverage won't be shown. Click here to find out more. see 22 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Sure, no problem |
Motivation
Modification
BC-breaking (Optional)
Use cases (Optional)
Checklist
Before PR:
After PR: