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

Categorical Exif Metadata support for Vision Dashboard #2348

Merged
merged 15 commits into from
Sep 25, 2023
Merged

Conversation

Advitya17
Copy link
Collaborator

@Advitya17 Advitya17 commented Sep 21, 2023

This PR adds support for categorical image metadata extracted from the exif of the image, propagated on both the Flyout and Model Overview Feature Cohort metrics.

Description

image

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.

@codecov-commenter
Copy link

codecov-commenter commented Sep 21, 2023

Codecov Report

Merging #2348 (4c2c8ba) into main (5833a88) will decrease coverage by 13.35%.
The diff coverage is 87.93%.

@@             Coverage Diff             @@
##             main    #2348       +/-   ##
===========================================
- Coverage   92.40%   79.06%   -13.35%     
===========================================
  Files         108       26       -82     
  Lines        5415     2269     -3146     
===========================================
- Hits         5004     1794     -3210     
- Misses        411      475       +64     
Flag Coverage Δ
unittests 79.06% <87.93%> (-13.35%) ⬇️

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

Files Changed Coverage Δ
..._vision/rai_vision_insights/rai_vision_insights.py 77.07% <45.45%> (ø)
...n/responsibleai_vision/utils/feature_extractors.py 93.87% <96.15%> (ø)
..._vision/responsibleai_vision/utils/image_reader.py 77.94% <100.00%> (ø)

... and 131 files with indirect coverage changes

2 similar comments
with Image.open(image_open_pointer) as im:
if image_mode is not None:
im = im.convert(image_mode)
image_array = asarray(im)
return image_array


def get_all_exif_feature_names(image_dataset):
Copy link
Contributor

Choose a reason for hiding this comment

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

can you add python tests for these new methods/changes?

@Advitya17 Advitya17 merged commit 057cbac into main Sep 25, 2023
116 checks passed
@Advitya17 Advitya17 deleted the agemawat/exif branch September 25, 2023 02:19
This pull request was closed.
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.

3 participants