-
Notifications
You must be signed in to change notification settings - Fork 15
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
Using r123 in molecular dynamics #9
Comments
I won't be able to respond for another week, but some quick questions:
|
Many thanks to your quick reply!
crng = threefry2x64(counter, key); See line 215 in https://github.com/xzhh/espressopp/blob/master/src/integrator/DPDThermostat.cpp
*Edit: I also used Philox2x64 with std::uniform_real_distribution but no any changes. |
Hi Zhen-Hao, I probably won't be able to help in a material way, but I did want to try to understand the setup a bit better: Thanks! |
Hello Michael @bergdorf Assuming there forms a triangular matrix for pair[i,j] (j>i>0) Regards, |
Hello,
We are software developers for MD simulation (https://github.com/espressopp/espressopp @junghans) and we have implemented r123 into a pair-based DPD thermostat.
In this case, we need to prepare a random number with a uniform distribution (-1/2 ~ 1/2) in calculating random force for every pair(i,j) at time t. And the counter - crng.v[0] is unique with the given {t, i, j} so that the reproducibility of data is possible when the MD restarts. Recently, we found an issue that using r123 in DPD leads to a shift-up of temperature during an NVT simulation (the system is the simplest LJ fluid). The temperature is actually NOT out of control and the shift is always around +2% ~ +3% above the target temperature (but very obvious to see after running averaging the data). Comparing with the verified boostRNG, the random numbers created by r123 do not look any strange, yielding the exact uni-distribution. Then we are unsure whether the counter-based rng introduces unexpected auto-correlation under some condition or whether there's something we missed to take into accounts in implementing r123.
Thank you for reading this and that would be nice if any ideas.
Best,
Zhen-Hao Xu, Uni-Mainz (DE)
The text was updated successfully, but these errors were encountered: