-
Notifications
You must be signed in to change notification settings - Fork 395
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
OD model type support in responsibleai_vision for EA #2233
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2233 +/- ##
===========================================
- Coverage 89.17% 78.72% -10.46%
===========================================
Files 121 26 -95
Lines 6634 2190 -4444
===========================================
- Hits 5916 1724 -4192
+ Misses 718 466 -252
Flags with carried forward coverage won't be shown. Click here to find out more.
|
1 similar comment
pred_labels = [str(i) for i in pred_labels] | ||
# concatenate all predicted labels into a single string | ||
predictions_joined.append(','.join(pred_labels)) | ||
self.predictions = np.array(predictions_joined) |
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.
is there a test covering this case already?
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.
Great catch! Didn't see a dedicated test_error_analysis_manager.py
file in responsibleai_vision
, but saw a flag in the main testing file. Some weird bug in pandas it seems when I test it locally, so can work on it in the next PR.
1 similar comment
cddf833
to
319eb66
Compare
…olbox into agemawat/ea_rai_vision
This PR adds OD support in EA due to the hybrid use-case of OD models.
Description
Checklist