| 1 | | 2006-02-16 23:22 appel |
| 2 | | |
| 3 | | * AUTHORS: Adding current e-mail addresses. |
| 4 | | |
| 5 | | 2006-02-16 17:23 marques |
| 6 | | |
| 7 | | * AUTHORS: updated list |
| 8 | | |
| 9 | | 2006-02-16 16:50 marques |
| 10 | | |
| 11 | | * external_libs/: expokit/COPYING, expokit/Copyright, |
| 12 | | metis-4.0/COPYING, metis-4.0/Copyright: Changed filenames from |
| 13 | | Copyright COPYING |
| 14 | | |
| 15 | | 2006-02-16 16:49 marques |
| 16 | | |
| 17 | | * configure.ac, liboct/Makefile.am, liboct/oct_f.c, |
| 18 | | build/acx_metis.m4, src/Makefile.am, |
| 19 | | external_libs/metis-4.0/Makefile.am: *) Metis is now only |
| 20 | | compiled when needed *) changed version number to 2.0 *) octopus |
| 21 | | fails a couple of tests in my machine. I will take a look before |
| 22 | | committing |
| 23 | | |
| 24 | | 2006-02-16 16:02 appel |
| 25 | | |
| 26 | | * liboct/: Makefile.am, oct_f.c: The header file metis.h was still |
| 27 | | included as <metis/metis.h>. Now we use the one that we ship in |
| 28 | | external_libs. |
| 29 | | |
| 30 | | 2006-02-16 12:37 appel |
| 31 | | |
| 32 | | * debian/changelog: Switching from 1.99devel to 2.0 for the |
| 33 | | upcoming release. |
| 34 | | |
| 35 | | 2006-02-16 12:36 appel |
| 36 | | |
| 37 | | * configure.ac, build/acx_metis.m4: I forgot to treat the |
| 38 | | HAVE_METIS properly, when I added the library to external_libs. |
| 39 | | Now HAVE_METIS is enabled by default, but can be disabled with |
| 40 | | --disable-metis at configure time. |
| 41 | | |
| 42 | | 2006-02-13 16:58 acastro |
| 43 | | |
| 44 | | * src/spectrum.F90: Bug fixed. The units were not properly handled |
| 45 | | in the cross_section_tensor subroutine. |
| 46 | | |
| 47 | | 2006-02-12 23:13 acastro |
| 48 | | |
| 49 | | * testsuite/rc_bugs/12-spin: Bug added. Sorry, but I had to add |
| 50 | | another bug, I think that it is important. See the explanation |
| 51 | | in the file. I will try to think about it the next days, to solve |
| 52 | | it asap. |
| 53 | | |
| 54 | | 2006-02-09 23:38 appel |
| 55 | | |
| 56 | | * src/liboct_parser.F90: o Adding the full filename to the |
| 57 | | variables error message. o Removing stdin stuff completely. |
| 58 | | |
| 59 | | 2006-02-09 23:32 appel |
| 60 | | |
| 61 | | * Makefile.am: There was a line continuation \ missing. During make |
| 62 | | install the share and testsuite directories have not been taken |
| 63 | | into account. |
| 64 | | |
| 65 | | 2006-02-09 22:49 appel |
| 66 | | |
| 67 | | * src/out.F90: The compilation failed when using --with-netcdf. The |
| 68 | | netcdf module doesn't have _m. Too much Perl :) ... |
| 69 | | |
| 70 | | 2006-02-09 21:43 appel |
| 71 | | |
| 72 | | * COPYING: Updating the top-level license file to account for |
| 73 | | Expokit and Metis as 'external libraries'. I explicitely state in |
| 74 | | the text, that both are _not_ GPL. |
| 75 | | |
| 76 | | Please have a look if you agree with the style. |
| 77 | | |
| 78 | | 2006-02-09 21:39 appel |
| 79 | | |
| 80 | | * Makefile.am, configure.ac, external_libs/Makefile.am, |
| 81 | | external_libs/expokit/Copyright, |
| 82 | | external_libs/expokit/Makefile.am, |
| 83 | | external_libs/expokit/expokit.f90, src/Makefile.am, |
| 84 | | external_libs/metis-4.0/Copyright, |
| 85 | | external_libs/metis-4.0/Makefile.am, |
| 86 | | external_libs/metis-4.0/balance.c, |
| 87 | | external_libs/metis-4.0/bucketsort.c, |
| 88 | | external_libs/metis-4.0/ccgraph.c, |
| 89 | | external_libs/metis-4.0/coarsen.c, |
| 90 | | external_libs/metis-4.0/compress.c, |
| 91 | | external_libs/metis-4.0/debug.c, external_libs/metis-4.0/defs.h, |
| 92 | | external_libs/metis-4.0/estmem.c, external_libs/metis-4.0/fm.c, |
| 93 | | external_libs/metis-4.0/fortran.c, |
| 94 | | external_libs/metis-4.0/frename.c, |
| 95 | | external_libs/metis-4.0/graph.c, |
| 96 | | external_libs/metis-4.0/initpart.c, |
| 97 | | external_libs/metis-4.0/kmetis.c, |
| 98 | | external_libs/metis-4.0/kvmetis.c, |
| 99 | | external_libs/metis-4.0/kwayfm.c, |
| 100 | | external_libs/metis-4.0/kwayrefine.c, |
| 101 | | external_libs/metis-4.0/kwayvolfm.c, |
| 102 | | external_libs/metis-4.0/kwayvolrefine.c, |
| 103 | | external_libs/metis-4.0/macros.h, |
| 104 | | external_libs/metis-4.0/match.c, |
| 105 | | external_libs/metis-4.0/mbalance.c, |
| 106 | | external_libs/metis-4.0/mbalance2.c, |
| 107 | | external_libs/metis-4.0/mcoarsen.c, |
| 108 | | external_libs/metis-4.0/memory.c, external_libs/metis-4.0/mesh.c, |
| 109 | | external_libs/metis-4.0/meshpart.c, |
| 110 | | external_libs/metis-4.0/metis.h, external_libs/metis-4.0/mfm.c, |
| 111 | | external_libs/metis-4.0/mfm2.c, |
| 112 | | external_libs/metis-4.0/mincover.c, |
| 113 | | external_libs/metis-4.0/minitpart.c, |
| 114 | | external_libs/metis-4.0/minitpart2.c, |
| 115 | | external_libs/metis-4.0/mkmetis.c, |
| 116 | | external_libs/metis-4.0/mkwayfmh.c, |
| 117 | | external_libs/metis-4.0/mkwayrefine.c, |
| 118 | | external_libs/metis-4.0/mmatch.c, external_libs/metis-4.0/mmd.c, |
| 119 | | external_libs/metis-4.0/mpmetis.c, |
| 120 | | external_libs/metis-4.0/mrefine.c, |
| 121 | | external_libs/metis-4.0/mrefine2.c, |
| 122 | | external_libs/metis-4.0/mutil.c, |
| 123 | | external_libs/metis-4.0/myqsort.c, |
| 124 | | external_libs/metis-4.0/ometis.c, |
| 125 | | external_libs/metis-4.0/parmetis.c, |
| 126 | | external_libs/metis-4.0/pmetis.c, |
| 127 | | external_libs/metis-4.0/pqueue.c, |
| 128 | | external_libs/metis-4.0/proto.h, |
| 129 | | external_libs/metis-4.0/refine.c, |
| 130 | | external_libs/metis-4.0/rename.h, |
| 131 | | external_libs/metis-4.0/separator.c, |
| 132 | | external_libs/metis-4.0/sfm.c, external_libs/metis-4.0/srefine.c, |
| 133 | | external_libs/metis-4.0/stat.c, external_libs/metis-4.0/struct.h, |
| 134 | | external_libs/metis-4.0/subdomains.c, |
| 135 | | external_libs/metis-4.0/timing.c, external_libs/metis-4.0/util.c: |
| 136 | | Both, Expokit and the Metis library are now declared as external |
| 137 | | libraries. The build system has been adapted accordingly to |
| 138 | | reflect this change. |
| 139 | | |
| 140 | | The inclusion of Metis should now finally solve the build |
| 141 | | problems, that people were encountering frequently with the MPI |
| 142 | | executable. |
| 143 | | |
| 144 | | 2006-02-09 21:34 acastro |
| 145 | | |
| 146 | | * src/: par_vec.F90, simul_box.F90: Bug fixed. Just an absolute |
| 147 | | value, but it created some awful problems. I also put an ASSERT |
| 148 | | in par_vec at a point where, if it had been before, it would have |
| 149 | | saved me quite a few hours to debug. Maybe it is useful in the |
| 150 | | future. This bug was pointed out by some Carlo Camilloni. |
| 151 | | |
| 152 | | 2006-02-09 21:08 appel |
| 153 | | |
| 154 | | * src/: atomic.F90, blas.F90, broad.F90, casida.F90, |
| 155 | | centergeom.F90, cf.F90, cf_inc.F90, cross_section.F90, |
| 156 | | crystal.F90, curv_briggs.F90, curv_gygi.F90, curv_modine.F90, |
| 157 | | curvlinear.F90, datasets.F90, derivatives.F90, |
| 158 | | derivatives_inc.F90, eigen.F90, eigen_arpack.F90, eigen_cg.F90, |
| 159 | | eigen_evolution.F90, eigen_plan.F90, eigen_trlan.F90, epot.F90, |
| 160 | | epot_inc.F90, f.F90, f_inc.F90, fftw2.F90, fftw3.F90, |
| 161 | | geom_opt.F90, geometry.F90, global.F90, grid.F90, gs.F90, h.F90, |
| 162 | | h_inc.F90, h_so.F90, hgh.F90, hs.F90, io.F90, lasers.F90, |
| 163 | | lcao.F90, liboct.F90, liboct_parser.F90, libxc.F90, |
| 164 | | linalg_adv.F90, linalg_basic.F90, linear_response.F90, |
| 165 | | linear_response_inc.F90, logrid.F90, main.F90, math.F90, |
| 166 | | mesh.F90, mesh_init.F90, mesh_lib.F90, messages.F90, mf.F90, |
| 167 | | mf_inc.F90, mix.F90, mpi.F90, mpi_debug.F90, multicomm.F90, |
| 168 | | multigrid.F90, nl_operator.F90, ode_solver.F90, |
| 169 | | ode_solver_inc.F90, opt_control.F90, out.F90, out_inc.F90, |
| 170 | | par_vec.F90, par_vec_inc.F90, pes.F90, pes_mask.F90, pes_rc.F90, |
| 171 | | phonon_spectrum.F90, phonons.F90, poisson.F90, poisson1D.F90, |
| 172 | | poisson2D.F90, poisson3D.F90, poisson_cg.F90, |
| 173 | | poisson_corrections.F90, poisson_multigrid.F90, profiling.F90, |
| 174 | | ps.F90, pulpo.F90, restart.F90, restart_inc.F90, root_solver.F90, |
| 175 | | root_solver_inc.F90, rotational_strength.F90, run.F90, |
| 176 | | scalar_mesh.F90, scf.F90, simul_box.F90, sparskit.F90, |
| 177 | | sparskit_driver.F90, sparskit_inc.F90, specie.F90, spectrum.F90, |
| 178 | | spline.F90, states.F90, states_inc.F90, states_kpoints.F90, |
| 179 | | static_pol.F90, static_pol_lr.F90, stencil_cube.F90, |
| 180 | | stencil_star.F90, stencil_starplus.F90, stencil_variational.F90, |
| 181 | | string.F90, systm.F90, td.F90, td_calc.F90, td_exp.F90, |
| 182 | | td_exp_split.F90, td_init.F90, td_rti.F90, td_write.F90, tm.F90, |
| 183 | | units.F90, unocc.F90, v_ks.F90, v_ks_inc.F90, varinfo.F90, |
| 184 | | wave_matching.F90, xc.F90, xc_KLI.F90, xc_OEP.F90, |
| 185 | | xc_OEP_SIC.F90, xc_OEP_inc.F90, xc_OEP_x.F90, xc_axc.F90, |
| 186 | | xc_functl.F90, xc_fxc.F90, xc_kxc.F90, xc_vxc.F90, |
| 187 | | xyz_adjust.F90, xyz_file.F90, xyzanim.F90: The MVAPICH |
| 188 | | implementation has serious I/O problems when one of the modules |
| 189 | | in the code is called 'system'. Besides the problem with reading |
| 190 | | from stdin, this was the main issue in Sahu's problem report. I |
| 191 | | spent more than a day to figure this out. Also thanks again to |
| 192 | | Sahu for his patience, while we were debugging this issue. |
| 193 | | |
| 194 | | As there were some thoughts on this previously, I take the |
| 195 | | opportunity to rename all modules according to |
| 196 | | |
| 197 | | sample/sample_mod -> sample_m |
| 198 | | |
| 199 | | Similarly all types have been converted according to |
| 200 | | |
| 201 | | sample_type -> sample_t |
| 202 | | |
| 203 | | This is actually Linux kernel convention :). |
| 204 | | |
| 205 | | 2006-02-09 19:13 appel |
| 206 | | |
| 207 | | * src/liboct_parser.F90: Some MPI implementations, e.g. the MVAPICH |
| 208 | | implementation for infiniband interconnects, do not forward stdin |
| 209 | | to all compute nodes. This problem was initially reported by Sahu |
| 210 | | on the octopus-users mailing list and recently showed up in a |
| 211 | | second posting. |
| 212 | | |
| 213 | | Reading the input file from stdin via e.g. |
| 214 | | |
| 215 | | octopus < Na2 |
| 216 | | |
| 217 | | is now disabled. This caused too many confusions, even in the |
| 218 | | serial case. Often people already had a file with name 'inp' in |
| 219 | | the working directory while trying to read the input via '<'. In |
| 220 | | that case they got tricked by the precedence. |
| 221 | | |
| 222 | | Octopus is now printing an error message, mentioning that reading |
| 223 | | input from stdin is deprecated. |
| 224 | | |
| 225 | | 2006-02-06 17:18 acastro |
| 226 | | |
| 227 | | * src/messages.F90: The write_warning and write_fatal were writing |
| 228 | | an awkard "WARNING: From Node x", with no information afterwards, |
| 229 | | from all the nodes. Only node zero was really printing the |
| 230 | | information. |
| 231 | | |
| 232 | | Maybe in the future we should allow for a way for all non-head |
| 233 | | nodes to write warning and fatal errors. |
| 234 | | |
| 235 | | 2006-02-05 12:33 acastro |
| 236 | | |
| 237 | | * src/: atomic.F90, hgh.F90, tm.F90: Not really a bug fix. The |
| 238 | | problem with the Gadolinium bug report ("rc_bugs/07-Atomic_Gd") |
| 239 | | is deep in the "egofv" routine, and I don't quite understand it. |
| 240 | | What I have done is: - for HGH pseudopotentials, the atomic |
| 241 | | wavefunctions are not strictly neccesary, so if they are not |
| 242 | | found, they are just not used. "egofv" does no longer kill the |
| 243 | | program, but passes back an error flag and the program emits a |
| 244 | | warning. The program can no longer do LCAO, but hopefully it |
| 245 | | could converge. I could not try it in the case of Gadolinium, |
| 246 | | because in any case the memory needed to get all the projectors |
| 247 | | is so large that it does not fit in my laptop. - for TM |
| 248 | | pseudopotentials, the wavefunctions are needed to build the |
| 249 | | projectors. So if the error is produced, the program stops. But |
| 250 | | the error message emitted now is more friendly than before. |
| 251 | | |
| 252 | | In any case, the proper solution to these issues will have to be |
| 253 | | done for next release. |
| 254 | | |
| 255 | | 2006-02-04 20:22 acastro |
| 256 | | |
| 257 | | * src/mesh_init.F90, testsuite/rc_bugs/08-TDELF_C2H4: Bug fixed. |
| 258 | | |
| 259 | | This bug was the one reported in 08-TDELF_C2H4, which I remove. |
| 260 | | The input of the file was not consistent any more, in any case. |
| 261 | | |
| 262 | | The bug showed up when trying to write a dx file. There was a |
| 263 | | seg-fault whent trying to pass the mesh to a cube, and I traced |
| 264 | | it back to be some kind of round-off error in the point that I |
| 265 | | changed in mesh_init. Probably it is a quick and dirty trick, but |
| 266 | | we should leave a deeper clarification of the mesh module for |
| 267 | | after the release. |
| 268 | | |
| 269 | | 2006-02-04 15:18 acastro |
| 270 | | |
| 271 | | * src/poisson.F90: Bug fixed. Some compilers (intel) seem to have |
| 272 | | problems manipulating big 3D arrays. |
| 273 | | |
| 274 | | 2006-02-04 02:45 acastro |
| 275 | | |
| 276 | | * src/out_inc.F90: Bug fixed. Under some circumstances, this bug |
| 277 | | stopped some parallel runs in marenostrum. I don't know why it |
| 278 | | did not show up always. |
| 279 | | |
| 280 | | 2006-02-03 19:22 acastro |
| 281 | | |
| 282 | | * src/systm.F90: The argument parallel_mask of system_init was |
| 283 | | flagged as optional. Then the internal argument parallel_mask_ |
| 284 | | was set to be equal to parallel_mask when it was present, as |
| 285 | | usual... except that then it was not used. Instead of fixing |
| 286 | | this, I noticed that system_init is only called once, and |
| 287 | | parallel_mask was passed. So no need for an optional argument |
| 288 | | there. |
| 289 | | |
| 290 | | 2006-02-03 18:27 acastro |
| 291 | | |
| 292 | | * src/run.F90: (Possible) bug fixed: The variable mode (now |
| 293 | | parallel_mask), used in the routine get_mode_parallelized (now |
| 294 | | renamed to get_parallel_mask), was not initialized to 0. Some |
| 295 | | compilers do not initialize variables to 0, so this could cause |
| 296 | | trouble. |
| 297 | | |
| 298 | | 2006-02-02 19:20 acastro |
| 299 | | |
| 300 | | * liboct_parser/: parse.c, parser_f.c: Bug fixed. * |
| 301 | | parse_block_string function (file parse.c): I think that it |
| 302 | | should return an error code whatever is read is not a string. For |
| 303 | | other case it is OK to convert, but not in this case. * |
| 304 | | parse_block_error (file parser_f.c): I am not sure about this |
| 305 | | one. I uncommented the exit function. I believe that the code |
| 306 | | should stop if an error is encountered. But whoever commented |
| 307 | | that code probably had reasons for it, so take a look, maybe I am |
| 308 | | wrong. |
| 309 | | |
| 310 | | Now the code seg-faulted if a non-string was found where a string |
| 311 | | should be found. |
| 312 | | |
| 313 | | For next release we have to improve the input error handling. |
| 314 | | |
| 315 | | 2006-01-31 12:11 acastro |
| 316 | | |
| 317 | | * libxc/gga_lb.c, testsuite/rc_bugs/05-Na2_LB94_Casida: Bug fixed. |
| 318 | | Removed the 05-Na2_LB94_Casida bug. The LB94 is still rather |
| 319 | | under suspiscion, in any case. |
| 320 | | |
| 321 | | I don't know why the bug was there, in fact, it looked that the |
| 322 | | code that I removed was there for a reason, but I could not trace |
| 323 | | it down because it was there since the first revision... |
| 324 | | |
| 325 | | 2006-01-30 16:55 acastro |
| 326 | | |
| 327 | | * doc/Makefile.am: Bug fixed. The compilation of octopus depended |
| 328 | | on the existence of emacs. I think this should not be the case, |
| 329 | | so I changed the Makefile.am in order that if emacs is not found, |
| 330 | | octopus.info is not attempted to be rebuilt. This way the |
| 331 | | compilation proceeds. |
| 332 | | |
| 333 | | 2006-01-26 13:25 appel |
| 334 | | |
| 335 | | * liboct_parser/parser_f.c: Correcting cut'n paste error that was |
| 336 | | found by Francesco Sottile. |
| 337 | | |
| 338 | | 2006-01-22 11:15 appel |
| 339 | | |
| 340 | | * src/: casida.F90, run.F90: Removing unused variables. |
| 341 | | |
| 342 | | 2006-01-22 11:15 appel |
| 343 | | |
| 344 | | * share/util/: Makefile.am, display_partitions.sh, |
| 345 | | oct-display_partitions: Renaming display_partitions.sh to |
| 346 | | oct-display_partitions and adding it to bin_SCRIPTS. |
| 347 | | |
| 348 | | 2006-01-14 07:44 acastro |
| 349 | | |
| 350 | | * src/epot.F90: The code should stop if a magnetic field is |
| 351 | | required, but the executable is the one that only has real |
| | 1 | 2007-05-07 06:59 lorenzen |
| | 2 | |
| | 3 | * : Inserted forgotten word in variable description. |
| | 4 | |
| | 5 | 2007-05-04 16:18 lorenzen |
| | 6 | |
| | 7 | * : Documented the DerivativesOrder variable. |
| | 8 | |
| | 9 | 2007-05-04 11:03 xavier |
| | 10 | |
| | 11 | * : * Made fftw mandatory for 2.1, because the code doesn't compile |
| | 12 | without it and it is required for a decent perfomance. |
| | 13 | |
| | 14 | 2007-03-23 18:35 xavier |
| | 15 | |
| | 16 | * : * Changed the files to build a Debian package in 2.1.x, now the |
| | 17 | the |
| | 18 | octopus deb can be built on a clean debian etch (or ubuntu) |
| | 19 | system |
| | 20 | using gfortran. |
| | 21 | |
| | 22 | * I had to disable the parallel binary, because the mpich |
| | 23 | available in |
| | 24 | Debian is still compiled with g77 and there are some problems |
| | 25 | compiling octopus with openmpi. |
| | 26 | |
| | 27 | * I moved the old debian/rules file to |
| | 28 | debian/rules.other_compilers . |
| | 29 | |
| | 30 | 2007-03-23 18:17 xavier |
| | 31 | |
| | 32 | * : More 2.1.x fixes: |
| | 33 | |
| | 34 | * Modified the README file to point user to the wiki. |
| | 35 | * A variable was defined twice when compiling with fftw2. |
| | 36 | |
| | 37 | 2007-03-23 15:48 xavier |
| | 38 | |
| | 39 | * : * Backport of the NP/NP_PART fix to 2.1.x |
| | 40 | |
| | 41 | 2007-03-22 18:26 lorenzen |
| | 42 | |
| | 43 | * : Revision of the RPM packaging infrastructure. The RPM spec file |
| | 44 | now takes |
| | 45 | care of building both serial and MPI executables. |
| | 46 | |
| | 47 | The rpm target of the main Makefile.am has been polished a bit to |
| | 48 | respect local build |
| | 49 | settings, so being root is not necessarily required any more. |
| | 50 | |
| | 51 | As soon as the 2.1 release will be done, I can take care of the |
| | 52 | RPMs for 32 Bit and also 64 Bit. |
| | 53 | |
| | 54 | 2007-03-12 12:35 xavier |
| | 55 | |
| | 56 | * : * Bugfix: The declaration of an argument was missing, as |
| | 57 | pointed out |
| | 58 | in ticket 77. This was fixed in trunk. |
| | 59 | |
| | 60 | The other bug is already fixed in trunk and in the 2.1.x branch. |
| | 61 | |
| | 62 | This fixes #77. |
| | 63 | |
| | 64 | 2007-02-25 21:27 acastro |
| | 65 | |
| | 66 | * : Backport of the bug corrected in trunk. However, even if arpack |
| | 67 | does |
| | 68 | not crash, it does not produce very good numbers in 2.1.x (in |
| | 69 | trunk, |
| | 70 | however, it seems to work much better...) |
| | 71 | |
| | 72 | 2007-02-07 15:12 lorenzen |
| | 73 | |
| | 74 | * : This is a backport of revisions r2647 and r2659 to |
| | 75 | branches/2.1.x. |
| | 76 | Those changes renamed the following macros of global.h: |
| | 77 | |
| | 78 | DOUBLE -> REAL_DOUBLE |
| | 79 | SINGLE -> REAL_SINGLE |
| | 80 | PRECISION -> REAL_PRECISION |
| | 81 | |
| | 82 | Again, the rationale was that -- unfortunately -- DOUBLE |
| | 83 | PRECISION is still |
| | 84 | a Fortran 90 reserved word and it is a bad idea to shadow any of |
| | 85 | those by |
| | 86 | a macro. This caused a compilation error with the mpif.h header |
| | 87 | file. With |
| | 88 | this commit I also revert the last change of Xavier in the 2.1.x |
| | 89 | branch, which |
| | 90 | circumvented the problem by undefining the macros but became |
| | 91 | superfluous by |
| | 92 | this more systematic renaming. |
| | 93 | |
| | 94 | 2007-02-06 19:06 xavier |
| | 95 | |
| | 96 | * : * Compilation fix: DOUBLE and PRECISION have to be undefined |
| | 97 | when |
| | 98 | including mpif.h. |
| | 99 | |
| | 100 | 2007-01-19 13:21 xavier |
| | 101 | |
| | 102 | * : Some fixes before 2.1.0rc2 release: |
| | 103 | |
| | 104 | * Fixed address of the pseudopotential generator in doc of the |
| | 105 | species |
| | 106 | block. |
| | 107 | * The file variables.local wasn't being added to the tarball. |
| | 108 | * Two tests used old variable names for pseudopotential species. |
| | 109 | |
| | 110 | 2006-12-13 06:48 acastro |
| | 111 | |
| | 112 | * : Backport of trunk bugfix 2623, to 2.1.x. |
| | 113 | |
| | 114 | 2006-12-09 12:18 acastro |
| | 115 | |
| | 116 | * : 2.1.x now also compiles if FFTW was not found at configure |
| | 117 | time. |
| | 118 | |
| | 119 | 2006-11-03 15:25 xavier |
| | 120 | |
| | 121 | * : * Backport of the LCAO fixes. |
| | 122 | |
| | 123 | 2006-10-27 18:23 acastro |
| | 124 | |
| | 125 | * : Backport of some bugs found in branch but also present in 2.1.x |
| | 126 | |
| | 127 | 2006-10-06 09:52 xavier |
| | 128 | |
| | 129 | * : Backport of alberto's bugfixes (changesets 2454 and 2403). |
| | 130 | |
| | 131 | 2006-10-06 09:15 athimm |
| | 132 | |
| | 133 | * : Missing netcdf detection fragement. |
| | 134 | |
| | 135 | 2006-10-06 09:08 athimm |
| | 136 | |
| | 137 | * : Remove the gui from 2.1.x. |
| | 138 | |
| | 139 | There are changes required that should change the variables' |
| | 140 | interface |
| | 141 | in octopus. Furthermore 2.1.x is release-ready other than the |
| | 142 | gui. |
| | 143 | |
| | 144 | 2006-10-06 09:06 athimm |
| | 145 | |
| | 146 | * : Backports the netcdf detection fixes from trunk, fixes #39. |
| | 147 | |
| | 148 | 2006-10-02 20:33 xavier |
| | 149 | |
| | 150 | * : This error check can be made simpler. |
| | 151 | |
| | 152 | 2006-10-02 16:37 xavier |
| | 153 | |
| | 154 | * : This is the bug that appeared during the Benasque school. An |
| | 155 | error |
| | 156 | condition was not checked. |
| | 157 | |
| | 158 | 2006-10-02 16:02 xavier |
| | 159 | |
| | 160 | * : A newline at the end of this file was missing and this caused |
| | 161 | problems |
| | 162 | with autoconf. |
| | 163 | |
| | 164 | 2006-09-13 13:58 acastro |
| | 165 | |
| | 166 | * : [Porting one bug already fixed in trunk] |
| | 167 | |
| | 168 | Bug fixed. |
| | 169 | |
| | 170 | The Portland compiler did not like very long lines even if they |
| | 171 | are |
| | 172 | just comments, and did not like to mix, in an interface block, |
| | 173 | the |
| | 174 | function or subroutine interface definitions, with the module |
| | 175 | procedure statements. I have looked up the standard, and I |
| | 176 | believe |
| | 177 | both things are legal, but in any case this way that compiler is |
| | 178 | happy |
| | 179 | and it does not harm the others compilers. |
| | 180 | |
| | 181 | 2006-08-24 20:25 appel |
| | 182 | |
| | 183 | * : A few more places where check_inp was still missing. |
| | 184 | |
| | 185 | 2006-08-24 15:07 acastro |
| | 186 | |
| | 187 | * : Bug fixed. |
| | 188 | |
| | 189 | The problem with the 11-sic.test (never converging) was the use |
| | 190 | of |
| | 191 | density mixing, instead of potential mixing. The potential mixing |
| | 192 | is |
| | 193 | necessary when using orbital dependent xc functionals. |
| | 194 | |
| | 195 | 2006-08-24 14:17 athimm |
| | 196 | |
| | 197 | * : Default to not use python. |
| | 198 | |
| | 199 | It is currently of little value and only adds to the release |
| | 200 | process |
| | 201 | (like forcing the release manager to perform gettext/varinfo.xml |
| | 202 | steps). |
| | 203 | |
| | 204 | 2006-08-24 14:14 athimm |
| | 205 | |
| | 206 | * : Make branch point to the next release (candiate). |
| | 207 | |
| | 208 | 2006-08-23 20:39 appel |
| | 209 | |
| | 210 | * : Creating branch for 2.1.x development series. |
| | 211 | |
| | 212 | 2006-08-23 20:36 appel |
| | 213 | |
| | 214 | * : Preparing the first release candidate of octopus 2.1.0. Setting |
| | 215 | version strings to 2.1.0rc1. |
| | 216 | |
| | 217 | 2006-08-23 09:39 appel |
| | 218 | |
| | 219 | * : Make the various output options public, so that they can be |
| | 220 | used in other parts of the code for calls to X(output_function). |
| | 221 | |
| | 222 | 2006-08-23 09:34 appel |
| | 223 | |
| | 224 | * : This push_sub had the wrong routine name. |
| | 225 | |
| | 226 | 2006-08-23 09:18 appel |
| | 227 | |
| | 228 | * : When using UserDefinedStates as initial states for a time |
| | 229 | propagation, octopus was always trying to read ground state |
| | 230 | files from the tmp directory, even when all initial orbitals |
| | 231 | were given as formula strings (I didn't think of this case, |
| | 232 | when I was initially adding the UserDefinedStates feature). |
| | 233 | Adding therefore a logical variable OnlyUserDefinedInitialStates |
| | 234 | (with default "no"). If true, then one can use arbitrary |
| | 235 | initial states for a time propagation (via UserDefinedStates) |
| | 236 | and octopus is not attempting to read ground state orbitals. |
| | 237 | |
| | 238 | This is e.g. useful to propagate coherent states, or gaussian |
| | 239 | wave packets. Might also be useful for excerices on time |
| | 240 | propagations in the summer school. |
| | 241 | |
| | 242 | 2006-08-23 08:58 appel |
| | 243 | |
| | 244 | * : Avoid division by zero, when scalar meshes with "zero" |
| | 245 | points are created. |
| | 246 | |
| | 247 | 2006-08-23 08:56 appel |
| | 248 | |
| | 249 | * : A !%Variable header was missing for one of the variables. |
| | 250 | |
| | 251 | 2006-08-23 08:55 appel |
| | 252 | |
| | 253 | * : A check_inp and a pop_sub were missing. |
| | 254 | |
| | 255 | 2006-08-22 10:16 xavier |
| | 256 | |
| | 257 | * : Added a new variable, LinearSolverMaxTol, that is the maximum |
| | 258 | tolerance that will be applied to the Linear Solver. |
| | 259 | |
| | 260 | 2006-08-18 12:28 xavier |
| | 261 | |
| | 262 | * : I forgot to change the filename in this test. |
| | 263 | |
| | 264 | 2006-08-17 13:36 xavier |
| | 265 | |
| | 266 | * : * Reverted from allocatables to pointers, because the code was |
| | 267 | not F90 |
| | 268 | standard. |
| | 269 | |
| | 270 | 2006-08-17 12:20 xavier |
| | 271 | |
| | 272 | * : * Iterations are counted once, even if the apply the |
| | 273 | hamiltonian twice |
| | 274 | per iteration. |
| | 275 | |
| | 276 | 2006-08-17 12:17 xavier |
| | 277 | |
| | 278 | * : * Added GPL header. |
| | 279 | |
| | 280 | 2006-08-17 11:52 xavier |
| | 281 | |
| | 282 | * : * Renamed the output file of dynamic polarizability to |
| | 283 | 'linear/alpha'. |
| | 284 | |
| | 285 | 2006-08-17 11:38 xavier |
| | 286 | |
| | 287 | * : * In the nl_operator class, the arrays were defined as |
| | 288 | pointers, i |
| | 289 | changed the definition to allocatables, this gives the compiler |
| | 290 | more |
| | 291 | room for optimization. |
| | 292 | * I had to change some code that relied on pointers to this |
| | 293 | arrays. |
| | 294 | Instead i used a pointer to the operators. |
| | 295 | |
| | 296 | It would be nice if someone can review this changes, specially |
| | 297 | for |
| | 298 | mpi, i don't know if this code is checked by the testsuite. |
| | 299 | |
| | 300 | 2006-08-17 10:17 appel |
| | 301 | |
| | 302 | * : When adding multi-dataset lables to the input files yesterday I |
| | 303 | forgot to change the directories in the *.test files that are |
| | 304 | used for the matches. This fixes the failed periodic tests of |
| | 305 | today. |
| | 306 | |
| | 307 | 2006-08-17 09:50 acastro |
| | 308 | |
| | 309 | * : Two *_inc.F90 files were missing from the noinst_HEADERS, |
| | 310 | so that the package constructed by make dist was incomplete. |
| | 311 | |
| | 312 | 2006-08-16 23:53 acastro |
| | 313 | |
| | 314 | * : The regeneration of the info file, when typing "make dist" was |
| | 315 | broken, |
| | 316 | due to some repetition of node names which apparently is not |
| | 317 | allowed. |
| | 318 | Maybe we are going to get rid of this file anyways, but anyways |
| | 319 | I fix this. |
| | 320 | |
| | 321 | The "make dist" still does not work, it breaks now in the python |
| | 322 | directory, complaining about config.rpath missing. |
| | 323 | |
| | 324 | 2006-08-16 12:57 appel |
| | 325 | |
| | 326 | * : Add routines which take care of resizing the simulation |
| | 327 | box for scattering calculations (by multiples of the asymptotic |
| | 328 | unit cell). The resizing is done such that the spacing |
| | 329 | of the "central region" matches in the end the spacing |
| | 330 | that is used for a periodic calculation of the asymptotic |
| | 331 | region. |
| | 332 | |
| | 333 | 2006-08-16 09:57 appel |
| | 334 | |
| | 335 | * : Only index 0 is available for spin unpolarized |
| | 336 | calculations. This will fix the failed periodic |
| | 337 | tests of today (that will arrive later). |
| | 338 | |
| | 339 | 2006-08-15 21:05 appel |
| | 340 | |
| | 341 | * : For spin-polarized calculations also output spin resolved bands |
| | 342 | and DOS. |
| | 343 | |
| | 344 | 2006-08-15 21:05 appel |
| | 345 | |
| | 346 | * : Add unscaled k-vectors to the output of the Wigner-Seitz cell. |
| | 347 | |
| | 348 | 2006-08-15 21:04 appel |
| | 349 | |
| | 350 | * : Add labels for multi dataset mode. |
| | 351 | |
| | 352 | 2006-08-15 21:04 appel |
| | 353 | |
| | 354 | * : Uncomment this for the moment, since Input/OutputDir is not |
| | 355 | (yet) |
| | 356 | used and we would create non-necessary directories in multi- |
| | 357 | dataset mode otherwise. |
| | 358 | |
| | 359 | 2006-08-13 21:47 appel |
| | 360 | |
| | 361 | * : Also include unscaled k-vectors in the output of the |
| | 362 | fermi energy. |
| | 363 | |
| | 364 | 2006-08-13 21:16 appel |
| | 365 | |
| | 366 | * : The scaling of kpoints in crystal_init was causing all my |
| | 367 | confusion about factors of two. Now I made my Wigner-Seitz |
| | 368 | routine compatible to crystal_init. |
| | 369 | |
| | 370 | 2006-08-13 18:17 appel |
| | 371 | |
| | 372 | * : For speed reasons Intel 9.1 is now also the default |
| | 373 | compiler for the i386 deb. |
| | 374 | |
| | 375 | 2006-08-13 10:10 appel |
| | 376 | |
| | 377 | * : When using a large number of states the record length was |
| | 378 | exceeded during output. Revert again to direct io. |
| | 379 | |
| | 380 | 2006-08-13 10:08 appel |
| | 381 | |
| | 382 | * : Reciprocal cell sizes were still not correct. |
| | 383 | |
| | 384 | 2006-08-12 20:24 appel |
| | 385 | |
| | 386 | * : o Allow to use FullWignerSeitzCell also for 1D periodic |
| | 387 | systems. |
| | 388 | |
| | 389 | o Correct some factors of two. |
| | 390 | |
| | 391 | 2006-08-12 20:24 appel |
| | 392 | |
| | 393 | * : o Add routine to compute the degeneracy matrix for scattering |
| | 394 | states. |
| | 395 | |
| | 396 | o Add option to choose between scaled and unscaled k-vectors |
| | 397 | for the output of bands. |
| | 398 | |
| | 399 | o Fix a couple of cases which were not MPI clean. |
| | 400 | |
| | 401 | 2006-08-12 20:06 appel |
| | 402 | |
| | 403 | * : Add routine to write mesh info to disk (similar to |
| | 404 | simul_box_dump). |
| | 405 | |
| | 406 | 2006-08-12 20:05 appel |
| | 407 | |
| | 408 | * : Also write total Kohn-Sham potential. |
| | 409 | |
| | 410 | 2006-08-12 17:39 acastro |
| | 411 | |
| | 412 | * : Bug fixed. |
| | 413 | |
| | 414 | The call to write_momentum should be done by all nodes, |
| | 415 | or else it stops when running in parallels. |
| | 416 | |
| | 417 | I removed the last digit of the test; I think that it is asking |
| | 418 | for too much precision, and that last digit is not obtained |
| | 419 | when running with different compilers or with different number |
| | 420 | of processors. |
| | 421 | |
| | 422 | 2006-08-12 16:20 acastro |
| | 423 | |
| | 424 | * : I disconnect the last match of this test, since the code |
| | 425 | produces different numbers depending on the compiler flags. |
| | 426 | Obvioiusly something weird is inside the algorithm. |
| | 427 | |
| | 428 | 2006-08-12 08:17 xavier |
| | 429 | |
| | 430 | * : * Renamed the "pol" calculation mode to "pol_fd" and the output |
| | 431 | file |
| | 432 | to "linear/polarizability_fd". |
| | 433 | |
| | 434 | 2006-08-12 02:10 acastro |
| | 435 | |
| | 436 | * : Bug fixed. |
| | 437 | |
| | 438 | The atomic wave functions of some atoms (like Ag with semicore) |
| | 439 | could |
| | 440 | not be calculated by the Schrodinger solver in the hgh module. |
| | 441 | This |
| | 442 | little change, for some reason, seems to solve this issue. |
| | 443 | |
| | 444 | 2006-08-12 01:24 acastro |
| | 445 | |
| | 446 | * : Bug fixed. |
| | 447 | The Ag_sc pseudopotential file had an extra line. |
| | 448 | |
| | 449 | 2006-08-10 18:45 appel |
| | 450 | |
| | 451 | * : Adding routine to calculate total and band resolved density of |
| | 452 | states. When running with periodic boundary conditions octopus |
| | 453 | now also writes the density of states (including band resolved |
| | 454 | DOS) and the fermi energy to 'static'. |
| | 455 | |
| | 456 | 2006-08-10 18:44 appel |
| | 457 | |
| | 458 | * : Adding to X(Hpsi) the possibility to compute (H-E)|psi>. |
| | 459 | This will be needed for scattering states. |
| | 460 | |
| | 461 | 2006-08-10 18:44 appel |
| | 462 | |
| | 463 | * : Forgot to update header. |
| | 464 | |
| | 465 | 2006-08-09 19:38 acastro |
| | 466 | |
| | 467 | * : After some update of the code (I could not find which one), the |
| | 468 | last |
| | 469 | number of this test stopped matching the previous value. However, |
| | 470 | I |
| | 471 | find it more likely that the number is correct now, rather than |
| | 472 | before. So I changed the value of the test. |
| | 473 | |
| | 474 | Optimal control should still be tagged as an "experimental |
| | 475 | feature, |
| | 476 | anyway. |
| | 477 | |
| | 478 | 2006-08-09 17:47 acastro |
| | 479 | |
| | 480 | * : I deactivate one of the tests that is failing, since it will |
| | 481 | take some time to figure out why, and it is not relevant at |
| | 482 | this point. |
| | 483 | |
| | 484 | 2006-08-09 12:38 xavier |
| | 485 | |
| | 486 | * : * Added a very basic printing of frequency dependent |
| | 487 | hyperpolarizability. |
| | 488 | |
| | 489 | 2006-08-09 09:34 appel |
| | 490 | |
| | 491 | * : Bug fixed. |
| | 492 | |
| | 493 | When running parallel in domains octopus was crashing for |
| | 494 | periodic boundary conditions. This happened because the |
| | 495 | mesh_index |
| | 496 | function was assigning an index 0 to several entries of opi%i |
| | 497 | in nl_operator.F90. Not only the "removed point" was missing |
| | 498 | in the interior mesh (as it should), but also the mirror point of |
| | 499 | the "removed point". All of that occured because simul_box_in_box |
| | 500 | was returning the wrong answer for the "mirror points" in the |
| | 501 | case |
| | 502 | of periodic boundary conditions. |
| | 503 | |
| | 504 | 2006-08-09 09:10 appel |
| | 505 | |
| | 506 | * : Similar to the time measurements of td steps, octopus now |
| | 507 | also prints the elapsed (wall) time of a SCF step. The same |
| | 508 | format as in the td case is used. |
| | 509 | This turned out to be useful for estimating how much time |
| | 510 | octopus spends in Hpsi operations during the scf cycle for |
| | 511 | a chosen grid size. |
| | 512 | |
| | 513 | 2006-08-08 19:53 acastro |
| | 514 | |
| | 515 | * : Bugs fixed. |
| | 516 | |
| | 517 | Related to the failure of the 04-box_oct_long.test |
| | 518 | (optimal-control). |
| | 519 | It gave run-time errors and crashed. However, now the tests keeps |
| | 520 | failing due to discrepancies in the numbers. |
| | 521 | |
| | 522 | I do not know now which numbers are correct, so I leave it as it |
| | 523 | is for the moment. |
| | 524 | |
| | 525 | 2006-08-08 16:11 acastro |
| | 526 | |
| | 527 | * : Bug fixed. |
| | 528 | |
| | 529 | The Casida test was failing, when the run-time checks are |
| | 530 | enabled, due |
| | 531 | to this very stupid bug. The bug was harmless, when the checks |
| | 532 | were |
| | 533 | not done. |
| | 534 | |
| | 535 | 2006-08-08 10:58 appel |
| | 536 | |
| | 537 | * : Add possibility to choose the full Wigner-Seitz cell of the |
| | 538 | reciprocal lattice for the sampling of the k-points. If the |
| | 539 | variable FullWignerSeitzCell is set to true no symmetry is |
| | 540 | taken into account for the choice of k-points (i.e. no |
| | 541 | Monkhorst-Pack scheme will be used). Instead the k-point |
| | 542 | sampling will range over the full Wigner-Seitz cell. Currently |
| | 543 | cubes, octahedrons and dodecahedrons are implemented. The |
| | 544 | shape of the Wigner-Seitz cell has to be selected manually by |
| | 545 | setting the variable WignerSeitzCellType. A more sophisticated |
| | 546 | automatic detection could be added later. |
| | 547 | |
| | 548 | 2006-08-06 15:58 appel |
| | 549 | |
| | 550 | * : Similar to angular momentum, octopus now also computes |
| | 551 | the expectation values of the momentum operator for the |
| | 552 | KS states at the end of a ground-state calculation. |
| | 553 | |
| | 554 | 2006-08-06 14:30 appel |
| | 555 | |
| | 556 | * : Octopus can now output also wavefunctions with index > 999, |
| | 557 | or 1024 respectively. |
| | 558 | The old bit-shift code in liboct/oct_f.c was clearly more |
| | 559 | compact than this but it was bound to work only for state |
| | 560 | indices < 32*32 (increasing val in (1 << val) beyond 31 |
| | 561 | would cause a left shift count which is larger than the width |
| | 562 | of the type). |
| | 563 | |
| | 564 | 2006-08-06 12:01 appel |
| | 565 | |
| | 566 | * : Bug fixed. The output of files in multi-dataset mode |
| | 567 | was not always ending up in the right directories. |
| | 568 | |
| | 569 | 2006-08-04 17:08 xavier |
| | 570 | |
| | 571 | * : * Bugfix: Some indexes were wrong, so trying to calculate the |
| | 572 | lr_elf resulted |
| | 573 | in segmentation fault. |
| | 574 | * Added push_sub/pop_sub to lr_calc_current. |
| | 575 | |
| | 576 | 2006-08-03 20:25 appel |
| | 577 | |
| | 578 | * : o Measure total runtime of the testsuite. This could be used |
| | 579 | to compare how fast octopus is running on a given hardware |
| | 580 | platform or for a given compiler. |
| | 581 | |
| | 582 | o Add option to the script to run testsuite without MPI runs. |
| | 583 | |
| | 584 | o Add datestamp to the mail subject of the mail-report. |
| | 585 | |
| | 586 | 2006-08-03 12:06 xavier |
| | 587 | |
| | 588 | * : * Initial version of dynamic hyperpolarizabilities. Now comes |
| | 589 | testing. |
| | 590 | |
| | 591 | 2006-08-03 09:33 xavier |
| | 592 | |
| | 593 | * : I had disabled this test by mistake. |
| | 594 | |
| | 595 | 2006-08-03 08:56 xavier |
| | 596 | |
| | 597 | * : Updated the matches of long tests to take into account the use |
| | 598 | of the |
| | 599 | RR projection. |
| | 600 | |
| | 601 | 2006-08-02 20:47 xavier |
| | 602 | |
| | 603 | * : Nothing important, only a bit of i18n ;-) |
| | 604 | |
| | 605 | 2006-08-02 15:28 appel |
| | 606 | |
| | 607 | * : No output log was appended to the mail reports of the |
| | 608 | daily testsuite. Let's see if this helps. |
| | 609 | |
| | 610 | 2006-08-02 15:26 appel |
| | 611 | |
| | 612 | * : The suffix was in the case of the MPI executables not really |
| | 613 | a suffix, rather a "inbetweenfix". Change octopus-suffix_mpi |
| | 614 | to octopus_mpi-suffix. |
| | 615 | |
| | 616 | 2006-08-02 15:16 appel |
| | 617 | |
| | 618 | * : o Adding Newton-Raphson method for matrices. Starting from a |
| | 619 | sufficiently good initial guess, this routine can be used |
| | 620 | to polish inverse matrices. In the case of Vandermonde matrices |
| | 621 | e.g. the Parker-Traub algorithm can be used to compute an |
| | 622 | initial guess. |
| | 623 | Due to the required matrix-matrix multiplication Newton-Raphson |
| | 624 | is however O(N^3). |
| | 625 | |
| | 626 | o Adding routine to estimate the quality of a inverse matrix. |
| | 627 | |
| | 628 | 2006-08-02 14:51 appel |
| | 629 | |
| | 630 | * : Adding an implementation of the Parker-Traub algorithm for the |
| | 631 | inversion of Vandermonde matrices. |
| | 632 | The algorithm inverts a Vandermonde matrix in O(N^2) operations |
| | 633 | compared to O(N^3) for methods that don't take structure into |
| | 634 | account. |
| | 635 | |
| | 636 | Refs.: |
| | 637 | F. Parker, Inverses of Vandermonde matrices, Amer. Math. |
| | 638 | Monthly, 71, 1964, p410-411 |
| | 639 | J. Traub, Associated polynomials and uniform methods for |
| | 640 | the solution of linear problems |
| | 641 | SIAM Review, 8, No. 3, 1966, p277-301 |
| | 642 | |
| | 643 | This will be needed for the method that I developed to compute |
| | 644 | scattering states. |
| | 645 | |
| | 646 | 2006-08-01 10:10 appel |
| | 647 | |
| | 648 | * : Bug fixed. The linear/excitations directory was not created |
| | 649 | in multi dataset mode because loct_mkdir was used instead of |
| | 650 | io_mkdir. However, loct_mkdir is not aware of multi datatsets. |
| | 651 | The 10-casida.test was failing because of this. |
| | 652 | |
| | 653 | 2006-08-01 08:54 xavier |
| | 654 | |
| | 655 | * : Added two new methods: wfs_are_complex and wfs_are_real. |
| | 656 | |
| | 657 | Both functions receive an states_t object and return true when |
| | 658 | wavefunctions are of the type specified in the name and false |
| | 659 | otherwise. |
| | 660 | |
| | 661 | 2006-08-01 08:50 appel |
| | 662 | |
| | 663 | * : Forgot ${exec_suffix} inside mail_report(). |
| | 664 | |
| | 665 | ... we are getting close to revitalizing the nightly builds and |
| | 666 | regression tests. |
| | 667 | |
| | 668 | 2006-08-01 08:32 appel |
| | 669 | |
| | 670 | * : Also the variable "last" was used without beeing intialized |
| | 671 | first. |
| | 672 | |
| | 673 | 2006-08-01 08:30 appel |
| | 674 | |
| | 675 | * : o Don't use compile flags as linking flags. |
| | 676 | |
| | 677 | o Remove duplicated preprocessing code. Don't delete prerequisite |
| | 678 | instead. |
| | 679 | |
| | 680 | 2006-07-31 19:03 xavier |
| | 681 | |
| | 682 | * : I forgot to put some of the lines required to create |
| | 683 | atomic_oct.f90. |
| | 684 | |
| | 685 | 2006-07-31 18:07 xavier |
| | 686 | |
| | 687 | * : * The FCDEBUGFLAGS was not used when linking. |
| | 688 | |
| | 689 | * The atomic.o file was rebuilt every time, changing the rule |
| | 690 | from |
| | 691 | "atomic.o: atomic_oct.f90" to "atomic.o: atomic.F90" solves the |
| | 692 | problem. |
| | 693 | |
| | 694 | * Using FCDEBUGFLAGS is possible to compile octopus using |
| | 695 | interfile |
| | 696 | optimizations, for example -ipa with pathf90 or -ipo with ifort. |
| | 697 | This |
| | 698 | is because with this flags, the compiler doesn't produce binary |
| | 699 | objects until the linking step, so the configure script doesn't |
| | 700 | work |
| | 701 | properly. |
| | 702 | |
| | 703 | 2006-07-31 17:57 xavier |
| | 704 | |
| | 705 | * : * More explicits bounds in vectorial operations. |
| | 706 | |
| | 707 | * The quadratic box test has undeterminisc behaviour. Leaving the |
| | 708 | default mixing instead of Mixing=0.5 "solves" the problem, but I |
| | 709 | am |
| | 710 | not 100% sure that this is the right thing to do. |
| | 711 | |
| | 712 | 2006-07-31 16:14 xavier |
| | 713 | |
| | 714 | * : * Using the Rayleigh-Ritz subspace diagonalization, the |
| | 715 | unstability of |
| | 716 | the self consistency iteration disappears, so I am enabling it by |
| | 717 | default. There is still the problem of non-determinisc behaviour, |
| | 718 | but |
| | 719 | now it takes 31 or 32 iterations for benzene and not 32, 33 or |
| | 720 | 127. I |
| | 721 | still have to check what happens with other compilers. |
| | 722 | |
| | 723 | * The results of some test changed slightly with this. So i |
| | 724 | updated |
| | 725 | them. |
| | 726 | |
| | 727 | * I put back the NP_PART size in CG eigensolvers, but i couldn't |
| | 728 | find |
| | 729 | why it should be like that. |
| | 730 | |
| | 731 | 2006-07-31 12:20 xavier |
| | 732 | |
| | 733 | * : A bit of cleaning in the CG Eigensolver: |
| | 734 | |
| | 735 | * All arrays were defined with NP_PART size, altough it's not |
| | 736 | needed. |
| | 737 | I only kept NP_PART size in arrays were the hamiltonian is |
| | 738 | applied. |
| | 739 | * Put explicit limits in vectorial operations. |
| | 740 | |
| | 741 | 2006-07-31 10:19 appel |
| | 742 | |
| | 743 | * : Besides FCFLAGS there is now a variable FCDEBUGFLAGS. I had to |
| | 744 | separate debug flags from regular flags since some files cannot |
| | 745 | be compiled with debug flags. The prime example is atomic.F90. |
| | 746 | If this file is compiled with a full set of debug flags all sorts |
| | 747 | of runtime errors occur, causing a lot of tests to fail. I guess |
| | 748 | nobody really bothers to fix the issues in the F77 spaghetti |
| | 749 | code. |
| | 750 | Therefore we exclude this file from runtime checking so that we |
| | 751 | can enable the cheks in the nightly builds *for the rest of the |
| | 752 | files*. |
| | 753 | |
| | 754 | To achive a conditional compilation with different FC flags I |
| | 755 | had to introduce in src/Makefile.am pattern based rules instead |
| | 756 | of suffix rules for the compilation |
| | 757 | |
| | 758 | 2006-07-31 09:57 xavier |
| | 759 | |
| | 760 | * : Put some explicit limits in vectorial operations. This is an |
| | 761 | (failed) attempt |
| | 762 | to solve bug 37. |
| | 763 | |
| | 764 | 2006-07-30 20:55 appel |
| | 765 | |
| | 766 | * : The variable first was being used without being defined. |
| | 767 | |
| | 768 | 2006-07-30 18:57 appel |
| | 769 | |
| | 770 | * : else clauses where missing for the case when no exec suffix |
| | 771 | is given. |
| | 772 | |
| | 773 | 2006-07-30 17:00 appel |
| | 774 | |
| | 775 | * : Seems to be smarter to sort this out by symlinks. |
| | 776 | |
| | 777 | 2006-07-30 16:38 appel |
| | 778 | |
| | 779 | * : Also need to take care of manual installs. |
| | 780 | |
| | 781 | 2006-07-30 15:55 appel |
| | 782 | |
| | 783 | * : o Add also program suffix for executables build with NAG. |
| | 784 | o Add missing LDFLAGS for MPI flavours. |
| | 785 | |
| | 786 | 2006-07-30 15:02 appel |
| | 787 | |
| | 788 | * : Add support for program suffixes like -debug. This allows to |
| | 789 | run the testsuite for different flavours of executables (e.g. |
| | 790 | with or without bound checking, etc). |
| | 791 | |
| | 792 | 2006-07-30 14:58 appel |
| | 793 | |
| | 794 | * : o On amd64 linking to the gdlib causes octopus to segfault |
| | 795 | always when |
| | 796 | statically linked executables are used (independent of the |
| | 797 | compiler). |
| | 798 | octopus doesn't even get to the point where it reads the input |
| | 799 | file. |
| | 800 | Add --disable-gdlib so that -Bstatic/-static can be used again. |
| | 801 | With |
| | 802 | commerical compilers the -static options are more important than |
| | 803 | having |
| | 804 | gdlib support, I think. |
| | 805 | |
| | 806 | Since the nightly builds were not running while I was away, this |
| | 807 | was not |
| | 808 | captured. In the end I had to do bisection in the revision |
| | 809 | numbers 2156-2310 |
| | 810 | to figure out that this problem was introduced with Miguels |
| | 811 | commit in |
| | 812 | r2185/r2186. |
| | 813 | |
| | 814 | o Adding optimized and debug executables to the package. They are |
| | 815 | build with |
| | 816 | the Intel compiler. The executables carry the program suffix |
| | 817 | -intel-9.1 and |
| | 818 | -intel-9.1-debug respectively. This is mainly used for nightly |
| | 819 | builds and |
| | 820 | regression testing. It increases the size of the packages |
| | 821 | considerably, |
| | 822 | so that we will use it only for internal purposes. |
| | 823 | |
| | 824 | o Remove outdated metis configure options. |
| | 825 | |
| | 826 | 2006-07-29 01:20 appel |
| | 827 | |
| | 828 | * : o adding extra treatment for scalar meshes with only a single |
| | 829 | point. |
| | 830 | o removing variable initialization inside the type definition |
| | 831 | of scalar_mesh_t. this is not conform with the f90 standard. |
| | 832 | |
| | 833 | 2006-07-29 01:16 appel |
| | 834 | |
| | 835 | * : o adding calling interface for transpose operators. this is |
| | 836 | required |
| | 837 | for solvers like bi-conjugate gradients or some gmres flavours. |
| | 838 | o adding verbosity flag and a bit more relaxed warning criteria. |
| | 839 | |
| | 840 | 2006-07-29 00:56 appel |
| | 841 | |
| | 842 | * : o adding emacs encoding lines also to the testsuite files. |
| | 843 | (I've chosen shell-script mode to get a reasonable syntax |
| | 844 | highlighting) |
| | 845 | o some more missing svn:keywords |
| | 846 | |
| | 847 | 2006-07-29 00:50 appel |
| | 848 | |
| | 849 | * : o add emacs encoding line |
| | 850 | o release preparation: update year entries in |
| | 851 | the header (200x -> 200x-2006) |
| | 852 | |
| | 853 | 2006-07-28 20:27 xavier |
| | 854 | |
| | 855 | * : * Now when the restart lr density is not found for a certain |
| | 856 | frequency, then octopus uses the restart lr density of the |
| | 857 | closest |
| | 858 | frequency available. |
| | 859 | |
| | 860 | 2006-07-28 20:24 appel |
| | 861 | |
| | 862 | * : Removing the nl_operator_operate_cmplx routine. Complex weights |
| | 863 | are now treated by nl_operator_operate directly if op%cmplx_op |
| | 864 | is enabled. |
| | 865 | |
| | 866 | 2006-07-28 20:14 appel |
| | 867 | |
| | 868 | * : Bug fixed. This was a copy'n paste mistake. Should be |
| | 869 | op%cmplx_op |
| | 870 | instead of op%const_w. |
| | 871 | |
| | 872 | 2006-07-28 20:12 appel |
| | 873 | |
| | 874 | * : stencil_center is a new field in nl_operator_t. The variable |
| | 875 | contains the index value of the central stencil point (i.e. the |
| | 876 | position which matches the application point of the stencil). |
| | 877 | |
| | 878 | 2006-07-28 20:09 appel |
| | 879 | |
| | 880 | * : o Fixing svn:keywords for the testsuite |
| | 881 | o Adding a bunch of missing $Id:$ |
| | 882 | |
| | 883 | 2006-07-28 20:07 xavier |
| | 884 | |
| | 885 | * : As pointed by Heiko, nspin was not correctly initialized. In |
| | 886 | this case |
| | 887 | is always 1, because information is not separated by spin |
| | 888 | components. |
| | 889 | |
| | 890 | 2006-07-28 19:50 appel |
| | 891 | |
| | 892 | * : Remove unused variables. |
| | 893 | |
| | 894 | Xavier: can you have a look at this warning of the NAG compiler |
| | 895 | |
| | 896 | Warning: em_resp.F90, line 578: Symbol NSPIN referenced but never |
| | 897 | set |
| | 898 | detected at DYNAMIC_OUTPUT_INIT@<end-of-statement> |
| | 899 | |
| | 900 | Thanks. |
| | 901 | |
| | 902 | 2006-07-28 19:49 appel |
| | 903 | |
| | 904 | * : o The NAG compiler was complaining about too long subroutine |
| | 905 | names (> 31 characters) |
| | 906 | |
| | 907 | o Adding GPL header to _inc file |
| | 908 | |
| | 909 | 2006-07-28 19:49 appel |
| | 910 | |
| | 911 | * : o Remove unused variables. |
| | 912 | o Use tabstops of 2 instead of 4. |
| | 913 | o Cleanup formatting a bit to make this code a |
| | 914 | bit more coherent with the rest. |
| | 915 | |
| | 916 | 2006-07-28 18:40 appel |
| | 917 | |
| | 918 | * : svn:keywords "Id" property was missing for these files. |
| | 919 | |
| | 920 | 2006-07-28 18:37 appel |
| | 921 | |
| | 922 | * : o Currently the generation of info files from svn sources is |
| | 923 | broken. Uncomment dh_installinfo so that deb packages can be |
| | 924 | build without info files (master source for documentation is |
| | 925 | the wiki anyways). |
| | 926 | |
| | 927 | o Build executables for x86_64 dynamically (used to revitalize |
| | 928 | the nightly builds and testuite runs). |
| | 929 | |
| | 930 | 2006-07-28 18:34 appel |
| | 931 | |
| | 932 | * : If available use svn info to extract the current revision |
| | 933 | number. |
| | 934 | In this way the find run is avoided. Gives a noticeable speedup |
| | 935 | on slow disks (e.g. Laptops). |
| | 936 | |
| | 937 | 2006-07-28 11:31 acastro |
| | 938 | |
| | 939 | * : Bug fixed. |
| | 940 | |
| | 941 | One more on the problem of passing strings from Fortran to C. |
| | 942 | |
| | 943 | 2006-07-27 23:37 xavier |
| | 944 | |
| | 945 | * : Now restart information for linear response is stored for every |
| | 946 | frequency. Only the delta density is written, this appears to be |
| | 947 | a |
| | 948 | good compromise between disk space and calculation time. |
| | 949 | |
| | 950 | 2006-07-27 22:56 acastro |
| | 951 | |
| | 952 | * : Bug fixed. |
| | 953 | |
| | 954 | One of the optimal control tests was failing because of this. |
| | 955 | |
| | 956 | 2006-07-27 21:39 acastro |
| | 957 | |
| | 958 | * : About thirty something tests were failing with the NAG compiler |
| | 959 | on the |
| | 960 | opterons. The reason for most of the bugs is removed with the |
| | 961 | change |
| | 962 | that I now commit, which has to do with the Fortran to C |
| | 963 | interface -- |
| | 964 | especially the string handling --, and with the difference |
| | 965 | between int |
| | 966 | and unsigned long int in 64 bit machines (I think). |
| | 967 | |
| | 968 | *However*: |
| | 969 | This change removes the bug, but I am very uncomfortable changing |
| | 970 | these subtleties in C, so I ask any of the C experts to take a |
| | 971 | look. |
| | 972 | |
| | 973 | *And*: |
| | 974 | Still five tests fail, due to unrelated reasons. |
| | 975 | |
| | 976 | 2006-07-26 13:39 xavier |
| | 977 | |
| | 978 | * : Corrected indentation. |
| | 979 | |
| | 980 | 2006-07-26 10:44 xavier |
| | 981 | |
| | 982 | * : * Now static polarizability is calculated by the dynamic |
| | 983 | polarizability code. The old code can be used by setting the |
| | 984 | PolStatic |
| | 985 | variable. |
| | 986 | * Updated tests. |
| | 987 | |
| | 988 | 2006-07-26 09:13 xavier |
| | 989 | |
| | 990 | * : Bigfix: An array was deallocated even when not allocated. |
| | 991 | |
| | 992 | 2006-07-25 13:26 acastro |
| | 993 | |
| | 994 | * : The construction of excited states was not possible in |
| | 995 | the one-electron case if the spin-unpolarized mode was being |
| | 996 | used. This was annoying. |
| | 997 | |
| | 998 | 2006-07-24 09:47 xavier |
| | 999 | |
| | 1000 | * : Bugfix: pop_sub() wasn't been called when using return to exit. |
| | 1001 | |
| | 1002 | 2006-07-14 10:31 xavier |
| | 1003 | |
| | 1004 | * : * The sign of the complex part of polarizabilty was uncorrect. |
| | 1005 | * Now a linear/cross_section_tensor file is created, from the |
| | 1006 | information of |
| | 1007 | linear response. The format is the same of the |
| | 1008 | cross_section_tensor file |
| | 1009 | generated by oct-cross-section, altough it is not separated by |
| | 1010 | spin. |
| | 1011 | |
| | 1012 | 2006-07-14 10:17 xavier |
| | 1013 | |
| | 1014 | * : Added the option to print the values of the coordinates over |
| | 1015 | the grid, this |
| | 1016 | is not very informative, but can be used to calculate new |
| | 1017 | functions from |
| | 1018 | other functions, for instance dipole density can be calculated as |
| | 1019 | the |
| | 1020 | density times r. |
| | 1021 | |
| | 1022 | 2006-07-11 22:05 janwer |
| | 1023 | |
| | 1024 | * : removed some debug statements, td local targets should work |
| | 1025 | |
| | 1026 | 2006-07-11 21:33 acastro |
| | 1027 | |
| | 1028 | * : Bugs fixed. |
| | 1029 | |
| | 1030 | 1. The loct_parse_expression routines were not working properly, |
| | 1031 | because the conversion from "Fortran string" to "C string" was |
| | 1032 | not |
| | 1033 | done. |
| | 1034 | |
| | 1035 | 2. In the laser_field routine, when the laser field was read from |
| | 1036 | file, the index "i" was used to loop over the various laser |
| | 1037 | fields, |
| | 1038 | but in this case there is only one (this should be eventually |
| | 1039 | extended). This caused an out of bounds. |
| | 1040 | |
| | 1041 | 2006-07-11 13:29 janwer |
| | 1042 | |
| | 1043 | * : bug fixed in laser.F90. Now we can read the optimized field |
| | 1044 | directly for a td calculation.(thanks to Alberto) |
| | 1045 | |
| | 1046 | 2006-07-10 18:33 janwer |
| | 1047 | |
| | 1048 | * : I have changed the routines that read a laser pulse from a |
| | 1049 | file, so that it is compatible with the output of the optimal |
| | 1050 | control run. (Very likely that it is still buggy...). |
| | 1051 | The user interface for td targets was also modified (still |
| | 1052 | buggy..). |
| | 1053 | Will fix both in the next days. |
| | 1054 | |
| | 1055 | jan |
| | 1056 | |
| | 1057 | 2006-07-10 09:47 xavier |
| | 1058 | |
| | 1059 | * : Fixed another memory leak. |
| | 1060 | |
| | 1061 | 2006-07-08 10:06 xavier |
| | 1062 | |
| | 1063 | * : Fixed a memory leak when \delta elf was calculated several |
| | 1064 | times. |
| | 1065 | |
| | 1066 | 2006-07-07 15:49 xavier |
| | 1067 | |
| | 1068 | * : * Now functions are printed even when calculating |
| | 1069 | polarizabilities for |
| | 1070 | several frequencies. The files go to linear/freq_w (with w the |
| | 1071 | frequency). |
| | 1072 | |
| | 1073 | * Modified the dynamical linear response tolerance code, so its |
| | 1074 | value is |
| | 1075 | never decreased with respect to the previous iteration. |
| | 1076 | |
| | 1077 | 2006-07-07 14:54 acastro |
| | 1078 | |
| | 1079 | * : Bug fixed. |
| | 1080 | The calculation of the determinant stopped emitting an error if |
| | 1081 | the |
| | 1082 | determinant was null. It should only stop if there is an attempt |
| | 1083 | to |
| | 1084 | calculate the inverse matrix. |
| | 1085 | |
| | 1086 | 2006-07-07 12:36 acastro |
| | 1087 | |
| | 1088 | * : Changed a bit the interface to the occupied_states subroutine. |
| | 1089 | Now it is used by the casida module. |
| | 1090 | |
| | 1091 | 2006-07-06 19:26 acastro |
| | 1092 | |
| | 1093 | * : I got tired of looking up the documentation for the possible |
| | 1094 | envelopes |
| | 1095 | of the laser field, so I added the corresponding !%Option in the |
| | 1096 | documentations for the TDLasers block. |
| | 1097 | |
| | 1098 | 2006-07-05 16:12 xavier |
| | 1099 | |
| | 1100 | * : * Now the LR SCF uses variable precision to calculate the |
| 391 | | exchange-correlation potential. |
| 392 | | |
| 393 | | 2006-01-11 13:56 micael |
| 394 | | |
| 395 | | * src/epot_inc.F90, src/h_so.F90, testsuite/rc_bugs/12-spin_orbit: |
| 396 | | *) Spin-orbit bug fixed. |
| 397 | | |
| 398 | | 2006-01-11 09:04 appel |
| 399 | | |
| 400 | | * src/simul_box.F90: Bug fixed: The array lsize in simul_box_in_box |
| 401 | | had a wrong index. When running with BoxShape = parallelepiped |
| 402 | | the mesh was not symmetric around the origin (in z-direction) and |
| 403 | | had the wrong size. |
| 404 | | |
| 405 | | 2006-01-04 23:06 acastro |
| 406 | | |
| 407 | | * src/td_rti.F90, testsuite/finite_systems_3d/restart.test, |
| 408 | | testsuite/rc_bugs/10-Restart: Bug fixed. Another one of the |
| 409 | | NP/NP_PART that so much fun we are having with. It affected the |
| 410 | | "etrs" evolution method. |
| 411 | | |
| 412 | | Now the restart.test is finally fully passed, so I enabled it |
| 413 | | back, and removed the rc_bugs/10-Restart file. |
| 414 | | |
| 415 | | 2006-01-04 19:40 acastro |
| 416 | | |
| 417 | | * src/xc_vxc.F90, testsuite/finite_systems_2d/2dbox.test, |
| 418 | | testsuite/rc_bugs/11-2dbox: Bug fixed. |
| 419 | | |
| 420 | | This one was pretty serious, since it affected all the |
| 421 | | spin-polarized calculations, at least in some computers, such |
| 422 | | as in the Berlin opteron cluster. It was the reason that |
| 423 | | some of the tests were disabled. |
| 424 | | |
| 425 | | Now the 2dbox.test and the first two matches of the |
| 426 | | restart.test are passed OK in the opteron. However, the last |
| 427 | | two matches of the restart.test are not passed in the |
| 428 | | opteron, due to some other different bug (so I did not enable |
| 429 | | it again), and the 2dbox.test fails in my PC. This is why |
| 430 | | although I enabled again 2dbox.test, I do not remove its file |
| 431 | | in the rc_bugs directory. |
| 432 | | |
| 433 | | 2005-12-28 16:10 acastro |
| 434 | | |
| 435 | | * src/poisson.F90: Just one warning. |
| 436 | | |
| 437 | | It has to do with the bug reported by Francesco Sottile, about |
| 438 | | divergent results with some 1.99 version and the rc1. In fact, |
| 439 | | there is no such difference, but the fact that the cg_corrected |
| 440 | | Poisson solver gives wrong results when using the "minimum" box |
| 441 | | shape. The two runs reported by Francesco were done with |
| 442 | | different solvers: fft and cg_corrected. |
| 443 | | |
| 444 | | I have been trying to figure out if it is a bug, but I am |
| 445 | | starting to think it is a fact: the method gives poor results |
| 446 | | with odd box geometries. This is why I thought about putting a |
| 447 | | warning/disclaimer, so that users don't get too mad, or screw |
| 448 | | them if they do. |
| 449 | | |
| 450 | | I always thought that cg and cg_corrected were essentialy |
| 451 | | equivalent in the error that they provided, the difference being |
| 452 | | that the latter is faster. But it seems not to be true; the cg |
| 453 | | method seems to behave better with odd geometries. That is why in |
| 454 | | the warning I recommend using cg. |
| 455 | | |
| 456 | | In any case I will try to investigate why and how this happens. |
| 457 | | |
| 458 | | 2005-12-20 15:50 acastro |
| 459 | | |
| 460 | | * src/scf.F90: The clean_stop was not working during the SCF |
| 461 | | procedure since it actually was called twice -- one for checking |
| 462 | | and then to really exit the SCF loop. Since now the function |
| 463 | | erases the "stop" file, the second call found no file at all. |
| 464 | | |
| 465 | | 2005-12-20 13:48 xavier |
| 466 | | |
| 467 | | * src/static_pol_lr.F90: Correction of an error message. |
| 468 | | |
| 469 | | 2005-12-19 16:47 marques |
| 470 | | |
| 471 | | * src/Makefile.am: Added integer.F90 to the Makefile so that it is |
| 472 | | included in the .tar.gz |
| 473 | | |
| 474 | | 2005-12-19 13:38 acastro |
| 475 | | |
| 476 | | * src/: geometry.F90, xyz_file.F90: Corrected the bugs pointed out |
| 477 | | by Carlo Camilloni. I just had to apply the patches that he had |
| 478 | | suggested, I think that they were just what was needed. |
| 479 | | |
| 480 | | 2005-12-17 10:14 marques |
| 481 | | |
| 482 | | * doc/: tutorial.tex, img/elf.eps, img/laser.eps, img/pi02.eps, |
| 483 | | img/pi03.eps, img/pi04.eps, img/spacing.eps, img/spacing.ps, |
| 484 | | pdf/Makefile.am, pdf/tutorial.pdf: The tutorial was moved to the |
| 485 | | wiki. Therefore, I delete the corersponding files from the |
| 486 | | distribution. |
| 487 | | |
| 488 | | 2005-12-16 10:22 marques |
| 489 | | |
| 490 | | * doc/pdf/Makefile.am: References to octopus-devel removed |
| 491 | | |
| 492 | | 2005-12-16 10:21 marques |
| 493 | | |
| 494 | | * doc/Makefile.am: References to octopus-devel removed. |
| 495 | | |
| 496 | | 2005-12-16 10:20 marques |
| 497 | | |
| 498 | | * doc/: octopus-devel.info, octopus-devel.texi, octopus.info, |
| 499 | | octopus.texi, pdf/octopus-devel.pdf: The information contained in |
| 500 | | the -devel.texi file was passed to the wiki. Therefore I am |
| 501 | | deleting these files. |
| 502 | | |
| 503 | | 2005-12-12 14:01 appel |
| 504 | | |
| 505 | | * doc/html/Makefile.am: make was aborting, probably just a typo: |
| 506 | | make[3]: *** No rule to make target `vars/*', needed by `all-am'. |
| 507 | | Stop. |
| 508 | | |
| 509 | | I replaced vars/* by vars* |
| 510 | | |
| 511 | | 2005-12-12 12:59 marques |
| 512 | | |
| 513 | | * doc/pdf/: octopus-devel.pdf, octopus.pdf, tutorial.pdf: The pdfs |
| 514 | | are now updated |
| 515 | | |
| 516 | | 2005-12-12 12:53 marques |
| 517 | | |
| 518 | | * doc/Makefile.am, doc/html/Makefile.am, share/util/Makefile.am, |
| 519 | | src/Makefile.am, build/octopus.spec.in, testsuite/Makefile.am: |
| 520 | | There were several "bugs" in the Makefiles that made "make rpm" |
| 521 | | not work. |
| 522 | | |
| 523 | | 2005-12-12 12:52 marques |
| 524 | | |
| 525 | | * configure.ac: Added one Makefile.am |
| 526 | | |
| 527 | | 2005-12-12 09:34 marques |
| 528 | | |
| 529 | | * configure.ac, doc/octopus.texi: As Heiko suggested, I removed the |
| 530 | | "_" from the version number |
| 531 | | |
| 532 | | 2005-12-12 09:33 marques |
| 533 | | |
| 534 | | * src/: epot.F90, epot_inc.F90, h_inc.F90, states_inc.F90, |
| 535 | | v_ks_inc.F90, xc_functl.F90: some cleaning... nothing really |
| 536 | | important, except that I changed index->iatom in the |
| 537 | | projector_type. |
| 538 | | |
| 539 | | 2005-12-12 09:31 marques |
| 540 | | |
| 541 | | * configure.ac, doc/octopus.texi: Retagged 1.88->2.0_rc1 |
| 542 | | |
| 543 | | 2005-12-11 16:45 marques |
| 544 | | |
| 545 | | * src/spectrum.F90: Bug fixed. BTW, the _info routines in this file |
| 546 | | are all terribly similar. Can we somehow merge them into one? |
| 547 | | |
| 548 | | 2005-12-08 18:07 micael |
| 549 | | |
| 550 | | * testsuite/rc_bugs/12-spin_orbit: *) Adding a bug file: |
| 551 | | spin-orbit is sometimes causing a segfault. |
| 552 | | |
| 553 | | 2005-12-08 09:55 marques |
| 554 | | |
| 555 | | * src/: mix.F90, td_write.F90: These continuation lines were |
| 556 | | skrewing my octopus analyzer tool. Please try not tu use "&" |
| 557 | | inside a type definition. |
| 558 | | |
| 559 | | 2005-12-07 23:26 appel |
| 560 | | |
| 561 | | * testsuite/oct-run_regression_test.pl: Forgot $ in front of opt_m. |
| 562 | | |
| 563 | | 2005-12-07 22:51 appel |
| 564 | | |
| 565 | | * testsuite/rc_bugs/02-Na_Chain: This input file is by now running |
| 566 | | properly and can be removed. |
| 567 | | |
| 568 | | 2005-12-07 22:50 appel |
| 569 | | |
| 570 | | * testsuite/oct-run_regression_test.pl: Added option -m which |
| 571 | | allows to perform only the matches of a given test without |
| 572 | | running the code. This turned out to be useful for checking the |
| 573 | | numbers in an existing working directory. |
| 574 | | |
| 575 | | 2005-12-07 22:49 appel |
| 576 | | |
| 577 | | * src/: gs.F90, lcao.F90, scf.F90, unocc.F90: In my opinion this is |
| 578 | | definitely a compiler bug of the NAG compiler. If the arguments |
| 579 | | of the routine lcao_wf are not sorted according to their intent |
| 580 | | flavour, the compiler creates an executable that produces |
| 581 | | non-deterministic! output, i.e. you can run the very same |
| 582 | | executable with the very same input several times from scratch |
| 583 | | and you get different numbers (probably some garbage numbers from |
| 584 | | memory were collected). This was very akward to debug. |
| 585 | | Especially since I had to run the code xillion times, as not |
| 586 | | every run was giving the wrong numbers ... Finally I wasted a |
| 587 | | full day on this :(, but at least the 3D/1D periodic stuff is now |
| 588 | | running properly on 64Bit and one of the tests in rc_bugs can be |
| 589 | | removed. |
| 590 | | |
| 591 | | Probably it is a good idea to sort all other arguments that are |
| 592 | | passed to the routines in the code in a similar fashion. |
| 593 | | |
| 594 | | 2005-12-07 22:47 appel |
| 595 | | |
| 596 | | * testsuite/periodic_systems_1d/sodium_chain.test: Adding a test |
| 597 | | for a chain of sodium atoms. Finally we have the first realistic |
| 598 | | test for 3D/1D periodic systems. To speed up the runtime of the |
| 599 | | test I had to stop the SCF cycle after one step, i.e. be careful, |
| 600 | | the results are not converged. |
| 601 | | |
| 602 | | 2005-12-07 17:17 marques |
| 603 | | |
| 604 | | * src/multicomm.F90: Thre was a bug in the distribution of the |
| 605 | | nodes that made the code crash with some combination of number of |
| 606 | | nodes. |
| 607 | | |
| 608 | | 2005-12-07 09:19 appel |
| 609 | | |
| 610 | | * debian/info: Remove octopus-faq.info from the build system. |
| 611 | | |
| 612 | | 2005-12-06 16:09 acastro |
| 613 | | |
| 614 | | * src/casida.F90: Bug fixed. The casida module was breaking when |
| 615 | | using non-interacting electrons. It is a bit absurd to run the |
| 616 | | casida module with non-interacting electrons, but it is nice for |
| 617 | | testing, and the program should not break like that in any case. |
| 618 | | |
| 619 | | 2005-12-06 14:58 acastro |
| 620 | | |
| 621 | | * src/scf.F90: I guess that line was there just for debugging... |
| 622 | | |
| 623 | | 2005-12-06 10:52 marques |
| 624 | | |
| 625 | | * doc/: Makefile.am, installation.texi, octopus-faq.info, |
| 626 | | octopus-faq.texi, octopus.info, octopus.texi: Removed octopus-faq |
| 627 | | and some info from octopus.texi that are now in the wiki. |
| 628 | | |
| 629 | | 2005-12-04 22:17 marques |
| 630 | | |
| 631 | | * src/: datasets.F90, syslabels.F90: This file was rename from |
| 632 | | src/syslabels.F90 to src/datasets.F90 as explained in the |
| 633 | | previous commit. |
| 634 | | |
| 635 | | 2005-12-04 22:15 marques |
| 636 | | |
| 637 | | * src/Makefile.am, src/broad.F90, src/casida.F90, |
| 638 | | src/centergeom.F90, src/cross_section.F90, src/curv_briggs.F90, |
| 639 | | src/curv_gygi.F90, src/curv_modine.F90, src/curvlinear.F90, |
| 640 | | src/derivatives.F90, src/eigen.F90, src/fftw2.F90, src/fftw3.F90, |
| 641 | | src/geom_opt.F90, src/geometry.F90, src/gs.F90, src/h.F90, |
| 642 | | src/hs.F90, src/io.F90, src/lasers.F90, src/linear_response.F90, |
| 643 | | src/main.F90, src/math.F90, src/messages.F90, src/mix.F90, |
| 644 | | src/mpi_debug.F90, src/multicomm.F90, src/ode_solver.F90, |
| 645 | | src/opt_control.F90, src/out.F90, src/out_inc.F90, |
| 646 | | src/phonon_spectrum.F90, src/phonons.F90, |
| 647 | | src/poisson_multigrid.F90, src/profiling.F90, src/restart.F90, |
| 648 | | src/restart_inc.F90, src/root_solver.F90, |
| 649 | | src/rotational_strength.F90, src/run.F90, src/scalar_mesh.F90, |
| 650 | | src/scf.F90, src/simul_box.F90, src/sparskit.F90, src/specie.F90, |
| 651 | | src/spectrum.F90, src/states.F90, src/static_pol.F90, |
| 652 | | src/static_pol_lr.F90, src/systm.F90, src/td.F90, |
| 653 | | src/td_write.F90, src/units.F90, src/unocc.F90, src/v_ks.F90, |
| 654 | | src/wave_matching.F90, src/xc.F90, src/xc_functl.F90, |
| 655 | | src/xyz_adjust.F90, src/xyz_file.F90, src/xyzanim.F90, |
| 656 | | testsuite/periodic_systems_1d/cosine_potential.test, |
| 657 | | testsuite/periodic_systems_1d/free_electrons.test: *) I finally |
| 658 | | cleaned the run.F90 file. The reason is that, with the new |
| 659 | | changes allowing for multi-parallelization, the ability to, |
| 660 | | e.g., run the gs if the td calculation did not find the |
| 661 | | restart wave-functions had to be removed, as there could be |
| 662 | | inconsistencies in the parallelization method. Now, if you |
| 663 | | want to run both the gs and td in the same file, use the |
| 664 | | %CalculationMode block. *) I changed slightly the |
| 665 | | initialization of the old syslabels stuff, allowing for |
| 666 | | sensible defaults for the variables. Now the only thing you |
| 667 | | really have to say are the CalculationModes for each run. This |
| 668 | | implied changing the order of the lines in the block (now |
| 669 | | modes, labels, and the order). You may have to change you |
| 670 | | input files to reflect this change. *) To make this multilpe |
| 671 | | runs useful, they had (sometimes) to share the same tmp |
| 672 | | directory (otehrwise, one couldn't start a td calculation after a |
| 673 | | gs). Until now they all had different tmps. Now by default |
| 674 | | they have all the same "tmp", but this can be changed by the |
| 675 | | variable TmpDir. *) I rename "syslabels" to "datasets". The |
| 676 | | reason is that ABINIT has this multiple datasets for years, |
| 677 | | and they work more our less like ours. So I thought it was a |
| 678 | | good idea to make the names more compatible. |
| 679 | | |
| 680 | | 2005-12-01 17:51 xavier |
| 681 | | |
| 682 | | * src/static_pol_lr.F90: Fixes in dynamic polarizability, now it |
| 683 | | gives the correct result for w = 0. |
| 684 | | |
| 685 | | 2005-11-30 16:28 micael |
| 686 | | |
| 687 | | * share/util/: vf.cfg, vf.net: *) This is a DX program I wrote |
| 688 | | some time ago to plot vector fields (very usefull to |
| 689 | | plot currents ;) Probably it could be greatly improved, or |
| 690 | | even merged into the |
| 691 | | existing mf.net program, but for now I'm just commiting what |
| 692 | | I have. |
| 693 | | |
| 694 | | 2005-11-30 16:11 micael |
| 695 | | |
| 696 | | * src/spectrum.F90: *) Added the pre-factors that were missing |
| 697 | | from the complex rotatory strength function. |
| 698 | | I also added two sum rules. |
| 699 | | |
| 700 | | 2005-11-30 15:17 marques |
| 701 | | |
| 702 | | * src/: epot.F90, epot_inc.F90: I believe that this solves the |
| 703 | | problem of Carlo (please check). There was a R_CONJ missing in |
| 704 | | the non-local part of the pseudopotential. |
| 705 | | |
| 706 | | 2005-11-30 14:18 rozzi |
| 707 | | |
| 708 | | * src/static_pol_lr.F90: Intel compiler was confused by unmatched |
| 709 | | single quote |
| 710 | | |
| 711 | | 2005-11-30 11:09 xavier |
| 712 | | |
| 713 | | * src/static_pol_lr.F90: * Dynamic polarizability is implemented |
| 714 | | but not tested yet. * Now an array of lr objects is used instead |
| 715 | | of only one, indexes are direction, sign and frequency. |
| 716 | | |
| 717 | | 2005-11-30 11:05 micael |
| 718 | | |
| 719 | | * src/scf.F90: *) Only the root node should output the local |
| 720 | | magnetic moments. |
| 721 | | |
| 722 | | 2005-11-30 10:51 marques |
| 723 | | |
| 724 | | * src/: cross_section.F90, hs.F90, spectrum.F90, td_exp.F90, |
| 725 | | td_init.F90, td_rti.F90: Rearranged a little the sections to |
| 726 | | which the variables belong to. |
| 727 | | |
| 728 | | 2005-11-30 10:14 marques |
| 729 | | |
| 730 | | * liboct_parser/parser_f.c, share/variables.local, src/epot.F90, |
| 731 | | src/xc.F90, src/xc_OEP.F90, src/xyz_adjust.F90: *) Documented a |
| 732 | | couple more variables, and made the corresponding changes in |
| 733 | | variables.local *) Fixed a bug in parse_expression. There is a |
| 734 | | variable in symrec name def, that defines if a variable has |
| 735 | | been defined, and that we should not forget to set. |
| 736 | | |
| 737 | | 2005-11-30 10:10 marques |
| 738 | | |
| 739 | | * testsuite/finite_systems_1d/helium.test: I don't see any reason |
| 740 | | why the helium test should not be done with th real executable. |
| 741 | | |
| 742 | | 2005-11-30 10:07 marques |
| 743 | | |
| 744 | | * testsuite/periodic_systems_1d/: cosine_potential.test, |
| 745 | | free_electrons.test: The Units should be "a.u" and not a.u. As |
| 746 | | this is the default value, I took away those lines. |
| 747 | | |
| 748 | | 2005-11-30 00:27 appel |
| 749 | | |
| 750 | | * src/Makefile.am: Forgot oct-make-st in EXTRA_PROGRAMS. |
| 751 | | |
| 752 | | 2005-11-29 23:51 appel |
| 753 | | |
| 754 | | * src/Makefile.am: make_st can also be removed from Makefile.am. |
| 755 | | |
| 756 | | 2005-11-29 23:33 appel |
| 757 | | |
| 758 | | * liboct_parser/parser_f.c, liboct_parser/symbols.c, |
| 759 | | src/liboct_parser.F90, src/make_st.F90, src/specie.F90, |
| 760 | | src/states.F90, src/states_inc.F90, src/td.F90: I know that we |
| 761 | | are actually in feature freeze. But I believe that this is a must |
| 762 | | have feature for every td code: there is now the possibility to |
| 763 | | specify arbitrary user defined orbitals as initial states for the |
| 764 | | time propagation. |
| 765 | | |
| 766 | | Similar to user defined species this is done with formula strings |
| 767 | | in the input file: |
| 768 | | |
| 769 | | %UserDefinedStates 1 | 1 | 1 | "exp(-r^2)*exp(-i*0.2*x)" % |
| 770 | | |
| 771 | | The first three columns specify dim, nst and nik value of the |
| 772 | | respective orbital. The last column specifies the formula for the |
| 773 | | orbital. Normalization constants are not required as the code |
| 774 | | takes care of that after reading the formula. Also note, that the |
| 775 | | parser now accepts "i" for the imaginary unit. |
| 776 | | |
| 777 | | Currently octopus reads all orbitals from the restart_gs |
| 778 | | directory and only the ones specified in the above block will be |
| 779 | | replaced in the restart_gs determinant. Combined with ExtraStates |
| 780 | | this can be used e.g. to setup scattering states. The |
| 781 | | modifications also obsolete the make_st program. |
| 782 | | |
| 783 | | 2005-11-29 23:17 appel |
| 784 | | |
| 785 | | * src/restart.F90: Only the root node should delete the stop file. |
| 786 | | |
| 787 | | 2005-11-29 23:16 appel |
| 788 | | |
| 789 | | * src/: out.F90, out_inc.F90: Using 'e' in the default Fortran |
| 790 | | output format causes for very small numbers output like |
| 791 | | 9.43732460066152-133 instead of 9.43732460066152E-133. Gnuplot |
| 792 | | cannot cope with the missing E. Therefore I switched to 'f' in |
| 793 | | gnuplot_mode. |
| 794 | | |
| 795 | | 2005-11-29 23:15 appel |
| 796 | | |
| 797 | | * src/messages.F90: The debug mode was extremely slow for the MPI |
| 798 | | executable. Now the debug_trace files in the debug directory are |
| 799 | | written only if DebugLevel >= 100. For smaller values only the |
| 800 | | root node writes to stderr. |
| 801 | | |
| 802 | | 2005-11-29 23:14 appel |
| 803 | | |
| 804 | | * src/string.F90: Added routine to convert Fortran strings to C |
| 805 | | strings. |
| 806 | | |
| 807 | | 2005-11-29 23:14 appel |
| 808 | | |
| 809 | | * src/lcao.F90: To deallocate only X(psi) a full fledged |
| 810 | | states_end() was called in lcao_end. However, since there is no |
| 811 | | states_init() in lcao_init() there were some arrays not |
| 812 | | available. I got segfaults due to this (although this shouldn't |
| 813 | | happen since states_end() checks if the pointers are associated. |
| 814 | | Perhaps a compiler bug?). |
| 815 | | |
| 816 | | 2005-11-29 23:13 appel |
| 817 | | |
| 818 | | * src/io.F90: Now also flush_messages can be enabled/disabled on |
| 819 | | the fly. Simply touch the files |
| 820 | | enable_flush_messages/disable_flush_messages. |
| 821 | | |
| 822 | | 2005-11-29 15:59 marques |
| 823 | | |
| 824 | | * src/: epot.F90, fftw3.F90, mesh.F90, mesh_init.F90: A phase was |
| 825 | | wrong in the code. The most important thing was a minus sign |
| 826 | | missing. I believe that the potential is OK now. |
| 827 | | |
| 828 | | 2005-11-28 17:32 xavier |
| 829 | | |
| 830 | | * src/spectrum.F90: There was a problem reading the multipoles file |
| 831 | | in spectrum_hs_from_mult. |
| 832 | | |
| 833 | | 2005-11-28 15:50 marques |
| 834 | | |
| 835 | | * src/spectrum.F90: There were a couple of '#' missing, which made |
| 836 | | oct-cross-section crash while readin the _vector files. I also |
| 837 | | took the opportunity of removing several lines that gave warning |
| 838 | | with ifc7. |
| 839 | | |
| 840 | | 2005-11-28 14:45 marques |
| 841 | | |
| 842 | | * src/td_rti.F90: removed some unused lines of code. |
| 843 | | |
| 844 | | 2005-11-28 14:41 marques |
| 845 | | |
| 846 | | * liboct_parser/grammar.y, liboct_parser/parse.c, |
| 847 | | liboct_parser/parse_exp.c, liboct_parser/symbols.c, |
| 848 | | liboct_parser/symbols.h, src/poisson_multigrid.F90, |
| 849 | | src/td_exp.F90, src/td_rti.F90: *) Added a very tiny feature to |
| 850 | | the parser. Now it will stop with a input error if one |
| 851 | | attempts to use a variable that has not been defined. This |
| 852 | | avoids misspells in the input file to be acepted as valid inputs. |
| 853 | | (most of the diff is whitespace, as the files were indented |
| 854 | | with tabs, when they should be indent with spaces) *) This |
| 855 | | allowed to find a couple of variables that were using "-". This |
| 856 | | is forbidden, as he "-" is going to be confused with the minus |
| 857 | | sign. For example, suzuki-troter is read as "suzuki" - |
| 858 | | "troter", and as these two variables have not been defined |
| 859 | | (i.e. are 0), the result will be 0, and not the expected |
| 860 | | value. |
| 861 | | |
| 862 | | 2005-11-28 12:03 marques |
| 863 | | |
| 864 | | * libxc/: lda.c, lda_c_vwn.c, lda_x.c, test.c: *) very tiny bug |
| 865 | | corrected in VWN (one of the constants was wrong in the 5th |
| 866 | | digit) *) fxc is now calculated numerically a bit better in case |
| 867 | | of the ferromagnetic gas. Actually I don't really know how to |
| 868 | | handle this case properly, as fxc should diverge (at least fx). |
| 869 | | |
| 870 | | 2005-11-27 10:32 appel |
| 871 | | |
| 872 | | * testsuite/oct-run_regression_test.pl: There was a \ missing. |
| 873 | | Without the \ the -c option was creating buggy testfiles. |
| 874 | | |
| 875 | | 2005-11-27 10:30 appel |
| 876 | | |
| 877 | | * testsuite/: finite_systems_2d/2dbox.test, |
| 878 | | finite_systems_3d/restart.test, rc_bugs/10-Restart, |
| 879 | | rc_bugs/11-2dbox: There's something fishy with the precision on |
| 880 | | 64Bit. The last input in the 2dbox test takes currently 114 scf |
| 881 | | cycles to converge on x86_64 but requires only 14 on i386. Also |
| 882 | | the eigenvalues and total energy come out differently. Similar |
| 883 | | things apply to the last input file in restart.test. |
| 884 | | |
| 885 | | To avoid spamming of octopus-devel I've disabled both tests |
| 886 | | (Enabled: No) and have copied them to the rc_bugs directory. |
| 887 | | Before we release this eventually has to be sorted. |
| 888 | | |
| 889 | | 2005-11-26 18:58 acastro |
| 890 | | |
| 891 | | * src/states.F90: The output of the information of which node |
| 892 | | manages which KS states was not good for the case where there is |
| 893 | | a combined parallelization scheme. I have redone it; now it is |
| 894 | | also better since only root node is doing the output, as it |
| 895 | | should be. [I noticed this when doing a run with 128 processors: |
| 896 | | 32 domains and four states groups. OK, I could only do a 10 |
| 897 | | minutes run in the debug queue, but still, I think it is a |
| 898 | | record. The speedup, by the way, was of about 80] |
| 899 | | |
| 900 | | 2005-11-26 17:43 acastro |
| 901 | | |
| 902 | | * src/: profiling.F90, restart.F90, td.F90: (*) Added the TIME_STEP |
| 903 | | tag to the profiling module. (*) A little bug fixed also in that |
| 904 | | module. (*) The clean_stop routine deletes the stop file, as |
| 905 | | Heiko suggested. I also found it annoying, never remembered |
| 906 | | to delete it. |
| 907 | | |
| 908 | | 2005-11-25 22:31 appel |
| 909 | | |
| 910 | | * src/: io.F90, out_inc.F90, scf.F90, td.F90: I found this quite |
| 911 | | useful for chasing bugs which were not showing up immediately: |
| 912 | | There is now the possibility to enable/disable the debug mode on |
| 913 | | the fly. Imagine a bug that shows up after 100 SCF cycles. One |
| 914 | | definitely does not want to run the code all the way in debug |
| 915 | | mode. The present modifications allow to run the first 99 cycles |
| 916 | | in normal mode and to switch only then to debug mode. |
| 917 | | |
| 918 | | To enable the debug mode on the fly you have to touch a file with |
| 919 | | name enable_debug_mode in the working directory, similarly to |
| 920 | | disable the debug mode simply touch disable_debug_mode. Contrary |
| 921 | | to the stop file the code takes care of the cleanup of the |
| 922 | | touched files in this case. (should the code also take care of |
| 923 | | the cleanup of the stop file?) What a pity that Fortran does not |
| 924 | | support signal handling ... |
| 925 | | |
| 926 | | If the code runs inside some queueing system which buffers output |
| 927 | | this could also be used to quickly check if octopus is still |
| 928 | | doing reasonable things or got e.g. stuck in an MPI deadlock. |
| 929 | | |
| 930 | | 2005-11-25 20:27 appel |
| 931 | | |
| 932 | | * src/states.F90, testsuite/finite_systems_2d/fock_darwin.test, |
| 933 | | testsuite/finite_systems_3d/restart.test: Adjusting the failing |
| 934 | | tests to the new format. |
| 935 | | |
| 936 | | 2005-11-24 21:55 appel |
| 937 | | |
| 938 | | * src/states.F90: I just noticed that I unintentionally modified |
| 939 | | the format for the output of the eigenvalues in my last commit. |
| 940 | | The altered format causes some tests to fail (the matches don't |
| 941 | | work as intended). I've now again changed the output of the |
| 942 | | eigenvalues, since in the spin-polarized case more than 80 |
| 943 | | columns were used. Please have a look if you agree with the new |
| 944 | | format (basically I use "up", "dn", "--" for spin up, spin down |
| 945 | | and spin unpolarized respectively). If you agree with the |
| 946 | | modification I volunteer to update the test files accordingly. |
| 947 | | |
| 948 | | 2005-11-24 19:42 appel |
| 949 | | |
| 950 | | * src/par_vec.F90: Forgot to remove debug output. |
| 951 | | |
| 952 | | 2005-11-24 19:37 appel |
| 953 | | |
| 954 | | * src/: io.F90, mesh.F90, mesh_init.F90, messages.F90, par_vec.F90, |
| 955 | | simul_box.F90, states.F90, xc.F90, xc_functl.F90: Support for |
| 956 | | FlushMessages was broken. When running in parallel filehandles |
| 957 | | were not treated properly. Instead of writing directly to some |
| 958 | | unit I've changed several places such that they use the write_* |
| 959 | | functions. |
| 960 | | |
| 961 | | 2005-11-24 18:36 appel |
| 962 | | |
| 963 | | * src/mesh_init.F90: Array index was missing in the last commit. |
| 964 | | |
| 965 | | 2005-11-24 17:08 rozzi |
| 966 | | |
| 967 | | * src/: mesh.F90, mesh_init.F90: Bug fixed in the building of the |
| 968 | | potential for periodic calculations: the box size was not correct |
| 969 | | |
| 970 | | 2005-11-24 13:55 acastro |
| 971 | | |
| 972 | | * src/mf_inc.F90: Just some corrections of the push_sub argument, |
| 973 | | which seem to have no relation to the actual subroutine name in |
| 974 | | many cases. |
| 975 | | |
| 976 | | 2005-11-24 13:39 marques |
| 977 | | |
| 978 | | * src/td_rti.F90: another NP NP_PART issue. |
| 979 | | |
| 980 | | 2005-11-24 13:09 marques |
| 981 | | |
| 982 | | * src/nl_operator.F90: Some cleaniong in nl_operator.F90. I think |
| 983 | | the routines _operate are more readable now. |
| 984 | | |
| 985 | | 2005-11-24 12:35 marques |
| 986 | | |
| 987 | | * src/: derivatives.F90, eigen_trlan.F90, mesh_init.F90, |
| 988 | | mf_inc.F90, multicomm.F90, nl_operator.F90, out_inc.F90, |
| 989 | | poisson.F90, run.F90, systm.F90: *) Cleaned the #if |
| 990 | | defined(HAVE_METIS) directives in the code. Now they only |
| 991 | | appear in 2 or 3 places (where they are really needed) *) The |
| 992 | | code now stops if you want to run in parallel in domains with a |
| 993 | | fft poisson solver. This made me waste 30 minutes ;(( |
| 994 | | |
| 995 | | Incidentally, I now get *exactly* the same result running in |
| 996 | | serial or in parallel (and not only approximately like before). |
| 997 | | This means that it was either a np<->np_part issue, or a problem |
| 998 | | with the HAVE_METIS (in fact there were a couple of |
| 999 | | inconsistencies in the code) |
| 1000 | | |
| 1001 | | 2005-11-24 10:12 appel |
| 1002 | | |
| 1003 | | * share/PP/defaults: Updating the PP/defaults file for Cd since by |
| 1004 | | now all elements are included. |
| 1005 | | |
| 1006 | | 2005-11-24 09:59 marques |
| 1007 | | |
| 1008 | | * share/PP/TM2/Cd.ascii: Sorry, but this pseudo has problems, so I |
| 1009 | | removed it. |
| 1010 | | |
| 1011 | | 2005-11-23 17:15 xavier |
| 1012 | | |
| 1013 | | * src/: linear_response_inc.F90, static_pol_lr.F90: * Bugfix in |
| 1014 | | linear response: Because density was put to zero before time, the |
| 1015 | | fxc term was not considered for the polarizability. This affected |
| 1016 | | calculations of polarizability and hyperopolarizability. |
| 1017 | | |
| 1018 | | * Added a check for DevelVersion to protect users from the |
| 1019 | | dynamic polarizability part which is not working. |
| 1020 | | |
| 1021 | | 2005-11-23 15:54 micael |
| 1022 | | |
| 1023 | | * src/h_inc.F90: *) Moved the Zeeman term from subroutine vlpsi to |
| 1024 | | subroutine magnetic_terms |
| 1025 | | as it seems more apropriate. |
| 1026 | | |
| 1027 | | 2005-11-23 15:05 micael |
| 1028 | | |
| 1029 | | * src/xc_axc.F90: *) Some more np/np_part issues fixed. |
| 1030 | | |
| 1031 | | 2005-11-23 14:54 micael |
| 1032 | | |
| 1033 | | * share/variables.local: *) This two lines can now go away. |
| 1034 | | |
| 1035 | | 2005-11-23 14:50 micael |
| 1036 | | |
| 1037 | | * src/xc_functl.F90: *) Documented the current functionals. |
| 1038 | | |
| 1039 | | 2005-11-23 14:21 micael |
| 1040 | | |
| 1041 | | * src/: h.F90, h_inc.F90, scf.F90, v_ks_inc.F90: *) Removed the |
| 1042 | | current-current interaction from the CDFT hamiltonian, because it |
| 1043 | | is |
| 1044 | | time-consuming and in practice it should be neglectable. |
| 1045 | | |
| 1046 | | 2005-11-23 12:24 appel |
| 1047 | | |
| 1048 | | * src/states.F90: Bug fixed. The bands were not written anymore. |
| 1049 | | |
| 1050 | | 2005-11-23 12:02 micael |
| 1051 | | |
| 1052 | | * src/h_inc.F90: *) Bug fixed: again a np/np_part issue. This time |
| 1053 | | in the CDFT part. |
| 1054 | | *) Corrected another 1/2 factor that was wrong in the magnetic |
| 1055 | | terms. |
| 1056 | | |
| 1057 | | 2005-11-22 23:22 appel |
| 1058 | | |
| 1059 | | * src/: Makefile.am, global.h, par_vec.F90, poisson.F90, |
| 1060 | | td_exp_split.F90: These modifications of the build system allow |
| 1061 | | for newline characters inside CPP macros. This overcomes the |
| 1062 | | problems with line truncation that we had with some compilers. |
| 1063 | | I've chosen \newline as newline sequence as \n was colliding with |
| 1064 | | things like eg. \nabla inside comments. The changes partially |
| 1065 | | obsolete Alberto's m4 check for long lines. However, I kept it |
| 1066 | | since it is also used for conf%fcflags. |
| 1067 | | |
| 1068 | | 2005-11-22 08:51 appel |
| 1069 | | |
| 1070 | | * src/v_ks_inc.F90: Because of too long lines the code did not |
| 1071 | | compile with the NAG compiler. |
| 1072 | | |
| 1073 | | 2005-11-21 17:16 acastro |
| 1074 | | |
| 1075 | | * src/casida.F90: Bug fixed. The casida module was broken. Yet |
| 1076 | | another np/np_part issue. |
| 1077 | | |
| 1078 | | 2005-11-21 14:14 marques |
| 1079 | | |
| 1080 | | * src/: eigen.F90, eigen_evolution.F90, eigen_plan.F90, |
| 1081 | | eigen_trlan.F90, epot.F90, epot_inc.F90, f_inc.F90, fftw3.F90, |
| 1082 | | geom_opt.F90, geometry.F90, grid.F90, gs.F90, h.F90, h_inc.F90, |
| 1083 | | h_so.F90, hgh.F90, lasers.F90, lcao.F90, linalg_adv_lapack.F90, |
| 1084 | | linalg_basic_int.F90, linear_response.F90, |
| 1085 | | linear_response_inc.F90, logrid.F90, make_st.F90, math.F90, |
| 1086 | | math_cg_inc.F90, math_inc.F90, mesh_init.F90, mf_inc.F90, |
| 1087 | | mix.F90, multicomm.F90, multigrid.F90, nl_operator.F90, |
| 1088 | | ode_solver_inc.F90, opt_control.F90, out_inc.F90, par_vec.F90, |
| 1089 | | par_vec_inc.F90, pes_mask.F90, pes_rc.F90, phonons.F90, |
| 1090 | | poisson.F90, poisson1D.F90, poisson2D.F90, poisson3D.F90, |
| 1091 | | poisson_cg.F90, poisson_corrections.F90, poisson_multigrid.F90, |
| 1092 | | ps.F90, restart_inc.F90, root_solver.F90, root_solver_inc.F90, |
| 1093 | | scalar_mesh.F90, scf.F90, sparskit_inc.F90, specie.F90, |
| 1094 | | spectrum.F90, spline.F90, states.F90, states_inc.F90, |
| 1095 | | states_kpoints.F90, static_pol.F90, static_pol_lr.F90, |
| 1096 | | stencil_star.F90, systm.F90, td_calc.F90, td_exp.F90, |
| 1097 | | td_exp_split.F90, td_rti.F90, td_write.F90, tm.F90, unocc.F90, |
| 1098 | | v_ks_inc.F90, xc_KLI.F90, xc_OEP.F90, xc_OEP_SIC.F90, |
| 1099 | | xc_OEP_inc.F90, xc_axc.F90, xc_fxc.F90, xc_kxc.F90, xc_vxc.F90, |
| 1100 | | xyz_file.F90: I changed the rest of the allocate -> ALLOCATE. Now |
| 1101 | | all the code has some rudimentary memory allocation error checks. |
| 1102 | | I am still not very happy about this solution. I have plans to |
| 1103 | | improve it (allowing also for statistics) but this will have to |
| 1104 | | wait for the next version. |
| 1105 | | |
| 1106 | | 2005-11-20 21:57 xavier |
| 1107 | | |
| 1108 | | * testsuite/rc_bugs/09-H2O_POL: Removed input file from bug that |
| 1109 | | was fixed in the previous commit. |
| 1110 | | |
| 1111 | | 2005-11-20 21:51 xavier |
| 1112 | | |
| 1113 | | * doc/octopus-devel.info, doc/octopus.info, doc/octopus.texi, |
| 1114 | | src/eigen_evolution.F90, src/geom_opt.F90, src/h_inc.F90, |
| 1115 | | src/linear_response.F90, src/linear_response_inc.F90, |
| 1116 | | src/make_st.F90, src/phonons.F90, src/static_pol.F90, |
| 1117 | | src/static_pol_lr.F90, src/td_write.F90: Bugfixes related to the |
| 1118 | | np/np_part transition: * Linear response. * Other parts where |
| 1119 | | st%X(psi) was initialized with np size. * Added an assert to |
| 1120 | | check input array size in h_inc.XHpsi. |
| 1121 | | |
| 1122 | | 2005-11-20 19:21 acastro |
| 1123 | | |
| 1124 | | * src/profiling.F90, src/states.F90, src/xc_KLI.F90, |
| 1125 | | src/xc_OEP_inc.F90, src/xc_OEP_x.F90, |
| 1126 | | testsuite/rc_bugs/04-Na2_Exx: Bug fixed. This time, 04-Na2_Exx. |
| 1127 | | It had to do with the parallelization. |
| 1128 | | |
| 1129 | | I do not put it as a test because I am not fully certain of the |
| 1130 | | validity of the result. I will try to find exact known results |
| 1131 | | for EXX and make a test for it. Also for the rest of the |
| 1132 | | orbital-dependent functionals. |
| 1133 | | |
| 1134 | | 2005-11-18 17:02 acastro |
| 1135 | | |
| 1136 | | * src/poisson_corrections.F90: When running in parallel in domains, |
| 1137 | | I got a crash there due to the np/np_part problem. |
| 1138 | | |
| 1139 | | 2005-11-18 15:56 acastro |
| 1140 | | |
| 1141 | | * src/h_inc.F90: Bug fixed. Forgot one pop_sub |
| 1142 | | |
| 1143 | | 2005-11-18 15:12 acastro |
| 1144 | | |
| 1145 | | * src/profiling.F90: I don't think the profiling_in and |
| 1146 | | profiling_out should be monitored by the push_sub/pop_sub, the |
| 1147 | | output gets rather ugly. |
| 1148 | | |
| 1149 | | 2005-11-18 14:52 acastro |
| 1150 | | |
| 1151 | | * src/out_inc.F90: The ASSERT(0) led to an arithmetic if in Fortran |
| 1152 | | that is not accepted by modern compilers. |
| 1153 | | |
| 1154 | | 2005-11-18 14:41 acastro |
| 1155 | | |
| 1156 | | * src/: h.F90, h_inc.F90, poisson.F90, profiling.F90, v_ks.F90, |
| 1157 | | v_ks_inc.F90: Added some more profiling tags. I know that we |
| 1158 | | should be in regression mode, but maybe we should have some way |
| 1159 | | to properly benchmark the 2.0 code against all future |
| 1160 | | developments. |
| 1161 | | |
| 1162 | | 2005-11-18 13:54 acastro |
| 1163 | | |
| 1164 | | * src/: nl_operator.F90, profiling.F90: Bug fixed in the profiling. |
| 1165 | | The nl_operator tag had two entries and no exit. |
| 1166 | | |
| 1167 | | I improved a little bit the output in the profiling.xxx files, so |
| 1168 | | that it is more readable. |
| 1169 | | |
| 1170 | | 2005-11-18 11:38 acastro |
| 1171 | | |
| 1172 | | * src/mesh_init.F90: The code was crashing in the marenostrum |
| 1173 | | machine at unexpected points near the call to mesh_partition, |
| 1174 | | when attempting to run systems with large meshes. The reason |
| 1175 | | seemed to be a memory problem, probably the code was legal but |
| 1176 | | the memory managemente was badly done by the compiler (mpxlf |
| 1177 | | compiler from ibm). With the changes that I did, it seems to work |
| 1178 | | fine now. I don't know, maybe in the way that it was done before, |
| 1179 | | some odd copies were being made, filling the cache, or the stack, |
| 1180 | | or whatever, maybe the ventilators, god knows. |
| 1181 | | |
| 1182 | | In general, it is probably not a good practice to pass a |
| 1183 | | structure and components of that structure separately to a |
| 1184 | | subroutine. Also, it is probably better to use allocate than |
| 1185 | | using automatic arrays. |
| 1186 | | |
| 1187 | | 2005-11-17 17:36 xavier |
| 1188 | | |
| 1189 | | * src/static_pol_lr.F90: The off-diagonal terms of the beta tensor |
| 1190 | | were wrong. |
| 1191 | | |
| 1192 | | 2005-11-17 15:16 acastro |
| 1193 | | |
| 1194 | | * testsuite/finite_systems_2d/fock_darwin.test: Added a test for |
| 1195 | | the magnetic field. For the moment, only in 2D, and with a null |
| 1196 | | Zeeman term. It is the spectrum of Fock-Darwin (one particle in |
| 1197 | | 2D confined by a Harmonic potential, and subject to a |
| 1198 | | perpendicular static constant magnetic field). |
| 1199 | | |
| 1200 | | 2005-11-17 14:58 acastro |
| 1201 | | |
| 1202 | | * src/unocc.F90: The unocc mode was broken due to the usual |
| 1203 | | np/np_part problem. |
| 1204 | | |
| 1205 | | 2005-11-17 14:33 xavier |
| 1206 | | |
| 1207 | | * src/geometry.F90: The atom_write_xyz function was printing the |
| 1208 | | file in all nodes except root, as a result oct-center-geom didn't |
| 1209 | | write the adjusted.xyz file. So I added a .not. |
| 1210 | | |
| 1211 | | 2005-11-17 11:43 acastro |
| 1212 | | |
| 1213 | | * liboct_parser/parser_f.c: Bug fixed. In some cases, the code |
| 1214 | | stopped in xyz_file_read when attempting to see if the block was |
| 1215 | | defined. This was due to the assert statement in the |
| 1216 | | loct_parse_block C function, which may detect non-null |
| 1217 | | block_name. |
| 1218 | | |
| 1219 | | I am not too sure about why this was happening, but in any case I |
| 1220 | | think it is much more robust this way. |
| 1221 | | |
| 1222 | | 2005-11-17 08:30 xavier |
| 1223 | | |
| 1224 | | * src/static_pol_lr.F90: Fixed compilation issues with complex |
| 1225 | | version. |
| 1226 | | |
| 1227 | | 2005-11-16 23:37 acastro |
| 1228 | | |
| 1229 | | * src/: epot.F90, h_inc.F90: I tested the static external magnetic |
| 1230 | | field. Apart from a 1/2 factor that shouldn't be there, it was |
| 1231 | | OK. |
| 1232 | | |
| 1233 | | I allowed for the possibility of having the external magnetic |
| 1234 | | field and *not* doing CDFT, they are independent things. |
| 1235 | | |
| 1236 | | The test that I did was to reproduce the Darwin-Fock spectrum in |
| 1237 | | 2D: eigenvalues of a 2D single-particle system, confined by a |
| 1238 | | harmonic potential, and subject to a perpendicular magnetic |
| 1239 | | field. This spectrum is analytical. |
| 1240 | | |
| 1241 | | However, I did not check the Zeeman term. |
| 1242 | | |
| 1243 | | 2005-11-16 17:55 xavier |
| 1244 | | |
| 1245 | | * src/static_pol_lr.F90, testsuite/rc_bugs/09-H2O_POL: Some |
| 1246 | | corrections for first static hyperpolarizability, and now it |
| 1247 | | seems to work fine (the spin polarized case still needs some |
| 1248 | | work). |
| 1249 | | |
| 1250 | | Unfortunately, the linear response code is broken after the |
| 1251 | | np->np_part transition, but I haven't been able to locate where |
| 1252 | | the problem is. So I am commiting a example input to the rc_bugs |
| 1253 | | directory. |
| 1254 | | |
| 1255 | | 2005-11-15 13:46 xavier |
| 1256 | | |
| 1257 | | * src/linalg_basic_blas.F90: Removed spaces before the # in a |
| 1258 | | couple of preprocessor directives, Compaq cpp doesn't like them. |
| 1259 | | |
| 1260 | | 2005-11-15 13:12 marques |
| 1261 | | |
| 1262 | | * src/global.h: This should fix the compilation issues from |
| 1263 | | yesterday. |
| 1264 | | |
| 1265 | | 2005-11-14 20:58 marques |
| 1266 | | |
| 1267 | | * src/: atomic.F90, broad.F90, casida.F90, cf_inc.F90, |
| 1268 | | cross_section.F90, crystal.F90, curv_gygi.F90, derivatives.F90, |
| 1269 | | derivatives_inc.F90, eigen_arpack.F90, eigen_cg.F90, global.h, |
| 1270 | | messages.F90: I spent more than half an hour searching for a bug |
| 1271 | | that didn't exist - it had been an allocate error (out of |
| 1272 | | memory), but, as usual, it appeared as a segmentation fault. |
| 1273 | | Therefore, I decided to put a very simple check for the |
| 1274 | | allocation errors. It is basically a preprocessor macro, and |
| 1275 | | changing allocate -> ALLOCATE. I started changing all allocates |
| 1276 | | in the code, but I will finish when I come back from Louvain. |
| 1277 | | |
| 1278 | | 2005-11-14 20:00 micael |
| 1279 | | |
| 1280 | | * src/v_ks_inc.F90: *) Sorry, I forgot to remove those lines. |
| 1281 | | |
| 1282 | | 2005-11-14 17:45 appel |
| 1283 | | |
| 1284 | | * src/: broad.F90, out.F90: Remove unused variables. |
| 1285 | | |
| 1286 | | 2005-11-14 17:45 appel |
| 1287 | | |
| 1288 | | * src/sparskit_inc.F90: Documented all Sparskit variables. |
| 1289 | | |
| 1290 | | 2005-11-14 17:26 appel |
| 1291 | | |
| 1292 | | * src/v_ks_inc.F90: The code did not compile. I commented some |
| 1293 | | lines in v_hartree() and added a FIXME at the corresponing |
| 1294 | | position. Micael, can you check if this part is still required? |
| 1295 | | |
| 1296 | | 2005-11-14 17:06 micael |
| 1297 | | |
| 1298 | | * src/restart_inc.F90: *) Increased the number of digits of the |
| 1299 | | number of iterations in file "wfns". |
| 1300 | | |
| 1301 | | 2005-11-14 16:58 micael |
| 1302 | | |
| 1303 | | * src/: h.F90, h_inc.F90: *) Removed the effective-mass model as |
| 1304 | | there were some precision issues when using it. |
| 1305 | | Also, I realized there is a better way of including this |
| 1306 | | model: I will make it a system |
| 1307 | | of units. |
| 1308 | | |
| 1309 | | 2005-11-14 16:15 marques |
| 1310 | | |
| 1311 | | * src/eigen_arpack.F90: Small bug corrected. h has to have intent |
| 1312 | | inout now. |
| 1313 | | |
| 1314 | | 2005-11-14 16:14 marques |
| 1315 | | |
| 1316 | | * share/variables.local: Forgot to commit this file. |
| 1317 | | |
| 1318 | | 2005-11-14 16:05 micael |
| 1319 | | |
| 1320 | | * testsuite/finite_systems_3d/spinors.test: *) Changed the spinors |
| 1321 | | test. Now it uses the Lanczos eigensolver. |
| 1322 | | |
| 1323 | | 2005-11-14 15:52 marques |
| 1324 | | |
| 1325 | | * src/: curvlinear.F90, global.F90, io.F90, main.F90: *) Moved |
| 1326 | | reading of DebugLevel from io to main *) Added variable |
| 1327 | | DevelVersion. As discussed with Heiko, the objective is that |
| 1328 | | when conf%devel_version is .false. certain areas of the code are |
| 1329 | | not accessible. This allows to "disconnect" some options, and |
| 1330 | | reconnect them with DebugLevel, i.w., without having to |
| 1331 | | comment the test in the source file. *) conf%devel_version is |
| 1332 | | only used now in curvlinear.F90 |
| 1333 | | |
| 1334 | | 2005-11-14 15:03 marques |
| 1335 | | |
| 1336 | | * src/: derivatives_inc.F90, eigen.F90, eigen_cg.F90, |
| 1337 | | eigen_evolution.F90, eigen_plan.F90, f_inc.F90, geometry.F90, |
| 1338 | | global.h, h_inc.F90, lcao.F90, poisson_cg.F90, |
| 1339 | | poisson_corrections.F90, poisson_multigrid.F90, scf.F90, |
| 1340 | | states.F90, states_inc.F90, td.F90, td_calc.F90, td_exp.F90, |
| 1341 | | td_write.F90, xc_vxc.F90: This change may cause some problems in |
| 1342 | | some cases. The problem is that, with the changes that Florian |
| 1343 | | and Heiko did, the wave-functions (or better, any function that |
| 1344 | | we want to calculate the derivative of) must be defined from |
| 1345 | | (1:m%np_part). If I remember well, this had to do with the |
| 1346 | | communication of the ghost points. What happened is that in |
| 1347 | | several places in the code, the functions were defined from |
| 1348 | | 1:m%np. An example, in gs.F90, st%psi is defined from |
| 1349 | | 1:m%np_part, while in td.F90 st%psi is defined from 1:m%np. |
| 1350 | | |
| 1351 | | This is therefore an attemp to to clarify this situation. All |
| 1352 | | functions have to be passed to derivatives defined from |
| 1353 | | 1:m%np_part. All tests work at the moment, but it is possible |
| 1354 | | that some parts of the code will fail (for example, I am not 100% |
| 1355 | | sure of td_exp::lanczos) |
| 1356 | | |
| 1357 | | Furthermore, there was a problem and the ASSERT macro was never |
| 1358 | | define. Now it is OK. I also added some ASSERTs to the |
| 1359 | | calculation of the derivatives. |
| 1360 | | |
| 1361 | | 2005-11-14 13:56 micael |
| 1362 | | |
| 1363 | | * src/eigen_plan.F90: *) Bug fixed: Lanczos eigensolver was not |
| 1364 | | working as there were some |
| 1365 | | inconsistencies between the size of the arrays and the calls |
| 1366 | | to the blas. |
| 1367 | | |
| 1368 | | 2005-11-14 13:36 marques |
| 1369 | | |
| 1370 | | * testsuite/periodic_systems_1d/: cosine_potential.test, |
| 1371 | | free_electrons.test: Following the recent changes of Alberto, the |
| 1372 | | tests were no longer working. Now they do. |
| 1373 | | |
| 1374 | | 2005-11-14 12:10 acastro |
| 1375 | | |
| 1376 | | * src/geometry.F90, src/scf.F90, testsuite/rc_bugs/06-Spinors: I |
| 1377 | | did not know that geometry_min_distance existed. It can be used |
| 1378 | | in geometry_atoms_are_too_close, of course. |
| 1379 | | |
| 1380 | | Also, I made sure that a meaningful default lmm_r is set in scf |
| 1381 | | when there is only one atom. It was wrong before due to the huge |
| 1382 | | value returned by geometry_min_distance. This way the 06-Spinors |
| 1383 | | bug file can also go away. |
| 1384 | | |
| 1385 | | 2005-11-14 11:38 acastro |
| 1386 | | |
| 1387 | | * share/samples/Makefile.am, src/geometry.F90, src/messages.F90, |
| 1388 | | testsuite/rc_bugs/01-Hartree_1D: Added a |
| 1389 | | geometry_atoms_are_too_close check to make sure that there are no |
| 1390 | | atoms sitting on top of each other in the input file. This way we |
| 1391 | | can get rid of one of the bug files, 01-Hartree_1D. |
| 1392 | | |
| 1393 | | I forgot to change the Makefile.am when I changed the sample |
| 1394 | | input files. |
| 1395 | | |
| 1396 | | Added a new line in write_fatal, otherwise the output looked |
| 1397 | | ugly. |
| 1398 | | |
| 1399 | | 2005-11-14 11:06 acastro |
| 1400 | | |
| 1401 | | * share/samples/Na2td: I added the Na2td sample input file, so that |
| 1402 | | one time-dependent run is also in the samples directory. |
| 1403 | | |
| 1404 | | 2005-11-14 11:01 acastro |
| 1405 | | |
| 1406 | | * share/samples/: Na2, bench-gs, benzene: After so many changes in |
| 1407 | | the input variables, the sample file had got wildly inaccuarte. I |
| 1408 | | removed most of the variables, leaving a more minimal set, I |
| 1409 | | think Miguel asked for that some time ago. I also renamed |
| 1410 | | bench-gs into benzene, and chaged it slightly. We can still use |
| 1411 | | it for benchmarking, anyways, but the name is more descriptive. |
| 1412 | | We should maybe add more examples before the commit, and describe |
| 1413 | | them better. |
| 1414 | | |
| 1415 | | 2005-11-14 10:24 marques |
| 1416 | | |
| 1417 | | * src/: h.F90, multicomm.F90, out.F90, scf.F90, states.F90, |
| 1418 | | states_inc.F90, td_write.F90, varinfo.F90: A small change to make |
| 1419 | | the code more usable. I transformed the OutputXXX and TDOutputXXX |
| 1420 | | variables into flags. In this way, the ~20 variables of this kind |
| 1421 | | are transformed in only 3: Output, TDOutput, and OutputHow |
| 1422 | | |
| 1423 | | 2005-11-14 10:18 marques |
| 1424 | | |
| 1425 | | * testsuite/oct-run_regression_test.pl: Small change to allow the |
| 1426 | | use of ; (written as '\;') in the precommand part of the match. |
| 1427 | | This is useful if we want to use awk scripts in the match. |
| 1428 | | |
| 1429 | | 2005-11-14 08:03 acastro |
| 1430 | | |
| 1431 | | * src/: global.F90, hgh.F90: This fixes, I think, the compilation |
| 1432 | | errors of yesterday night. |
| 1433 | | |
| 1434 | | 2005-11-14 00:34 acastro |
| 1435 | | |
| 1436 | | * src/mpi.F90: I believe that the #else there should be an #endif. |
| 1437 | | Otherwise some problems could show up when running with one |
| 1438 | | processor but with one mpi executable. |
| 1439 | | |
| 1440 | | 2005-11-13 22:22 acastro |
| 1441 | | |
| 1442 | | * src/curv_gygi.F90, testsuite/finite_systems_3d/curvilinear.test: |
| 1443 | | Added one first test for the curvlinear coordinates stuff. It is |
| 1444 | | a bit slow, but the initialization time of the curvilinear runs |
| 1445 | | are that way. |
| 1446 | | |
| 1447 | | Also, corrected a small error in the documentation of the |
| 1448 | | CurvGygiA variable. |
| 1449 | | |
| 1450 | | 2005-11-13 21:09 acastro |
| 1451 | | |
| 1452 | | * src/ps.F90: Bug fixed. The code crashed when trying to run |
| 1453 | | Helium with the HGH pseudopotential. |
| 1454 | | |
| 1455 | | 2005-11-13 11:04 acastro |
| 1456 | | |
| 1457 | | * src/: Makefile.am, oct_test.F90: Removed the oct-test program, |
| 1458 | | which was there for internal use, and should not be there in the |
| 1459 | | release. |
| 1460 | | |
| 1461 | | 2005-11-12 23:07 acastro |
| 1462 | | |
| 1463 | | * src/xc_functl.F90, testsuite/finite_systems_2d/2dbox.test: The |
| 1464 | | default correlation should not be the same in 3D and 2D. |
| 1465 | | |
| 1466 | | I added tests for the exchange and correlation potential in 2D |
| 1467 | | (the Attacalite one). |
| 1468 | | |
| 1469 | | 2005-11-12 20:47 acastro |
| 1470 | | |
| 1471 | | * libxc/lda_c_amgb.c: Bug fixed. One wrong character kept me busy |
| 1472 | | the whole afternoon. |
| 1473 | | |
| 1474 | | 2005-11-12 11:25 appel |
| 1475 | | |
| 1476 | | * src/messages.F90: Adapting string lengths in |
| 1477 | | messages_print_stress. Octopus was printing on x86_64 garbage |
| 1478 | | like this |
| 1479 | | |
| 1480 | | ^@^@^PS©^@^@^@^@^@ìËÿ)^A^@^@^@~@S©^@^@^@^@^@PS©^@^@^@^@^@ÐQ©^@^@^@^@^@ |
| 1481 | | |
| 1482 | | to static/info. This was causing grep to assume it is a binary |
| 1483 | | file. Therefore the matches of the testsuite were failing. |
| 1484 | | |
| 1485 | | 2005-11-11 13:00 marques |
| 1486 | | |
| 1487 | | * src/: Makefile.am, casida.F90, epot.F90, epot_inc.F90, |
| 1488 | | geometry.F90, global.F90, grid.F90, gs.F90, h.F90, io.F90, |
| 1489 | | liboct_parser.F90, main.F90, mesh.F90, mesh_init.F90, |
| 1490 | | messages.F90, mf.F90, mpi.F90, mpi_debug.F90, mpi_debug_inc.F90, |
| 1491 | | multicomm.F90, out.F90, out_inc.F90, par_vec.F90, profiling.F90, |
| 1492 | | restart.F90, restart_inc.F90, run.F90, scf.F90, specie.F90, |
| 1493 | | states.F90, states_inc.F90, td.F90, td_calc.F90, td_write.F90, |
| 1494 | | v_ks.F90, xc_KLI.F90, xc_OEP.F90, xc_OEP_SIC.F90, xc_OEP_inc.F90, |
| 1495 | | xc_OEP_x.F90: Some rearrangements (cleaning) in the mpi stuff |
| 1496 | | *) The stuff that was in mpi.F90 is now in mpi_debug.F90 *) Now |
| 1497 | | mpi_mod has low level MPI stuff *) There is now a mpi_grp that |
| 1498 | | handles MPI groups (including a mpi_grp_is_root that, in my |
| 1499 | | opinion, is much more readable than (rank == 0) *) The rest of |
| 1500 | | the code was changed to use this infrastructure |
| 1501 | | |
| 1502 | | 2005-11-11 11:11 acastro |
| 1503 | | |
| 1504 | | * src/poisson.F90: I commit a commented subroutine, poisson_test, |
| 1505 | | in the poisson module. It implements a check for the Hartree |
| 1506 | | integrator, either in 2D or in 3D (it calculates the Hartree |
| 1507 | | potential of a Gaussian charge, and compares it to the exact |
| 1508 | | analytical result). This I didn't do it for 1D because for the |
| 1509 | | soft-Coulomb interaction actually I couldn't think about an exact |
| 1510 | | result. |
| 1511 | | |
| 1512 | | I am not sure about wether or not to put it commented out or not. |
| 1513 | | After all, we are in regression mode. But I wanted it to be in |
| 1514 | | the code after all. It would be nice to have this kind of x_test |
| 1515 | | routines for some of the modules, so that their particular tasks |
| 1516 | | can be checked independently. |
| 1517 | | |
| 1518 | | 2005-11-10 23:24 acastro |
| 1519 | | |
| 1520 | | * testsuite/finite_systems_2d/2dbox.test: I checked the 2D Hartree |
| 1521 | | solver, so I added a test to the 2D file: a qd with 2 electrons |
| 1522 | | interacting with only the Hartree term. |
| 1523 | | |
| 1524 | | 2005-11-09 16:45 acastro |
| 1525 | | |
| 1526 | | * testsuite/finite_systems_2d/2dbox.test: Added a first test for 2D |
| 1527 | | calculations. This is just an infinite potential box example. I |
| 1528 | | will put more as a continue checking the 2D part of the code. |
| 1529 | | |
| 1530 | | 2005-11-09 14:06 acastro |
| 1531 | | |
| 1532 | | * src/: mesh.F90, specie.F90: Just a couple of stupid bugs that |
| 1533 | | showed up in my 2D calculations. |
| 1534 | | |
| 1535 | | 2005-11-09 12:01 marques |
| 1536 | | |
| 1537 | | * src/: eigen_cg.F90, geometry.F90, grid.F90, hgh.F90, |
| 1538 | | messages.F90, multicomm.F90, run.F90, scf.F90, td.F90, tm.F90, |
| 1539 | | v_ks.F90, varinfo.F90, xc.F90: Some more "cleaning" of the output |
| 1540 | | of octopus. Now the variable "stars" disapeared, replaced by the |
| 1541 | | function messages_print_stress, that was given some more options. |
| 1542 | | |
| 1543 | | 2005-11-09 10:13 marques |
| 1544 | | |
| 1545 | | * libxc/: lda.c, xc.h, xc_f.c: *) Alberto, I think that the "break" |
| 1546 | | in the fxc calculation should be a "continue" (i.e., he should |
| 1547 | | skip the rest of the current cycle, but should get out of the |
| 1548 | | cycle). Can you please check? *) Xavier, I guess that the |
| 1549 | | calculation of Kxc should have a similar to the one Alberto |
| 1550 | | put in for fxc. Can you please check? *) I cleaned a little bit |
| 1551 | | the includes, whitespaces, and changed "(*p)." to "p->" as it |
| 1552 | | is more C-like. |
| 1553 | | |
| 1554 | | 2005-11-08 23:01 acastro |
| 1555 | | |
| 1556 | | * libxc/lda.c: Bug fixed. |
| 1557 | | |
| 1558 | | The numerical functional derivation was not working properly in |
| 1559 | | some spin-polarized cases (Perdew-Zunger parameterization, |
| 1560 | | Na4(+1), it was just giving lots of NaN). Now, at least for |
| 1561 | | those cases, it seems to work. I still have to do some more |
| 1562 | | testing on the Casida module, though. |
| 1563 | | |
| 1564 | | In any case, the PZ kernel should be obtained analytically. Maybe |
| 1565 | | in the future. |
| 1566 | | |
| 1567 | | 2005-11-08 16:08 acastro |
| 1568 | | |
| 1569 | | * src/mix.F90: Removed two FIXMEs. |
| 1570 | | |
| 1571 | | 2005-11-08 16:03 xavier |
| 1572 | | |
| 1573 | | * src/: multigrid.F90, poisson_multigrid.F90: All multigrid |
| 1574 | | variables are documented now. |
| 1575 | | |
| 1576 | | 2005-11-08 14:51 acastro |
| 1577 | | |
| 1578 | | * src/: broad.F90, casida.F90: The LinCasida, LinEigenvalues and |
| 1579 | | LinPetersilka variables are gone. The oct-broad utility just |
| 1580 | | broadens whatever spectrum it finds. |
| 1581 | | |
| 1582 | | 2005-11-08 08:24 appel |
| 1583 | | |
| 1584 | | * src/eigen_arpack.F90: Fix compilation issues from last night. The |
| 1585 | | MPI version of the code was not compiling. |
| 1586 | | |
| 1587 | | 2005-11-08 08:18 appel |
| 1588 | | |
| 1589 | | * testsuite/rc_bugs/08-TDELF_C2H4: This bug will probably a bit |
| 1590 | | harder to trace, but I believe it's a real bug. The code stops |
| 1591 | | after 200 SCF cycles with an ** Address Error **. |
| 1592 | | |
| 1593 | | 2005-11-07 20:14 marques |
| 1594 | | |
| 1595 | | * src/: curvlinear.F90, derivatives.F90, eigen.F90, |
| 1596 | | eigen_arpack.F90, epot.F90, f.F90, geom_opt.F90, h.F90, main.F90, |
| 1597 | | mesh_init.F90, messages.F90, mix.F90, multigrid.F90, pes_rc.F90, |
| 1598 | | poisson.F90, poisson_multigrid.F90, restart.F90, run.F90, |
| 1599 | | scf.F90, spectrum.F90, states.F90, states_kpoints.F90, systm.F90, |
| 1600 | | td_exp.F90, td_init.F90, td_rti.F90, v_ks.F90, varinfo.F90: I did |
| 1601 | | some cleaning in the way of handling the input errors (making the |
| 1602 | | use of input_error more consistent), and in the output of the |
| 1603 | | options (using messages_print_var_option throughout the code). |
| 1604 | | |
| 1605 | | There is some need now of "beautifying" the output, by adding |
| 1606 | | some spaces and stars. In any case, from my experience with |
| 1607 | | Silvana, I would be happier if octopus would output more about |
| 1608 | | what it is doing... |
| 1609 | | |
| 1610 | | 2005-11-07 17:40 xavier |
| 1611 | | |
| 1612 | | * src/specie.F90: Updated documentation for Species block. |
| 1613 | | |
| 1614 | | 2005-11-07 16:41 marques |
| 1615 | | |
| 1616 | | * liboct/varinfo.c: Small bug corrected regarding newlines in the |
| 1617 | | varinfo file. |
| 1618 | | |
| 1619 | | 2005-11-07 15:43 appel |
| 1620 | | |
| 1621 | | * src/: gs.F90, lcao.F90, scf.F90, unocc.F90: The 1D periodic tests |
| 1622 | | were not passed anymore on x86_64. Fixing pointer deallocation. |
| 1623 | | |
| 1624 | | 2005-11-07 15:14 appel |
| 1625 | | |
| 1626 | | * src/epot_inc.F90: Remove unused variables. |
| 1627 | | |
| 1628 | | 2005-11-07 15:12 appel |
| 1629 | | |
| 1630 | | * src/global.F90: The new compilerflag feature can produce rather |
| 1631 | | long source lines (depending on the amount of compiler options |
| 1632 | | used). |
| 1633 | | |
| 1634 | | The code didn't compile with NAG, as this compiler doesn't like |
| 1635 | | long lines |
| 1636 | | |
| 1637 | | Error: global.F90, line 181: Line too long - truncated |
| 1638 | | Error: global.F90, line 181: Unterminated character literal |
| 1639 | | |
| 1640 | | 2005-11-07 14:50 acastro |
| 1641 | | |
| 1642 | | * src/curvlinear.F90: The code will now stop if the two non-working |
| 1643 | | adaptive coordinate schemes attempt to be used. They will not |
| 1644 | | make it before the release. |
| 1645 | | |
| 1646 | | 2005-11-07 14:40 acastro |
| 1647 | | |
| 1648 | | * src/: Makefile.am, global.F90, main.F90: I thought that it would |
| 1649 | | be cool if the code outputs what compiler and what compiler flags |
| 1650 | | were used to make the executable. |
| 1651 | | |
| 1652 | | 2005-11-07 10:05 appel |
| 1653 | | |
| 1654 | | * testsuite/rc_bugs/: 01-Hartree_1D, 02-Na_Chain, 03-Na2_MGGA, |
| 1655 | | 04-Na2_Exx, 05-Na2_LB94_Casida, 06-Spinors, 07-Atomic_Gd: This is |
| 1656 | | a set of non-working input files. I've used the format of the |
| 1657 | | test files so that we can use them eventually for the testsuite |
| 1658 | | if they are working properly. The files contain a short bug |
| 1659 | | description and the output of octopus respectively. |
| 1660 | | |
| 1661 | | I decided to add them to the cvs (besides the copy I put in the |
| 1662 | | wiki) so that everybody has easy file access and we have a |
| 1663 | | revision history on the bug tracking process (poor mans bugzilla |
| 1664 | | :). |
| 1665 | | |
| 1666 | | Perhaps some of the features used in these files should be simply |
| 1667 | | disabled for the release of 2.0. In that case view this list just |
| 1668 | | as reminder that we have to stop the code with an appropriate |
| 1669 | | error message. Otherwise we have to look into the bugs ... |
| 1670 | | |
| 1671 | | Some further things I noticed |
| 1672 | | |
| 1673 | | - According to a comment in xc_OEP.F90:57 full OEP seems to be |
| 1674 | | only half |
| 1675 | | implemented. Is the comment outdated, or should we stop the |
| 1676 | | code? The |
| 1677 | | same applies to the Amaldi correction term. In both cases the |
| 1678 | | code seems |
| 1679 | | to run so that the user could assume by mistake that the |
| 1680 | | output is correct. |
| 1681 | | |
| 1682 | | - There are 9 FIXME's in the sources: |
| 1683 | | |
| 1684 | | appel@miraculix:..octopus/src> grep -i fixme *.F90 | wc -l |
| 1685 | | 9 |
| 1686 | | |
| 1687 | | - ... and 31 warnings: |
| 1688 | | |
| 1689 | | appel@miraculix:..octopus/src> grep -i warning *.F90 | grep |
| 1690 | | -v write | wc -l |
| 1691 | | 31 |
| 1692 | | |
| 1693 | | 2005-11-07 10:00 appel |
| 1694 | | |
| 1695 | | * share/util/: oct-atomic_occupations, oct-run_periodic_table: |
| 1696 | | Update the scripts. oct-run_periodic_table has now support for |
| 1697 | | ExtraStates and ElectronicTemperature and oct-atomic_occupations |
| 1698 | | has improved ordering for occupation numbers (4s states appear |
| 1699 | | before 3d states). |
| 1700 | | |
| 1701 | | 2005-11-07 09:59 appel |
| 1702 | | |
| 1703 | | * testsuite/: oct-run_regression_test.pl, |
| 1704 | | finite_systems_3d/multigrid.test, finite_systems_3d/restart.test, |
| 1705 | | finite_systems_3d/so.test: There was a / missing. |
| 1706 | | |
| 1707 | | 2005-11-07 09:58 appel |
| 1708 | | |
| 1709 | | * src/: io.F90, ode_solver_inc.F90, root_solver.F90, |
| 1710 | | root_solver_inc.F90: Documented a bunch of variables. |
| 1711 | | |
| 1712 | | 2005-11-05 16:06 acastro |
| 1713 | | |
| 1714 | | * src/: derivatives.F90, f.F90: The default stencil when using |
| 1715 | | curvilinear coordinates should be the "stencil_starplus" one, not |
| 1716 | | just the star. In fact, the star should *not* be used with curv |
| 1717 | | coordinates, so I made the code stop in that case. |
| 1718 | | |
| 1719 | | 2005-11-04 18:26 acastro |
| 1720 | | |
| 1721 | | * src/run.F90: This is to make sure that the code does not crash if |
| 1722 | | running in parallel but without metis. |
| 1723 | | |
| 1724 | | 2005-11-04 18:12 marques |
| 1725 | | |
| 1726 | | * src/: poisson_cg.F90, scf.F90: poisson_cg1 now works (also in |
| 1727 | | parallel). This is the PoissonSolver = cg |
| 1728 | | |
| 1729 | | 2005-11-04 17:29 marques |
| 1730 | | |
| 1731 | | * src/: casida.F90, epot.F90, epot_inc.F90, geometry.F90, |
| 1732 | | global.F90, grid.F90, io.F90, liboct_parser.F90, main.F90, |
| 1733 | | mesh.F90, mesh_init.F90, messages.F90, multicomm.F90, |
| 1734 | | par_vec.F90, profiling.F90, restart_inc.F90, scf.F90, specie.F90, |
| 1735 | | states.F90, states_inc.F90, systm.F90, td_write.F90, v_ks.F90, |
| 1736 | | xc.F90, xc_KLI.F90, xc_vxc.F90: *) Forces wre not working in |
| 1737 | | parallel. Now they seem to work. |
| 1738 | | |
| 1739 | | *) Sorry, Alberto, I had also fixed the xc. I changed your |
| 1740 | | solution, for the following resaon: I think it is much more |
| 1741 | | clear to use the mf_integrate function than to call MPI |
| 1742 | | directly. The disadvantage is that this is (maybe) slower and |
| 1743 | | more memory consuming (although temporarely). However, I think |
| 1744 | | it is worth it. The same applies to the forces. |
| 1745 | | |
| 1746 | | *) I rename the variables in mpiv in global. I think it is more |
| 1747 | | clear now. I will probably create a module called mpi_group, |
| 1748 | | or something like that to contain rank/size/comm to be used |
| 1749 | | consistently in the code. |
| 1750 | | |
| 1751 | | 2005-11-04 17:06 acastro |
| 1752 | | |
| 1753 | | * src/: xc.F90, xc_vxc.F90: The exchange and correlation energies |
| 1754 | | were badly calculated when using the parallelization in domains. |
| 1755 | | |
| 1756 | | 2005-11-04 16:32 acastro |
| 1757 | | |
| 1758 | | * src/states_inc.F90: Sorry about the last commit, I just |
| 1759 | | replicated Migues's bug fix. |
| 1760 | | |
| 1761 | | 2005-11-04 15:23 acastro |
| 1762 | | |
| 1763 | | * src/states_inc.F90: Bug fixed. When running in parallel in |
| 1764 | | domains, the code crashed at that point because the state |
| 1765 | | communicator is null. |
| 1766 | | |
| 1767 | | 2005-11-04 09:47 rozzi |
| 1768 | | |
| 1769 | | * src/: epot.F90, out.F90, states.F90, states_kpoints.F90: A |
| 1770 | | contribution to the documentation of the variables. |
| 1771 | | |
| 1772 | | 2005-11-04 08:59 marques |
| 1773 | | |
| 1774 | | * src/states_inc.F90: Small bug corrected in the parallel version. |
| 1775 | | |
| 1776 | | Alberto, the Na2 ground-states seems to work in my machine. |
| 1777 | | You'll have to use the cg_corrected Poisson solver (the cg |
| 1778 | | crashes), and decrease the PoissonSolverThreshold to 1e-7. Note |
| 1779 | | that using the fft solver does not stop octopus with an error, |
| 1780 | | but it gives non-sense!!! |
| 1781 | | |
| 1782 | | 2005-11-04 08:35 appel |
| 1783 | | |
| 1784 | | * src/td_rti.F90: Fix compilation problems. Same story as |
| 1785 | | yesterday: Don't put single quotes ' in the documentation. This |
| 1786 | | causes the compilation of the code to fail ... |
| 1787 | | |
| 1788 | | 2005-11-03 18:43 marques |
| 1789 | | |
| 1790 | | * build/var2html.pl, doc/octopus.info, doc/octopus.texi, |
| 1791 | | share/variables.local, src/curv_gygi.F90, src/geom_opt.F90, |
| 1792 | | src/h.F90, src/hs.F90, src/io.F90, src/out.F90, src/pes.F90, |
| 1793 | | src/pes_rc.F90, src/phonons.F90, src/poisson3D.F90, src/scf.F90, |
| 1794 | | src/spectrum.F90, src/static_pol.F90, src/systm.F90, src/td.F90, |
| 1795 | | src/td_exp.F90, src/td_init.F90, src/td_rti.F90, |
| 1796 | | src/td_write.F90, src/unocc.F90, src/v_ks.F90: I finished putting |
| 1797 | | the documentatio of the variables. I will now make a list of |
| 1798 | | undocummented variables. |
| 1799 | | |
| 1800 | | 2005-11-03 15:37 appel |
| 1801 | | |
| 1802 | | * share/util/: Makefile.am, oct-atomic_occupations, |
| 1803 | | oct-run_periodic_table: There's now a utility to compute the |
| 1804 | | atomic occupations and ExtraStates for all Pseudopotentials that |
| 1805 | | ship with octopus: |
| 1806 | | |
| 1807 | | &nb |