Skip to content
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

[Model Overview] Metrics Template support with OD-specific optional args #2002

Merged
merged 11 commits into from
Mar 20, 2023

Conversation

Advitya17
Copy link
Collaborator

@Advitya17 Advitya17 commented Mar 13, 2023

Description

This PR defines enums and template functions for object detection support in the Model Overview section (and parts of Data Characteristics view).

This PR also adds in optional args (i.e. object_detection_{true/predicted}_y to the dataset interfaces) to store the OD label representation:
image

This PR explores an alternate direction to #1995 .

image

image

image

Checklist

  • I have added screenshots above for all UI changes.
  • I have added e2e tests for all UI changes.
  • Documentation was updated if it was needed.

@Advitya17 Advitya17 requested a review from natalie-isak March 13, 2023 15:19
@Advitya17 Advitya17 changed the title Object Detection template support with OD-specific optional args [Option 2] Object Detection template support with OD-specific optional args Mar 13, 2023
@Advitya17 Advitya17 requested a review from vigupta March 14, 2023 13:35
@codecov-commenter
Copy link

codecov-commenter commented Mar 14, 2023

Codecov Report

Merging #2002 (52ab00c) into main (096f4ca) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2002   +/-   ##
=======================================
  Coverage   92.37%   92.38%           
=======================================
  Files          95       95           
  Lines        4880     4882    +2     
=======================================
+ Hits         4508     4510    +2     
  Misses        372      372           
Flag Coverage Δ
unittests 92.38% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
responsibleai/responsibleai/_interfaces.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Advitya17 Advitya17 marked this pull request as ready for review March 14, 2023 22:54
@Advitya17 Advitya17 changed the title [Option 2] Object Detection template support with OD-specific optional args Model Overview template support with OD-specific optional args Mar 14, 2023
@Advitya17 Advitya17 changed the title Model Overview template support with OD-specific optional args [Model Overview] Metrics Template support with OD-specific optional args Mar 14, 2023
@@ -31,6 +31,8 @@ export interface IDataset {
feature_metadata?: IFeatureMetaData;
images?: string[];
index?: string[];
object_detection_true_y?: number[][][];
Copy link
Collaborator Author

@Advitya17 Advitya17 Mar 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. @imatiach-msft where are these python args passed to the typescript interfaces? Wish to emulate something similar to feature_metadata:

image

image

Also the changes may involve defining a new interface, parsing python list to that interface, and including that interface in unit tests so I propose doing that in the next PR to avoid overloading this one.

CC @natalie-isak

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for investigating this! My concern here is that when implementing the explainability logic, I was planning on relying on the indexes of the bounding boxes. Would that still be possible with a dictionary representation? What would be the keys of the dictionary?

1 similar comment
@imatiach-msft imatiach-msft merged commit a40e046 into main Mar 20, 2023
@imatiach-msft imatiach-msft deleted the agemawat/od_template branch March 20, 2023 20:55
gaugup pushed a commit that referenced this pull request Mar 23, 2023
…rgs (#2002)

* OD label support ckpt

* Surfaced OD metrics

* numLabels update

* lint fixes

* main fetch

* line end changes

* var fix

* ts lint fix

* lint autofixes
gaugup added a commit that referenced this pull request Mar 23, 2023
* Add validations for model predictions

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>

* Add test

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>

* Undo change

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>

* Update notebooks for categorical features to pass through FeatureMetadata (#2011)

* Update responsibleaidashboard-diabetes-decision-making.ipynb

* Update responsibleaidashboard-census-classification-model-debugging.ipynb

* Update responsibleaidashboard-census-classification-model-debugging.ipynb

* Update responsibleaidashboard-diabetes-decision-making.ipynb

* Update responsibleaidashboard-diabetes-decision-making.ipynb

* Update responsibleaidashboard-diabetes-regression-model-debugging.ipynb

* Update responsibleaidashboard-census-classification-model-debugging.ipynb

* Update responsibleaidashboard-diabetes-decision-making.ipynb

* Update responsibleaidashboard-housing-classification-model-debugging.ipynb

* Update responsibleaidashboard-housing-classification-model-debugging.ipynb

* Update responsibleaidashboard-census-classification-model-debugging.ipynb

* Update responsibleaidashboard-diabetes-decision-making.ipynb

* Update responsibleaidashboard-diabetes-regression-model-debugging.ipynb

* Update responsibleaidashboard-housing-decision-making.ipynb

* Update responsibleaidashboard-multiclass-dnn-model-debugging.ipynb

* Update responsibleaidashboard-diabetes-decision-making.ipynb

* Update responsibleaidashboard-diabetes-regression-model-debugging.ipynb

* Update responsibleaidashboard-housing-classification-model-debugging.ipynb

* Update responsibleaidashboard-housing-decision-making.ipynb

* [Model Overview] Metrics Template support with OD-specific optional args (#2002)

* OD label support ckpt

* Surfaced OD metrics

* numLabels update

* lint fixes

* main fetch

* line end changes

* var fix

* ts lint fix

* lint autofixes

* Object Detection Model Overview Flask template (#2004)

* flask endpoint infra template

* docstring and lint fixes

* lint fixes

* lint fixes

* lint autofixes

* error statement fix

* Model Overview: Object Detection Widgets for Aggregate Methods, Class selection, & IoU Threshold (#1997)

* agg method & iou thresh widgets ckpt

* class selection widget

* widget option format fix

* comment fixes

* widget size & lint fixes

* semiclon lint fix

* od widget options refactor

* lint complexity fixes

* lint complexity fixes

* comment fixes

* refactor to reduce complexity

* complexity refactor

* lint fixes

* autolint fixes

* Address code review comments

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>

---------

Signed-off-by: Gaurav Gupta <gaugup@microsoft.com>
Co-authored-by: tongy-msft <91754176+tongyu-microsoft@users.noreply.github.com>
Co-authored-by: Advitya Gemawat <agemawat@ucsd.edu>
Co-authored-by: Advitya Gemawat <agemawat@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants