Add support for MDA minimize_vectors#188
Add support for MDA minimize_vectors#188PardhavMaradani wants to merge 5 commits intoMDAnalysis:mainfrom
Conversation
|
Hi, I am unable to figure out the build error below. Could you please help me and let me know if there is something I am missing? I was able to build this locally on my ubuntu environment, which is how I was able to run the performance tests with a working version in the referenced MDA PR #5235. Thanks |
|
Here are the benchmark results comparing against mda version (serial): Show a full benchmark results runHere are some comparative results using MDAnalysis's own |
|
I re-enabled the workflow. Try pushing something to get the tests to run. |
|
@hmacdope could you review, please? |
I pushed an empty commit. It seems like the workflow requires an approval to run. Thanks |
|
About the test failures:
|
|
The macos build failures show the following errors: These seem to be fixed in the latest (v1.9.5) of google benchmark (through their PR google/benchmark#2108). Our submodule was pointing to an older commit |
|
@PardhavMaradani I changed the repo settings for actions. Hopefully, you won't need approval for re-running CI. If it's still necessary please let me know. |
|
Updating google benchmark to v1.9.5 fixed the macos build issues. The same commit ( Capturing the relevant sections of the logs here so this is preserved: DetailsAdding a |

This PR adds support for MDAnalysis
minimize_vectorsas described in MDA Issue #5234. MDA PR #5235 adds the backend support for this function in MDAnalysis. Thanks