Skip to content

Fix issue #61#62

Merged
lohedges merged 1 commit intomainfrom
fix_61
Aug 8, 2025
Merged

Fix issue #61#62
lohedges merged 1 commit intomainfrom
fix_61

Conversation

@lohedges
Copy link
Contributor

@lohedges lohedges commented Aug 8, 2025

This PR closes #61 by using strict=False when loading the state dict of an existing MACE model. This allows the use of old models with a newer version of MACE, where the new state dict might contain additional keys that weren't present in the old version.

@lohedges lohedges requested a review from ElliotWChan August 8, 2025 13:59
@lohedges lohedges added the bug Something isn't working label Aug 8, 2025
@lohedges lohedges merged commit 80486ac into main Aug 8, 2025
2 of 6 checks passed
@lohedges lohedges deleted the fix_61 branch August 8, 2025 14:11
Copy link

@ElliotWChan ElliotWChan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MACEMLE model is currently broken with mace-torch 0.3.14

2 participants