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

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.49%. Comparing base (33eae51) to head (ddc98ef).
⚠️ Report is 78 commits behind head on main.

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           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[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.