-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add van Hove function tutorials #13
Conversation
@MBartkowiakSTFC I'm unable to push to my branch for some reason I think you need to update the rules.
|
61f0ff3
to
7d0b975
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The examples work on my computer. The tutorial is good.
Please have a look at the changes I made to the text. If you agree with them, you are welcome to merge the changes.
Looks good thanks for that. |
Added tutorial 2 which goes through some background on the van Hove function and a step-by-step guide on calculating the van Hove function with MDANSE using a liquid argon trajectory.
Clean up some more files that were missed in #12.
Updated the conversion script in tutorial 1 so that it works with the new chunking change. Adjusted the tutorial 1 figures so that they have a smaller width and are centered to enhance readability.
@MBartkowiakSTFC feel free to make any typo fixes/changes etc.