Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

[WIP] Add PIMD choice of dynamical masses #42

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

OndrejMarsalek
Copy link
Collaborator

I am looking at some issues with propagation accuracy and this is useful to have. The same can be achieved by using PA-CMD and manually putting in omega_n as the target frequency, but this is more convenient.

I would like to to move the printing of the "normal mode report" somewhere nicer so that it is not printed twice - accepting suggestions.

@OndrejMarsalek
Copy link
Collaborator Author

One problem that I notice is that in test runs with 32 beads (but not with 8 or 16), LAMMPS clients crash with ERROR: Out of range atoms - cannot compute PPPM (../pppm_tip4p.cpp:107). This also happens in the master and with <normal_modes mode="pa-cmd">[0.0301987497472]</normal_modes>. I will show a test case if I can't figure it out.

As the dynamics runs at nT, the potential should also be scaled down
by n to achieve the same effect as in PIMD with temperature T and
target frequency omega_P.
@OndrejMarsalek
Copy link
Collaborator Author

@ceriottm, are you interested in having this option for easy comparison to other programs, or shall we drop it? I have no strong preference either way.

@ceriottm
Copy link
Collaborator

ceriottm commented Apr 24, 2017 via email

@OndrejMarsalek
Copy link
Collaborator Author

Yes, something like that, with no frequency specified.

I still think that Sphinx is the best solution for documentation, both written and automatically generated (input file and the code itself). I can spend some time on it in the summer and I think we really should have it for 2.0.

@mahrossi
Copy link
Collaborator

Hey @OndrejMarsalek should we resurrect this, test, wrap up and merge or is this functionality outdated?

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

Successfully merging this pull request may close these issues.

3 participants