Repository navigation
nvidia-alchemi: hardening fixes and H100 revalidation for #212 - #1
Merged
anogassis merged 1 commit intoOct 23, 2026
Merged
Conversation
Security - Bump GitPython 3.1.62, hydra-core 1.3.7, multidict 6.9.1 and urllib3 2.8.0 to clear published advisories flagged by dependency review. - Verify checkpoint size and SHA-256 before deserializing; runtime model aliases resolve only to hash-verified checkpoints baked into the image. - Make cached checkpoints read-only and reserve logs.out from artifact replacement. Behavior - Require CUDA by default; CPU runs only when explicitly requested for tests. - Propagate trajectory-sink and distributed-run failures instead of swallowing them. - Fix indentation in the embedded script template. - Mark training, multi-GPU, periodic MACE and broader cuEquivariance use as experimental in catalog descriptions. Tests - Add 11 regression tests (43 total), including hash-before-load checks for the checkpoint downloader. Validation - 43/43 tests on CPU during image build and on one H100 NVL, no skips. - MACE small-0b/medium-0b2 and AIMNet2 inference, MACE relaxation and 100-step NVT revalidated on H100 with independent numerical checks. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Owner
|
Thank you @faranm-stack . Looks good. |
This was referenced Oct 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Hardening fixes and revalidation for the
nvidia-alchemiagent in microsoft#212: dependency updates that clear the Dependency Review advisories, checkpoint verification before loading, GPU-required defaults, error propagation, 11 new regression tests, and a README verification section that reflects reproducible results.Type of change
agents/<name>/)starter-kits/<name>/)docs/schemas/**).github/**)Related issue / tracking
refs microsoft#212 — targets the PR branch
add-nvidia-alchemi-agent.Schema impact
breaking-changelabel and link the rollout plan belowValidation checklist
README.md, schemas) is updated to match the change..auto-registry/**(it is generated post-merge)..DS_Store,.env,*.swp,.idea/,.vs/, etc.).Reviewer notes (optional)
Changes (all under
agents/nvidia-alchemi/, 17 files):logs.outis protected from artifact replacement; script-template indentation fixed.Validation (on this exact code):
pip checkpassed; all three checkpoints matched their pinned hashes.release_smoke.py.small-0b, MACEmedium-0b2and AIMNet2 on water and ethanol: GPU vs CPU within 5e-6 eV (energy) and 1.4e-5 eV/Å (force); rotation-invariant energies, equivariant forces, net force < 5e-7 eV/Å. The default MACE CUDA path (cuEquivariance on) matched cuEquivariance off within 1.4e-6 eV/Å.medium-0b2FIRE relaxation of distorted water and ethanol: energy dropped 0.14 and 0.82 eV; final max force 0.017 and 0.021 eV/Å.README and CHANGELOG are not copied into the image, so the documentation update does not change the validated image contents.
Not covered: multi-GPU, training, periodic/multi-system MACE and cuEquivariance beyond single-system nonperiodic runs remain experimental and are disclosed as such. The agent and tool descriptions still call cuEquivariance experimental while the README now records the single-system result; happy to align the wording either way.