<div dir="ltr"><div>I don't think there is any error, just some confusion due to the rather complex implementation of parallelization over pairs of bands in exact exchange. The second dimension of arrays "psi" and "hpsi" does not matter in practice, as long as nothing goes out of bounds. Unused variables "fac" are the consequences of changes made to improve openMP parallelization.</div><div><br></div><div> I am not saying that I am happy with the current status of the exact-exchange routines. quite the opposite: I am VERY unhappy and I think that some serious cleaning is badly needed (also in conjunction with the forthcoming inclusion of the new code with localization)</div><div><br></div><div>Paolo<br></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 14, 2018 at 4:26 PM Lucas <<a href="mailto:lucas.baguet@upmc.fr">lucas.baguet@upmc.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Dear QE developers,<br>
<br>
I have a question concerning a routine in the "exx" library.<br>
<br>
In "vexx_k", we want to apply the vexx potential to "m" vectors ("m" <br>
being an input parameter). However, the arrays "psi" and "hpsi" are <br>
declared with "max_ibands" as second dimension. "max_ibands" is not used <br>
elsewhere in the routine, but "m" is. Note also that in "vexx_gamma" the <br>
second dimension of "psi" and "hpsi" is "m".<br>
<br>
Is there some reason for keeping "max_ibands" in that case?<br>
<br>
Also, in both "vexx_k" and "vexx_gamma", a local array "fac" is declared <br>
and allocated but does not seem to be used.<br>
<br>
Sincerely,<br>
Lucas Baguet<br>
<br>
PS : I noticed that point while working on the ACFDT code with Maria <br>
Hellgren and Lorenzo Paulatto at the IMPMC, Sorbonne Université, Paris.<br>
_______________________________________________<br>
developers mailing list<br>
<a href="mailto:developers@lists.quantum-espresso.org" target="_blank">developers@lists.quantum-espresso.org</a><br>
<a href="https://lists.quantum-espresso.org/mailman/listinfo/developers" rel="noreferrer" target="_blank">https://lists.quantum-espresso.org/mailman/listinfo/developers</a><br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Paolo Giannozzi, Dip. Scienze Matematiche Informatiche e Fisiche,<br>Univ. Udine, via delle Scienze 208, 33100 Udine, Italy<br>Phone +39-0432-558216, fax +39-0432-558222<br><br></div></div></div></div></div>