Skip to content

Conversation

@HaoZeke
Copy link
Member

@HaoZeke HaoZeke commented Dec 4, 2025

The nuclear option. @pfebrer we should merge this one in today if #195 can't be resolved soon (but @Luthaf is taking a look so let's hold off until the end of the day).

@HaoZeke
Copy link
Member Author

HaoZeke commented Dec 4, 2025

obviously the intent is to revert this later, so:

git revert 9ee3c09483c17dd62673c1fea9a322526bca4bba
# make real fixes here
git commit . -m "feat(mtaplumed): fixed annoying cross version bug"
git push
# request review & profit

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

Here is the build version of the cookbook in this pull request: documentation.zip, you can view it locally by unzipping documentation.zip and open the index.html with your favorite browser.

@Luthaf
Copy link
Contributor

Luthaf commented Dec 4, 2025

ok, I think we can merge this for now, I don't understand what's wrong with the example. It looks like some kind of repeatable memory corruption, making plumed try to execute the model with per_atom=True, even when loading the model from Python or C++ the capabilities are properly set with per_atom = false.

@pfebrer pfebrer merged commit 23725e8 into lab-cosmo:main Dec 4, 2025
4 checks passed
@HaoZeke HaoZeke deleted the disableMTAPlumed branch December 4, 2025 20:34
Luthaf added a commit that referenced this pull request Dec 12, 2025
Luthaf added a commit that referenced this pull request Dec 12, 2025
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.

3 participants