Skip to content

Add auto3d models test <engine> health check - #107

Merged
isayev merged 1 commit into
mainfrom
feat/models-test
Jun 13, 2026
Merged

Add auto3d models test <engine> health check#107
isayev merged 1 commit into
mainfrom
feat/models-test

Conversation

@isayev

@isayev isayev commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Picks up a deferred CLI item from the parity review: a models health-check command.

auto3d models test <engine> loads the engine and runs a single tiny (methane) forward pass, surfacing common environment problems up front instead of mid-run:

  • missing torchani for ANI engines → exit 3 (DependencyError)
  • failed/blocked aimnet registry download → exit 3/relevant code
  • broken custom model file → handled error
  • loads but emits non-finite output → exit 5 (NumericalError)

Supports --gpu/--no-gpu and --gpu-idx. Also fixes the models info unknown-engine hint to list real engine names (ModelFactory.available_models()) rather than upper-cased internal dict keys.

Docs (README, docs/source/cli.rst, CHANGELOG) and fast CLI tests included (model load mocked). Fast gate: 674 passed; ruff clean on src/.

Still deferred: wiring progress.py's live OptimizationDisplay into the run loop — that needs a dedicated design for cross-process progress reporting without disturbing the spawn/CUDA isolation, so it's intentionally not in this PR.

Loads an engine and runs one tiny (methane) forward pass to verify it works in
the current environment, surfacing the common setup problems up front: a missing
torchani for the ANI engines, a failed/blocked aimnet registry download, or a
broken custom model file. Exits 0 on success, 3 if a dependency is missing, 5 if
the model loads but produces non-finite output. Also fixes the `models info`
unknown-engine hint to list the real engine names (ModelFactory.available_models)
instead of the upper-cased internal dict keys. Docs + tests included.
@isayev
isayev merged commit 25128fc into main Jun 13, 2026
8 of 9 checks passed
@isayev
isayev deleted the feat/models-test branch June 13, 2026 14:36
isayev added a commit that referenced this pull request Jul 11, 2026
Add auto3d models test <engine> health check
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.

1 participant