-
Notifications
You must be signed in to change notification settings - Fork 647
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
MDAnalysis.analysis.pca
: Implement parallelization or mark as unparallelizable
#4680
Comments
PCA is not easily parallelizable as far as I know so I will just mark it as not parallelizable for now. |
orbeckst
added a commit
that referenced
this issue
Aug 23, 2024
- fix #4680 - PCA explicitly marked as not parallelizable (at least not with simple split-apply-combine)
5 tasks
orbeckst
added a commit
that referenced
this issue
Aug 26, 2024
- fix #4680 - PCA explicitly marked as not parallelizable (at least not with simple split-apply-combine) - add tests - update CHANGELOG
yuxuanzhuang
pushed a commit
to yuxuanzhuang/mdanalysis
that referenced
this issue
Sep 9, 2024
- fix MDAnalysis#4680 - PCA explicitly marked as not parallelizable (at least not with simple split-apply-combine) - add tests - update CHANGELOG
hmacdope
pushed a commit
that referenced
this issue
Sep 10, 2024
* Fixed high dimensional GroupBase indexing. * fixed pep8 issues * Removed sanitisation * Fix #4687 -- rdkit values in azure CI (#4688) * Investigate rdkit issue * Update azure-pipelines.yml * fix numpy 2.0 import block * fix imports * mark analysis.pca.PCA as not parallelizable (#4684) - fix #4680 - PCA explicitly marked as not parallelizable (at least not with simple split-apply-combine) - add tests - update CHANGELOG * disable gsd * disable gsd in azure * reduce timeout and set logical * fix azure * restore timeout to 200 --------- Co-authored-by: Matthew Davies <128810112+MattTDavies@users.noreply.github.com> Co-authored-by: Irfan Alibay <IAlibay@users.noreply.github.com> Co-authored-by: Oliver Beckstein <orbeckst@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Given a newly implemented parallelization (https://docs.mdanalysis.org/dev/documentation_pages/analysis/parallelization.html), implement it for all classes in
analysis.pca
module. You may useanalysis.RMSD
indevelop
as a reference.You need to add:
analysis.pca
module (or explicitly limit available backends if the classes aren't parallelizable)testsuite/analysis/conftest.py
, analogous with existing onesclient_...
fixtures to all tests using intestsuite/MDAnalysisTests/analysis/test_pca.py
, and modify the way run() method is calledThe text was updated successfully, but these errors were encountered: