-
Notifications
You must be signed in to change notification settings - Fork 183
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
[MLOps 1.5] Expand the built-ins: CV #862
Conversation
What does a configuration of actually using these look like? If I wanted to use a classes thing and an image thing I'd have to specify something like: pipeline_outputs.image:
- std_per_channel
pipeline_outputs.classes:
- count_num_classes ? |
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, would be good to see some frontend examples to see how usable these all are
|
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.
looks great overall, few comments
Co-authored-by: Benjamin Fineran <[email protected]>
Co-authored-by: Benjamin Fineran <[email protected]>
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!
Feature description
detected_classes, number_detected_objects, mean_score_per_detection, std_score_per_detection
.float/int
or a dictionary (so that in the future we can easily extract the information from the built-in functions and send them to loggers viaFunctionLogger
).Coverage:
As specified in the PRD: