-
Notifications
You must be signed in to change notification settings - Fork 361
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
Exif Nested Metadata support for Image Dashboards #2542
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2542 +/- ##
==========================================
- Coverage 83.96% 78.15% -5.81%
==========================================
Files 95 36 -59
Lines 4951 2619 -2332
==========================================
- Hits 4157 2047 -2110
+ Misses 794 572 -222
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
if (Array.isArray(expectedNumberOfCohorts)) { | ||
cy.get(getChartItems(chartIdentifier)) | ||
.its("length") | ||
.should("be.gte", expectedNumberOfCohorts[0]) | ||
.and("be.lte", expectedNumberOfCohorts[1]); | ||
} else { | ||
cy.get(getChartItems(chartIdentifier)).should( | ||
"have.length", | ||
expectedNumberOfCohorts |
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.
I think these come from a settings file for each dataset (singleFeatureCohorts or multiFeatureCohorts). I think it might be better to just set these to the correct values for the vision notebooks or only do this type of very loose check for vision notebooks only, since we still want to validate this is consistent for tabular and text notebooks.
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.
@imatiach-msft these tests are unfortunately so flaky that different python versions were showing different values and hence failing at different steps. For diff versions, this gate (Exif Nested Metadata support for Image Dashboards · microsoft/responsible-ai-toolbox@34f2f52 (github.com) has failed differently across all 3 notebooks.
This pull request includes changes to the
responsibleai_vision
package, specifically to thefeature_extractors.py
andimage_reader.py
files. The changes aim to improve the extraction and processing of image metadata. The most important changes include the addition ofExifTags
andIFDRational
from thePIL
library, the creation of a new functionprocess_data
to handle different data types, and the modification of theappend_exif_features
andget_all_exif_feature_names
functions to handle nested tags in the Exif data.Improvements to image metadata extraction and processing:
responsibleai_vision/responsibleai_vision/utils/feature_extractors.py
: ImportedExifTags
andIFDRational
from thePIL
library to handle additional metadata types. Created a new functionprocess_data
to handle different data types in the Exif data. Modified theappend_exif_features
function to handle nested tags in the Exif data. [1] [2]responsibleai_vision/responsibleai_vision/utils/image_reader.py
: ImportedExifTags
from thePIL
library and created a dictionaryIFD_CODE_LOOKUP
to map IFD block tags. Modified theget_all_exif_feature_names
function to handle nested tags in the Exif data. [1] [2] [3]Description
Ask from 2 customers to expand exif metadata support by including nested types.
Checklist