Skip to content
This repository has been archived by the owner on Aug 2, 2019. It is now read-only.

Package request: atomistica #1

Open
tdaff opened this issue Jul 7, 2017 · 4 comments
Open

Package request: atomistica #1

tdaff opened this issue Jul 7, 2017 · 4 comments

Comments

@tdaff
Copy link
Contributor

tdaff commented Jul 7, 2017

https://github.com/Atomistica/atomistica

@tdaff
Copy link
Contributor Author

tdaff commented Sep 12, 2017

@jameskermode Thanks for adding this. I've not used Atomisitca at all, but would there be any benefit to adding the LAMMPS interface too?

@jameskermode
Copy link
Member

Sorry for slow response - yes, would be good to have both LAMMPS and ASE bindings.

@tdaff
Copy link
Contributor Author

tdaff commented Oct 19, 2017

Hmm.

Atomistica and QUIP both use the same MPI_Context.f90 module and I'm not able to link both to LAMMPS at the same time as it is complaining about multiple definitions of all the functions in that module. Has anyone managed to compile LAMMPS linked with both libraries? @pastewka @jameskermode

@pastewka
Copy link

I have run into the same problem. A small part of libAtoms is used by Atomistica and this leads to clashes. The simplest solution would be to rename the modules in Atomistica. The clean solution would be to use the same MPI_Context etc. in both, but I don't see how that can be done without a lot of manual work from whoever compiles the code.

Will implement the first option.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants