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

UI Improvement for Aggregate Object Detection Labels #2431

Merged
merged 3 commits into from
Nov 29, 2023

Conversation

Advitya17
Copy link
Collaborator

@Advitya17 Advitya17 commented Nov 27, 2023

Existing view:
image

Now:
image

Copilot-generated summary:
This pull request includes changes related to the vision data explorer and image list components. The most important changes include modifying the code to check for the absence of both correct and incorrect labels before selecting an image, changing the font weight of the predicted label in the image list to bold, and adding support for displaying both the correct and incorrect object detection aggregate labels in the image list component.

Main changes:

  • libs/e2e/src/lib/describer/modelAssessment/visionDataExplorer/ensureAllVisionDataExplorerViewElementsBeforeSelectionAreNotPresent.ts: Modified the code to check for the absence of both correct and incorrect labels before selecting an image.
  • libs/interpret-vision/src/lib/VisionExplanationDashboard/Controls/ImageList.styles.ts: Changed the font weight of the predicted label in the image list to bold.
  • libs/interpret-vision/src/lib/VisionExplanationDashboard/Controls/ImageList.tsx: Added support for displaying both the correct and incorrect object detection aggregate labels in the image list component. [1] [2]

Description

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.

1 similar comment
@Advitya17 Advitya17 merged commit 26e8cca into main Nov 29, 2023
94 checks passed
@Advitya17 Advitya17 deleted the agemawat/od_agg_labels branch November 29, 2023 14:44
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.

2 participants