Skip to content

Conversation

@Luthaf
Copy link
Member

@Luthaf Luthaf commented Oct 6, 2025

Will be required for metatensor/metatrain#725

Contributor (creator of pull-request) checklist

  • Tests updated (for new features and bugfixes)?
  • Documentation updated (for new features)?
  • Issue referenced (for PRs that solve an issue)?

Reviewer checklist

  • CHANGELOG updated with public API or any other important changes?

@Luthaf Luthaf requested a review from frostedoyster October 6, 2025 09:43
return libs_list[0]
deps_path += list(glob.glob(os.path.join(libs_dir, "libdeepmd_op_cuda.so")))
deps_path += list(glob.glob(os.path.join(libs_dir, "libdeepmd_dyn_cudart.so")))
# there is also a dependency on libmpi, but it is not distributed in the wheel
Copy link
Member Author

Choose a reason for hiding this comment

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

@frostedoyster I'm a bit worried about this, especially when trying to use a deepmd model with lammps. I'm not sure what happens if both are using a different MPI library.

Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps we can mention, in the metatrain docs, the fact that LAMMPS should be used with the Python version that they distribute, if you want to use DeepMD-metatrain with LAMMPS

Copy link
Member Author

Choose a reason for hiding this comment

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

but the lammps they distribute does not have pair_style metatomic

@Luthaf Luthaf force-pushed the deepmd-deps branch 2 times, most recently from 80a155c to 13baaa8 Compare October 6, 2025 14:01
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