Skip to content

fix: restore MACE-MPA-0 checkpoints replaced with MP-0 in MaceUrls removal - #607

Merged
CompRhys merged 2 commits into
TorchSim:mainfrom
lil-lon:fix-low-level-tutorial-mace-mpa-0
Sep 19, 2026
Merged

CompRhys merged 2 commits into
TorchSim:mainfrom
lil-lon:fix-low-level-tutorial-mace-mpa-0

Conversation

@lil-lon

@lil-lon lil-lon commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Summary

The tutorial text says it uses MACE-MPA-0, but the code loaded mace_mp(model="medium"), which is the MACE-MP-0 medium checkpoint per mace_mp_urls. The mismatch comes from #524, which replaced MaceUrls.mace_mpa_medium with model="medium" and unintentionally changed the loaded model. Use model="medium-mpa-0" to restore the documented model.

Checklist

Before a pull request can be merged, the following items must be checked:

  • Doc strings have been added in the Google docstring format.
  • Run ruff on your code.
  • Tests have been added for any new functionality or bug fixes.

@lil-lon lil-lon changed the title Fix low_level_tutorial to load MACE-MPA-0 as documented fix: restore MACE-MPA-0 checkpoints replaced with MP-0 in MaceUrls removal Sep 19, 2026
@CompRhys

Copy link
Copy Markdown
Member

thanks!

@CompRhys
CompRhys merged commit b979b26 into TorchSim:main Sep 19, 2026
60 of 64 checks passed
@lil-lon
lil-lon deleted the fix-low-level-tutorial-mace-mpa-0 branch September 19, 2026 11:35
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.

2 participants