Skip to content
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

Improve analysis (rerun, on the fly output) #196

Open
rc83 opened this issue Oct 3, 2019 · 1 comment
Open

Improve analysis (rerun, on the fly output) #196

rc83 opened this issue Oct 3, 2019 · 1 comment

Comments

@rc83
Copy link
Collaborator

rc83 commented Oct 3, 2019

Faunus do – beside running MC simulations – also analysis of data. It would be beneficial to perform the same analysis on an existing trajectory without actually running the simulation. It shall be considered that several properties may change during the simulation, e.g., an atomic charge, and not all formats can handle them, e.g., xtc format stores only atomic coordinates, pqr format stores coordinates, charges and radii.

The analysis is performed in parallel to the simulation. However only summary values are usually written at the end. It can be reasonable to write out the analysis result progressively. Either values for every (n-th) step if desirable or as an average over a block of n steps. The output can be written to separate files or in various formats, e.g., csv, hdf5, depending on the nature of data.

@mlund mlund added this to the future release milestone Nov 5, 2019
mlund added a commit that referenced this issue Dec 15, 2019
* Move file writing from destructor to _to_disk() throughout Analysis
namespace. Save to disk every macro-step. Minor check for atom array
when reading molecule types

* Add templated inv_sqrt() function and test; clarify manual

* Add ideal cluster test and update cluster documentation which did not
show properly on readthedocs

* Allow control of expensive debug check in virtual volume move and minor
code cleanup

* Fix json compilation error introduced in previous commit

Related to #196
@mlund
Copy link
Owner

mlund commented Dec 20, 2019

mdtraj uses an HDF5 trajectory format that I suspect could be expanded with charges:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants