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

108 postw90 opt fix #153

Merged
merged 56 commits into from
Dec 22, 2017
Merged

108 postw90 opt fix #153

merged 56 commits into from
Dec 22, 2017

Conversation

jryates
Copy link
Member

@jryates jryates commented Dec 17, 2017

This merges in the parallel performance improvements to postw90. This closes #113

Thomas Ponweiser added 30 commits February 24, 2017 21:40
 * utility_zgemm_new: safer and more general replacement for utility_zgemm
 * utility_zgemmm: helper method for matrix-matrix-matrix products using blas
 * utility_rotate_new: replacement for utility_rotate using blas
Thomas Ponweiser and others added 26 commits March 8, 2017 17:51
Currently, Wannier90 spends the useless memory in some parts, especially in the parallel run.
For this reason, for considering systems with large number of k points and bands, it might lead to memory error.

In addition to this, there are some points in which memory is used uselessly and they are also related to the parallelisation of both core parts (wannierization and disentanglement) and post-processing parts. 
Although I have my working parallel version of Wannier90, I will make an effort to apply, if any, the difference to the already-committed version.

H. Lee
There seems to be a bug in the subroutine get_gauge_overlap_matrix in get_oper.F90.
Since the first dimension of eigval is num_bands,
in the argument of call utility_zgemmm
eigval(:,ik_a) should be eigval(wm_a:wm_a+ns_a-1,ik_a).

H. Lee
…nnier90 into 108_POSTW90OPT_FIX

Merge upstream changes
Update postw90_common.F90
…nnier90 into 108_POSTW90OPT_FIX

merge from upstream
@jryates jryates merged commit 3f93450 into develop Dec 22, 2017
@jryates jryates deleted the 108_POSTW90OPT_FIX branch December 22, 2017 14:20
manxkim pushed a commit to manxkim/wannier90 that referenced this pull request Jan 10, 2021
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.

Parallel postw90 routines
3 participants