Skip to content

MPI support on the AMBER-QUICK API interface to run parallel free energy calculation#453

Draft
lizhen62017 wants to merge 10 commits intomerzlab:masterfrom
lizhen62017:master
Draft

MPI support on the AMBER-QUICK API interface to run parallel free energy calculation#453
lizhen62017 wants to merge 10 commits intomerzlab:masterfrom
lizhen62017:master

Conversation

@lizhen62017
Copy link

Hi Kurt,

I have implemented a local patch to resolve this issue. #392

Sadly, it does not compile well on the standalone version of QUICK. Thank you for taking a look at it.

@agoetz
Copy link
Collaborator

agoetz commented Mar 4, 2026

@lizhen62017 thanks for submitting this draft PR.

You accidentally added your builddir, can you please remove it?

Please also add a comment with details for this PR, explaining in more detail what is the objective and how you addressed it. Thanks.

Adding @ohearnk to the discussion.

@agoetz agoetz requested review from agoetz and ohearnk March 4, 2026 23:17
@ohearnk
Copy link
Collaborator

ohearnk commented Mar 9, 2026

Hi @lizhen62017, thanks for working on this.

I'd like to get this tested and merged before the upcoming Amber release. In order to do so, could you update this PR to only include the MPI-related changes? Specifically, please remove any local changes for other work (e.g., print statements related to PySCF / LUCJ).

Moreover, it looks like many changes included in the current HEAD commit of the master branch have been reverted in this PR (DL-FIND updates, other bugfixes, etc..). If you could update these files, that would be appreciated.

If you can do this by this week, then we can likely get this into the yearly Amber release. Let me know if you have any questions.

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