move confusion matrix text to tooltip

[dberenbaum]

Related to https://iterativeai.slack.com/archives/C01SR9Q12LB/p1681991760838949?thread_ts=1681958116.002649&cid=C01SR9Q12LB

Before:

After:

Doesn't necessarily look better for this example, but generally I think it will work better to have the values in the tooltip for anything beyond binary classification.

[codecov-commenter]

Codecov Report

Patch and project coverage have no change.

Comparison is base (33eae51) 94.49% compared to head (ddc98ef) 94.49%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #127   +/-   ##
=======================================
  Coverage   94.49%   94.49%           
=======================================
  Files          19       19           
  Lines         726      726           
  Branches      113      113           
=======================================
  Hits          686      686           
  Misses         27       27           
  Partials       13       13           

Impacted Files	Coverage Δ
src/dvc_render/vega_templates.py	93.87% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[daavoo]

Change makes sense to me.

BUT 😅 , every change like this makes me wonder if we could come up with a better UX to customize these things.
To avoid having to be opinionated and still not requiring users to write a full new Vega template

[dberenbaum]

BUT 😅 , every change like this makes me wonder if we could come up with a better UX to customize these things.
To avoid having to be opinionated and still not requiring users to write a full new Vega template

Yup, this is the direction I was going with #109. Agree that it would be way more intuitive to customize from a UI like in VS Code rather than mess around with Vega or dvc.yaml.

In this case, however, it seemed not worth having an option to customize, and it's probably okay to be opinionated.

[dberenbaum]

Not sure if it's worth a release. I'd be fine waiting on other changes.

[shcheklein]

Not sure if it's worth a release. I'd be fine waiting on other changes.

why not, let's release ... we should be publishing things as we go :)

[dberenbaum]

@shcheklein Do you plan another PR about this like you mentioned in https://iterativeai.slack.com/archives/C01SR9Q12LB/p1682002565617949?

[shcheklein]

@dberenbaum yes! I'll try to do it today. It might touch this repo. Kk, let's wait a bit :)

[dberenbaum]

Okay, no rush. I'll set a reminder to release if I don't see anything in the next few days.