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.

Sign up for GitHub

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

Jump to bottom

Difficult to read explanations in UI #1774

Closed
saraesteveznewtral opened this issue Oct 18, 2022 · 1 comment · Fixed by #1815
Closed

Difficult to read explanations in UI #1774

saraesteveznewtral opened this issue Oct 18, 2022 · 1 comment · Fixed by #1815
Assignees
@leiyre
Labels
area: ui Indicates that an issue or pull request is related to the User Interface (UI) type: bug Indicates an unexpected problem or unintended behavior
Milestone
v1.1.0

Comments

@saraesteveznewtral
Copy link

When using shap to understand in which parts of the text the model its putting attention on, the following happens:

image

As the probabilities of some transformers are heavily located on the extremes (or really close to 0 or to 1), the colours on the shap representation are very dark and it's difficult to read the words inside to make an analysis. Just by changing the intensity of this colours would be enough.
@leiyre
Copy link
Member

leiyre commented Oct 25, 2022

Hi @saraesteveznewtral, there is definitely a reading and usability problem, we will improve it as soon as possible. thank you for reporting

@Amelie-V Amelie-V added area: ui Indicates that an issue or pull request is related to the User Interface (UI) type: bug Indicates an unexpected problem or unintended behavior labels Oct 28, 2022
leiyre added a commit that referenced this issue Nov 2, 2022
@leiyre
fix: Improve explanation readability
3348f35 
This PR decreases the intensity of the colors for good readability of words with high values

closes #1774
@leiyre leiyre mentioned this issue Nov 2, 2022
Merged
This was linked to pull requests Nov 2, 2022
fix: Remove the extra letter "y" #1814
Merged
fix: Improve explanation readability #1815
Merged
@leiyre leiyre removed a link to a pull request Nov 2, 2022
fix: Remove the extra letter "y" #1814
Merged
@frascuchon frascuchon modified the milestones: v1.0.1, v1.0.0, v1.1.0 Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leiyre leiyre

Labels
area: ui Indicates that an issue or pull request is related to the User Interface (UI) type: bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Milestone
v1.1.0
Development

Successfully merging a pull request may close this issue.

fix: Improve explanation readability argilla-io/argilla
4 participants
@frascuchon @leiyre @Amelie-V @saraesteveznewtral