bjmorgan / kinisi

A Python package for estimating diffusion properties from molecular dynamics simulations.
https://kinisi.readthedocs.io
MIT License
55 stars 12 forks source link

Require an argument for the start of the diffusive regime. #29

Closed arm61 closed 1 year ago

arm61 commented 1 year ago

This is a breaking change, hence the increase in minor version number.

This will mean that the diffusion method will take a required argument that is the start of the diffusive regime in the simulation (where the analysis of the diffusion relation will begin).

Therefore, if you identify diffusion as starting at 4 ps (see notebooks for examples), the following is required:

diff.diffusion(4)

Any feedback is welcome.