-
Notifications
You must be signed in to change notification settings - Fork 668
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
Wrong rdf values from InterRDF_s when given 'density=True' #2811
Labels
Milestone
Comments
orbeckst
pushed a commit
to VOD555/mdanalysis
that referenced
this issue
Jul 10, 2020
- fix MDAnalysis#2811 (see also MDAnalysis/pmda#120) - add test to compare InterRDF and InterRDF_s - add description for density option (was not documented since 0.19.0) - updated docs for rdf - updated docs with equationi - attributes for InterRDF and InterRDF_s - reference equations - minor grammar/style fixes - remove ill-advised usage of InterRDF_s from docs - update CHANGELOG
orbeckst
added a commit
that referenced
this issue
Jul 12, 2020
* Fixes #2811 * enable hitherto undocumented kwarg density for InterRDF_s (but changed meaning of True/False): If you used density=False since 0.19.0 to calculate a single particle density then you now have to use density=True. The default remains the same: calculate the RDF and not the density. * Fix normalization for InterRDF_s(..., density=True) * add test to compare InterRDF and InterRDF_s * add description for density option * update CHANGELOG
orbeckst
pushed a commit
that referenced
this issue
Jul 12, 2020
- fix #2811 (see also MDAnalysis/pmda#120) - add test to compare InterRDF and InterRDF_s - add description for density option (was not documented since 0.19.0) - updated docs for rdf - updated docs with equationi - attributes for InterRDF and InterRDF_s - reference equations - minor grammar/style fixes - remove ill-advised usage of InterRDF_s from docs - update CHANGELOG
orbeckst
added a commit
that referenced
this issue
Jul 12, 2020
- fix (because it was not documented before) - density=True now means "calculate the single site density"; default is still "calculate RDF" (now meaning density=False) - update docs - update tests (add test for default behavior) - update CHANGELOG - backported issue #2811 fix (directly rebased onto master instead of added to PR #2798
PicoCentauri
pushed a commit
to PicoCentauri/mdanalysis
that referenced
this issue
Mar 30, 2021
- fix MDAnalysis#2811 (see also MDAnalysis/pmda#120) - add test to compare InterRDF and InterRDF_s - add description for density option (was not documented since 0.19.0) - updated docs for rdf - updated docs with equationi - attributes for InterRDF and InterRDF_s - reference equations - minor grammar/style fixes - remove ill-advised usage of InterRDF_s from docs - update CHANGELOG
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Expected behavior
If we set
density=True
, the results fromrdf.InterRDF_s
should be the same asrdf.InterRDF
when given a single-atom to single-atom pair.See also MDAnalysis/pmda#120
Actual behavior
The results from
rdf.InterRDF_s
will be 1/N ofrdf.InterRDF
.Where N = nA * nB, and nA, nB are the number of atoms in atomgroup A and atomgroup B.
Code to reproduce the behavior
Current version of MDAnalysis
MDAnalysis1.0.0
python 3.6
Ubuntu 18.04
The text was updated successfully, but these errors were encountered: