two (very) minor issues

Post here questions linked with issue while running the EPW code

Moderator: stiwari

Post Reply
hlee
Posts: 415
Joined: Thu Aug 03, 2017 12:24 pm
Affiliation: The University of Texas at Austin

two (very) minor issues

Post by hlee »

Dear developers:

I would like to report two very minor issues as follows:

(1) When iverbosity=2 and nbndfs>10, the cube files with the band index greater than 10 are overwritten due to the formatted output with "i1".
You can find these cases in the files of eliashberg_setup.f90 and eliashberg_write.f90.

(2) Currently, the subroutine "set_ndnmbr" doesn't consider the case using the number of cores greater than 1000. Is there a specific reason to stick to the case using the number of cores small than 1000? I can easily modify the code, but I am just wondering whether there is a part in the code which limits the number of cores.

Sincerely,

Hyungjun Lee

sponce
Site Admin
Posts: 616
Joined: Wed Jan 13, 2016 7:25 pm
Affiliation: EPFL

Re: two (very) minor issues

Post by sponce »

Dear Hyungjun Lee,

Thank you for reporting those two small bugs.

1) Indeed. I modified this and commit it (see Github trunk).

2) This was for historical reasons. I raised it to 99,999 cores (also committed).
Note however that I would not recommend to use much more than 1000 cores at the moment. See benchmark: http://epw.org.uk/Main/Benchmarks

Best,
Samuel
Prof. Samuel Poncé
Chercheur qualifié F.R.S.-FNRS / Professeur UCLouvain
Institute of Condensed Matter and Nanosciences
UCLouvain, Belgium
Web: https://www.samuelponce.com

hlee
Posts: 415
Joined: Thu Aug 03, 2017 12:24 pm
Affiliation: The University of Texas at Austin

Re: two (very) minor issues

Post by hlee »

Dear Samuel Ponc:

Thank you for your reply and for giving me useful advice for scalability in EPW.

Sincerely,

Hyungjun Lee

Post Reply