Better error messages for incompatible models

[joanise]

PR Goal?

Two related changes to improve how we handle old and/or incompatible models.

• everyvoice checkpoint inspect now uses the embedded model_info to directly target the right class, if possible, with the same fallback cascade as before if it doesn't work
• starting the demo/app via everyvoice demo or a specific app.py script like we do on HF should produce more helpful error messages in cases of failures.

Fixes?

My hope is to reduce the troubleshooting efforts required next time we try to start a model with the wrong version of EV.

Feedback sought?

@SamuelLarkin the changes in checkpoint.py refactor your code, please review the code carefully

@deltork opinion on the error messages since you were troubleshooting the system that brought me to want to make these improvements.

Priority?

normal

Tests added?

yeah, not really. We don't have models saved in the repo accessible for CI, so none of these exception are exercised. I want to create a bank of old models created with various versions of EV for regression testing, but that's not a job for this PR.

How to test?

everyvoice inspect a variety of models and see slightly improved output, and especially no errors due to trying other kinds of models in the cases where the model_info field tells us what the right kind is.

Confidence?

high

Version change?

nah

Related PRs?

nope

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
everyvoice/demo/app.py	51% smaller
everyvoice/tests/test_cli.py	46% smaller
everyvoice/base_cli/checkpoint.py	40% smaller
everyvoice/utils/__init__.py	3% smaller
everyvoice/cli.py	0% smaller
everyvoice/config/text_config.py	0% smaller
pyproject.toml	Unsupported file format

[github-actions]

CLI load time: 0:00.27
Pull Request HEAD: 21fc92dec8781292dae4aca56c884df53df650b4
Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package
import time:      1707 |     127255 | everyvoice.cli

[codecov]

Codecov Report

❌ Patch coverage is 92.06349% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.51%. Comparing base (0efb318) to head (21fc92d).

Files with missing lines	Patch %	Lines
everyvoice/base_cli/checkpoint.py	89.58%	2 Missing and 3 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #786      +/-   ##
==========================================
+ Coverage   82.97%   83.51%   +0.54%     
==========================================
  Files          47       47              
  Lines        4158     4198      +40     
  Branches      611      614       +3     
==========================================
+ Hits         3450     3506      +56     
+ Misses        576      561      -15     
+ Partials      132      131       -1     

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

[roedoejet]

A nice solid improvement on checkpoint inspection including some sensible refactoring. Lgtm, thanks @joanise !

[roedoejet]

Is there no way to suggest a version?