Skip to content

New module to analize water dynamics.#295

Closed
alejob wants to merge 2 commits intoMDAnalysis:developfrom
alejob:feature-watdyn
Closed

New module to analize water dynamics.#295
alejob wants to merge 2 commits intoMDAnalysis:developfrom
alejob:feature-watdyn

Conversation

@alejob
Copy link
Member

@alejob alejob commented Jun 2, 2015

This module has five new analysis methods, all focus on water dynamics:
Hydrogen Bond Lifetime (HBL), similar to an already implemented module, but focused and optimized for water
Water Orientation Relaxation (WOR)
Angular Distribution (AD)
Mean Square Displacement (MSD)
Surivival Probability (SP)

Documentation and tests has been added.

This work was developed and used in: Araya-Secchi, R., Perez-Acle, T., Kang, S. G., Huynh, T., Bernardin, A., Escalona, Y., Garate J., Martínez A., García I., Sáez J.C., & Zhou, R. (2014). Characterization of a novel water pocket inside the human Cx26 hemichannel structure. Biophysical journal, 107(3), 599-612.

Please tell me any changes or improvements deemed necessary.

This module have five new analysis methods:
Hydrogen Bond Lifetime (HBL)
Water Orientation Relaxation (WOR)
Angular Distribution (AD)
Mean Square Displacement (MSD)
Surivival Probability (SP)
@richardjgowers
Copy link
Member

Hey Alejandro

If you can merge develop into this branch and then push it again I think the automated testing should happen

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd rather have the names as "WaterOrientationalRelaxation" rather than WOR.

@alejob
Copy link
Member Author

alejob commented Jun 3, 2015

Hi Richard, I'll make the changes that you suggest and I'll push it again from my development branch to make the automatic merge.

Thanks for yours suggests.

@orbeckst
Copy link
Member

orbeckst commented Jun 9, 2015

@alejob thanks for the contribution. In addition to @richardjgowers 's comments, see my comments on the pull request.

@alejob
Copy link
Member Author

alejob commented Jun 9, 2015

Hi Oliver, thanks for the suggestions/corrections. I'm working on both feedback, yours and from @richardjgowers.

Documentation fixed and updated.
@alejob alejob closed this Jun 10, 2015
@alejob alejob deleted the feature-watdyn branch July 23, 2018 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants