Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Google Colab FLARE+LAMMPS crashing #389

Open
francescomambretti opened this issue Jan 4, 2024 · 3 comments
Open

Google Colab FLARE+LAMMPS crashing #389

francescomambretti opened this issue Jan 4, 2024 · 3 comments

Comments

@francescomambretti
Copy link

Hi, I am running the Google Colab https://colab.research.google.com/drive/1qgGlfu1BlXQgSrnolS4c4AYeZ-2TaX5Y where it is shown the FLARE+LAMMPS working. Up to a few weeks ago, the notebook was running perfectly. This morning, I've tried to run it again and it fails in compiling LAMMPS.
In particular, the second cell code crashes while making LAMMPS with the following error:

/content/lammps/src/compute_flare_std_atom.cpp: In member function ‘virtual void LAMMPS_NS::ComputeFlareStdAtom::init()’: /content/lammps/src/compute_flare_std_atom.cpp:84:13: error: ‘class LAMMPS_NS::Neighbor’ has no member named ‘add_request’; did you mean ‘old_nrequest’? 84 | neighbor->add_request(this, NeighConst::REQ_FULL | NeighConst::REQ_OCCASIONAL); | ^~~~~~~~~~~ | old_nrequest /content/lammps/src/compute_flare_std_atom.cpp:84:43: error: ‘REQ_FULL’ is not a member of ‘LAMMPS_NS::NeighConst’ 84 | neighbor->add_request(this, NeighConst::REQ_FULL | NeighConst::REQ_OCCASIONAL); | ^~~~~~~~ /content/lammps/src/compute_flare_std_atom.cpp:84:66: error: ‘REQ_OCCASIONAL’ is not a member of ‘LAMMPS_NS::NeighConst’ 84 | neighbor->add_request(this, NeighConst::REQ_FULL | NeighConst::REQ_OCCASIONAL); | ^~~~~~~~~~~~~~

Does the code work for you? What has changed since a few weeks ago?
Thanks

Francesco

@YuuuXie
Copy link
Collaborator

YuuuXie commented Jan 5, 2024

Hi @francescomambretti , this is likely because LAMMPS itself has updated its version. You can change lammps to an older version and it is expected to work

@francescomambretti
Copy link
Author

Hi, thanks for your answer but I am not so sure it's completely right. With all the versions from 2021 it doesn't work. I have managed to make it work with the 28Mar2023 one, however, in case you want to fix the colab.

@yicao-elina
Copy link

I encounter the exact same issue with the lammps compile! @YuuuXie could you please check the versionof lammps you ask us to install in the colab? thanks a lot!!!

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

No branches or pull requests

3 participants