You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While trying to update the building from source guide, I tried building my own copy of LAMMPS and ran into a number of issues:
The use of the ptr argument to lammps_open is deprecated; the function now returns a pointer.
Unfortunately this makes it unclear how to fix lammps_open, which we need to define ourselves to avoid incompatibilities with how rsmpi defines MPI_Comm.
Thankfully, the current signature void(*) (int, **char, MPI_Comm, **void) is loosely ABI-compatible with the old signature void (int, **char, MPI_Comm, **void) in most modern calling conventions.
...unfortunately I'm getting a segfault when I try to test with this.
Virtually any update related to any of this is going to require adding CI Testing on a matrix of MPI/no-MPI, system/build, and a small number of "tested" lammps versions, because it's beyond the point where I have the bandwidth to test all of this.
The text was updated successfully, but these errors were encountered:
While trying to update the building from source guide, I tried building my own copy of LAMMPS and ran into a number of issues:
ptr
argument tolammps_open
is deprecated; the function now returns a pointer.lammps_open
, which we need to define ourselves to avoid incompatibilities with how rsmpi defines MPI_Comm.void(*) (int, **char, MPI_Comm, **void)
is loosely ABI-compatible with the old signaturevoid (int, **char, MPI_Comm, **void)
in most modern calling conventions.mpi-sys
The text was updated successfully, but these errors were encountered: