Commit Graph

3550 Commits

Author SHA1 Message Date
giannozz 2deb62402a Initialization of uninitialized variables:
- hubbard_max (used in allocate_nlpot, not initialized by post-processing
  codes) set to 0 in module ldaU
- istep (wasn't set in scf case) set to 0 in module control_flags
  (why does istep run from 0 up instead of 1 ???)
- r_m set to 0 in module noncolin (make_pointlist expects a value)


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3613 c92efa57-630b-4861-b058-cf58834340f0
2006-12-18 09:56:37 +00:00
cavazzon 4230054c57 - syntax error fixed in scalar compilation
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3612 c92efa57-630b-4861-b058-cf58834340f0
2006-12-18 09:26:53 +00:00
cavazzon 3c26c23a7a - other fix in task groups, loops count when number of states
is not a multiple of the size of the group


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3611 c92efa57-630b-4861-b058-cf58834340f0
2006-12-17 23:32:38 +00:00
cavazzon eab09a43f7 - task groups, clean ups and bugs fix
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3610 c92efa57-630b-4861-b058-cf58834340f0
2006-12-17 23:11:16 +00:00
cavazzon 0f9699edbb - removed extra tokent "," found by xlf
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3609 c92efa57-630b-4861-b058-cf58834340f0
2006-12-16 17:43:43 +00:00
cavazzon 5d3881262a - Task Groups parallelization strategy for CP
generalized to all architectures


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3608 c92efa57-630b-4861-b058-cf58834340f0
2006-12-16 17:27:33 +00:00
umari 546d765916 Changement to the treatment of the Berry's phase electric
field. Now the projectors for the hermitean el. field operator
which depend on the wavefunctions,
are calculated just once and store on disk for every diagonalization

It permits much faster execution in particular for cg minimization.

P.U.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3607 c92efa57-630b-4861-b058-cf58834340f0
2006-12-15 23:18:19 +00:00
umari 8ba15ee8b6 Changements for faster treatment of berry's phase
finite electric field
P.U.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3606 c92efa57-630b-4861-b058-cf58834340f0
2006-12-15 23:05:28 +00:00
cavazzon 80bd402965 - LDA plus U added to CP 3.2,
potentials and energy is ok, on the forces there is still
  some small problems. Further clean ups required.
  Contributed by
  Patrick Sit


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3605 c92efa57-630b-4861-b058-cf58834340f0
2006-12-15 17:58:58 +00:00
giannozz 32683ebdd7 Variable "pi34" was meaning different things in different places.
This has lead to unintended side effects with the last cleanup.
For the time being, pi34 = 3*pi/4 is removed from constants.f90;
pi34 = 3/(4*pi) is left in more_functionals.f90 as local variable;
pi34 = (3/(4*pi))^(1/3) is left in functionals.f90 and
lsda_functionals.f90 as local variable


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3604 c92efa57-630b-4861-b058-cf58834340f0
2006-12-14 17:10:36 +00:00
cavazzon 149f28d775 - fixed a bug included with the last update
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3603 c92efa57-630b-4861-b058-cf58834340f0
2006-12-14 16:28:26 +00:00
giannozz 3a98e2d9d4 More patches from Axel:
- correct an unquoted string (iosys) in PW/input.f90:483
(this was the cause of tonight compilation failere)
- correct a few incorrect format strings
- make more use of the constants module and thus
  provide more consistent units. NOTE, this has some
  numerical changes in the outputs, as in some places
  rather low precision and inconsistent numbers were
  used for unit conversion.
- convert all(?) single precision constants to double
  using the attached little perl program.
  exceptions: efermi.f90 (as it is supposed to be rewritten
  anyways), plotbands.f90 (it uses single precision everywhere,
  which may result in saving a significant amount of memory,
  so i converted the two double precision constants to single).
Unused routine 'set_fft_grid' removed


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3602 c92efa57-630b-4861-b058-cf58834340f0
2006-12-14 08:53:47 +00:00
umari 52eedb4ee5 In electtric field calculation with nspin==2,
the polarization lacked a factor of 2.
P.U.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3601 c92efa57-630b-4861-b058-cf58834340f0
2006-12-13 15:07:23 +00:00
giannozz 86fd46c179 Variables for pressure are always initialized
Some cleanup of input_parameters


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3600 c92efa57-630b-4861-b058-cf58834340f0
2006-12-13 08:50:19 +00:00
giannozz 34dd9fe1ef Fix for yesterday's changes to pressure in clusters
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3599 c92efa57-630b-4861-b058-cf58834340f0
2006-12-13 07:59:14 +00:00
giannozz 87f57550d8 Check on starting magnetization should be performed for scf calculations
only, not necessarily for non-scf calculations


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3598 c92efa57-630b-4861-b058-cf58834340f0
2006-12-12 17:33:36 +00:00
giannozz 783c0d42e1 Variables read in module pres_ai_mod have been added to input_parameters:
the data is read into input_parameters and copied to module pres_ai_mod,
like for all the other input variables. This allows a better separation
between reading and initialization phases, removes many dependencies upon
upon pres_ai_mod. Module pres_ai_mod moved to CPV where it belongs.
Makefiles and dependency files updated. UNTESTED: it may or may not work..


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3597 c92efa57-630b-4861-b058-cf58834340f0
2006-12-12 15:35:11 +00:00
giannozz 19aaac89fa More miscellanous cleanup from Axel:
- disable locales via use of LC_ALL=C in all shell scripts
   (which will hopefully make sorted lists more consistent
    between people running with different locales).
 - remove redundant files from PW, NMR_new, VdW, VIB;
   rename subroutines if their functionality is noI
   unchanged from the original source it was taken from.
 - corresponding updates to Makefiles and dependencies.
 - whitespace cleanups (eleminate remaining tabs).
 - a few more float->DBLE


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3596 c92efa57-630b-4861-b058-cf58834340f0
2006-12-12 11:02:09 +00:00
giannozz 166ec98487 Restart with US PP and electric fields should be fixed now
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3595 c92efa57-630b-4861-b058-cf58834340f0
2006-12-11 16:52:18 +00:00
giannozz 50889b7e3d Yet another bug in phonon restart: it wasn't working if the run was
stopped after convergence of a scf loop and before the first iteration
of the next scf loop. The workaround is not smart (the scf cycle is
restarted from the beginning) but it seems to work. A better workaround
requires a complete restructuring of the restart procedure.

There is still a mysterious problem with US PP: restarting during the
electric field calculation produces wrong results for Z* and phonons.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3594 c92efa57-630b-4861-b058-cf58834340f0
2006-12-11 15:14:02 +00:00
sbraccia 7432487fe6 Dependencies updated. C.S.
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3593 c92efa57-630b-4861-b058-cf58834340f0
2006-12-11 14:55:42 +00:00
sbraccia f4171e694c Few improvments of image parallelization algorithm for NEB and SMD (both potential
energy and free-energy calculations): now it works for local file-systems also.
Variables names conflicting with  fortran keywords (target) have been changed.
Clean-up of meta-dynamics code.
C.S.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3592 c92efa57-630b-4861-b058-cf58834340f0
2006-12-11 14:51:54 +00:00
giannozz 9e294c022d More cleanup from Axel:
- replace variables that have the names of keywords:
  IF (this should make the code non-compilable!),
  INT, SCALE.
- CALL getenv -> CALL get_env  (which is the only place to call
  getenv(). BTW, newer fortran standards now recommend to use
  CALL get_environment_variable, so it might be needed to have
  only one platform dependend place)
- makefile cleanup in VIB (more consistent with subdirs, not yet
  perfect).
- whitespace fix (avoid tabs!).


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3591 c92efa57-630b-4861-b058-cf58834340f0
2006-12-11 10:19:53 +00:00
dalcorso 8234ba2542 An old version of the compute_ppsi.f90 routine was committed.
Bug fix in the parallel case.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3590 c92efa57-630b-4861-b058-cf58834340f0
2006-12-06 15:43:27 +00:00
dalcorso 4ae5643260 Small problem with make tar.
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3589 c92efa57-630b-4861-b058-cf58834340f0
2006-12-06 13:49:03 +00:00
giannozz 9e86b75f67 No reason to have in the same routine both nsp and ntyp pointing to nsp
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3588 c92efa57-630b-4861-b058-cf58834340f0
2006-12-06 13:13:53 +00:00
dalcorso 84d83c2309 Added a routine that writes on file the matrix elements of the p operator.
Compute_qdipol.f90 moved to the PW directory.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3587 c92efa57-630b-4861-b058-cf58834340f0
2006-12-06 10:39:16 +00:00
giannozz d4da64ec0c More cleanup from Axel:
- workaround for gfortran (it is sort of working)
- single precision constants promoted to double precision
- dfloat-> DBLE
- DIMAG ->AIMAG
- DCMPLX->CMPLX
Note that ALL calls to dfloat, dimag, dcmplx have been added recently,
since I removed all of them no more than a few months ago


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3586 c92efa57-630b-4861-b058-cf58834340f0
2006-12-02 17:58:50 +00:00
giannozz 1e235af88b More patches by Axel: all occurrences of "index" replaced by "idx".
"index" is an instrinsic and shouldn't be used as a variable.
control-M characters removed from qexml.f90


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3585 c92efa57-630b-4861-b058-cf58834340f0
2006-12-01 14:41:55 +00:00
giannozz 6c2b6abe1c Axel's patch for times - should hopefully work on all machines
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3584 c92efa57-630b-4861-b058-cf58834340f0
2006-11-29 09:05:35 +00:00
giannozz 6bc54e352d Some more cleanup of the noncolinear case
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3583 c92efa57-630b-4861-b058-cf58834340f0
2006-11-29 08:40:05 +00:00
giannozz ffb671c18b wfcatom and swfcatom have two indices also in the noncollinear case,
in analogy to evc.

Apparently swfcatom_nc was used but not allocated in noncollinear
calculations with LDA+U (not sure it was implemented, though)

There is still something I don't understand in wfcinit: the starting
wavefunctions needs to be set to 0 for n>npw at the end of the routine,
while it should be sufficient to clean them at the beginning. This might
be a sign that we are overwriting a zone that should not be overwritten.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3582 c92efa57-630b-4861-b058-cf58834340f0
2006-11-28 17:25:00 +00:00
giannozz fa33e60827 Forgot to remove iceil.o from the makefile
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3581 c92efa57-630b-4861-b058-cf58834340f0
2006-11-28 14:05:09 +00:00
giannozz 49bbd2debf More cleanup from Axel
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3580 c92efa57-630b-4861-b058-cf58834340f0
2006-11-28 08:20:23 +00:00
giannozz 051e53fa77 Unused routine deleted
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3579 c92efa57-630b-4861-b058-cf58834340f0
2006-11-28 08:18:28 +00:00
sbraccia cd13fca1f7 Estimator for the efficiency of the parallel diagonalizer updated to the
new version of parallel Davidson (it checks the efficiency of the generalized
eigensolver). C.S.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3577 c92efa57-630b-4861-b058-cf58834340f0
2006-11-27 22:26:55 +00:00
giannozz 84610c754e Obsolete file removed
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3576 c92efa57-630b-4861-b058-cf58834340f0
2006-11-27 17:00:48 +00:00
sbraccia 1941a5ceaa Fixed a nasty bug in the algorithm for parallel inversion of a triangular matrix.
This was probably responsible for the crashes of the parallel version of Davidson.
C.S.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3575 c92efa57-630b-4861-b058-cf58834340f0
2006-11-27 16:04:25 +00:00
giannozz 6e55c387ee Axel's patch: removal of unused variables, replacement of variable "index"
with a less dangerous "idx" ("index" is an intrinsic: some compilers may
complain, or even do strange things)


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3574 c92efa57-630b-4861-b058-cf58834340f0
2006-11-27 15:06:51 +00:00
giannozz dc6d1c35e6 Removed multiple prints in parallel execution (Giovanni Cantele)
Removed unused variables (Axel)


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3573 c92efa57-630b-4861-b058-cf58834340f0
2006-11-27 14:42:47 +00:00
giannozz 914e901c3d Wavefunctions in pwscf are stored also for the noncollinear case
as the usual two-index array evc(2*npwx,nbnd), no longer as a
different three-index array evc_nc(npwx,2,nbnd).

I am not really sure that this way is better than the previous
one: sometimes it is more convenient, sometimes it is less, but
I think that having the same array makes easier the transition
to a code that keeps everything in memory. Apologies to anybody
preferring three indices: anyway the physical memory layout
hasn't changed, so the transition from three-index to two-index
arrays shouldn't be a major problem


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3572 c92efa57-630b-4861-b058-cf58834340f0
2006-11-27 12:47:25 +00:00
dalcorso 72ebd925a1 Added two flags, lrpa and lnoloc to print the dielectric constant
calculated with DV_xc=0 (RPA), or with DV_H=0 and DV_xc=0.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3571 c92efa57-630b-4861-b058-cf58834340f0
2006-11-27 08:58:23 +00:00
sbraccia db435eee07 Fixed another bug related to a missing communicator. C.S.
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3570 c92efa57-630b-4861-b058-cf58834340f0
2006-11-27 00:54:55 +00:00
giannozz 389fcf5872 dependencies updated
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3569 c92efa57-630b-4861-b058-cf58834340f0
2006-11-25 10:47:06 +00:00
giannozz 20771d8d9c Documentation updated
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3568 c92efa57-630b-4861-b058-cf58834340f0
2006-11-25 10:45:31 +00:00
giannozz 4f9753d5c5 "make tar" saves makefiles in VIB and ifcmods.sh that should be there
Compilation for VIB fixed.

This is the N-th and last time I fix a problem introduced by somebody
else in VIB/: next time I will disable its testing and leave it rot.


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3567 c92efa57-630b-4861-b058-cf58834340f0
2006-11-25 10:34:34 +00:00
giannozz d0035e974e Minor correction
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3566 c92efa57-630b-4861-b058-cf58834340f0
2006-11-24 11:51:42 +00:00
giannozz f866bc8bed Modified to read fhi format also with abinit header
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3565 c92efa57-630b-4861-b058-cf58834340f0
2006-11-24 11:42:04 +00:00
cavazzon 7652cc7cb0 - updates required for large simulations on Cray XD1
git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3564 c92efa57-630b-4861-b058-cf58834340f0
2006-11-23 10:55:19 +00:00
cavazzon c14512f2ac - epsilon documentation updated,
contributed by Andrea Benassi


git-svn-id: http://qeforge.qe-forge.org/svn/q-e/trunk/espresso@3563 c92efa57-630b-4861-b058-cf58834340f0
2006-11-23 10:48:06 +00:00