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

A builder for classical md workflows #1010

Closed
wants to merge 134 commits into from

Conversation

orionarcher
Copy link
Contributor

@orionarcher orionarcher commented May 7, 2024

This PR implements ElectrolyteBuilder and SolventBenchmarkBuilder classes for analysis of trajectories generated by atomate2.openmm.

The core Builder classes are supported by a variety of utility functions that interact with MDAnalysis, SolvationAnalysis, and TransportAnalysis. In addition to the usual get_items, process_items, update_targets workflow, users can also manually create an MDAnalysis.Universe directly from the taskdoc with ElectrolyteBuilder.instantiate_universe. This makes manual analysis of a small number of systems much easier.

Everything is reasonably well tested and once CI passes this will be ready for review.

…mport to __init__.py to keep imports the same.
@orionarcher
Copy link
Contributor Author

orionarcher commented Aug 18, 2024

@tschaume This is ready to merge!

Note that I manually bumped transport analysis to version 0.1.2.

@orionarcher
Copy link
Contributor Author

@tsmathis or @tschaume would you mind merging this?

@orionarcher
Copy link
Contributor Author

orionarcher commented Aug 30, 2024

Type errors are coming from the electronic structure code, I think all tests are passing otherwise.

@orionarcher
Copy link
Contributor Author

This PR has been superceded by PR #1087, which includes the changes to emmet-core, and PR #1088, which includes the emmet builders. The commit history has also been cleaned up significantly.

Closing.

@orionarcher orionarcher closed this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants