-
Notifications
You must be signed in to change notification settings - Fork 6
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
RDF example notebook missing information #54
Comments
Thanks for bringing this to our attention @RainierBarrett , the wording there is confusing and I suspect this example was not properly updated since the old freud 1.x days. Since freud v2.x, accumulating an RDF over multiple frames of trajectory data is controlled through the Hope this clears everything up, if you have any more questions feel free to ask! |
We'll update this example for the freud v3 release, since |
Great! Thanks :) |
We'll keep the issue open for tracking purposes. I'll close it for good after we update the example at some point in the future. |
In the module_intros/density.RDF-AccumulateFluid.ipynb notebook, there is the following line:
However, the example in the cell below calls the same function for plotting RDFs defined in a previous cell, which does not feature any
accumulate
call. Is there a missing definition here, or just a change in how thecompute
method behaves?As-is, it's a little confusing how to properly use this method going off of this doc, or how/whether
accumulate
should be used.The text was updated successfully, but these errors were encountered: