Merge "Expanded the help text regarding the -r2 and -ac options." into release-4-6

Expanded the help text regarding the -r2 and -ac options.

Fixes #862

Change-Id: If7ff131648881da8b1749bc736e141ff0b307ca6

Restart IO operations when interrupter by signal

Fixes #798

Change-Id: I762412abf8138bbbe3d386d015e6af3522bd5235

Reworked FindFFTW.cmake

* make user input FFTW_{LIBRARY|INCLUDE_DIR} work and advanced
* ade more documentation
* reconsider test if failed in the last run
* improve error messages to the user
* check if fftw is compiled with sse support
* fixed detection on Cray

Change-Id: I0b7d440d372da7cf897ca956ccd882e21b109591

Introduce include/gmxconfig.h

This header file is for #defines that should be available in an installed
header file that can be used by projects that depend on GROMACS. The
src/config.h used in building GROMACS is not installed, and so is not
available to such projects. CMake is used to configure
include/gmxconfig.h.cmakein and create include/gmxconfig.h.

Change-Id: Ide36fb4c0958d5faa647263411b3cfb91baa1d61

Merge remote-tracking branch 'gerrit/release-4-5-patches' into release-4-6

Conflicts:
CMakeLists.txt

Change-Id: I67d65e260f68442841d8b901d77780b8d1505340

Merge "Fixes problems with g_msd and g_energy" into release-4-5-patches

Fixes problems with g_msd and g_energy

* Fixes a problem where g_msd will stall on reading a file
  with xy periodic boxes.

* Fixes a problem with gmx_energy ignoring the -skip option.
  Was correct in 4.0.7, but was wrong by 4.5.5. Redmine issue #910

Change-Id: I4247a4bd1cf1ab15ea6157646c97452216a887aa

Merge "Improved pdb2gmx error message" into release-4-5-patches

Improved pdb2gmx error message

Increased the extent to which pdb2gmx makes use of the encapsulation
available with the t_atoms struct. Now the function that writes the
error message has an easier time being more descriptive of what
is wrong when the atoms in the input coordinate file do not match
those of the .rtp entry.

Change-Id: Ie39dea18efa358f90decad6491ddc04e3756ac39

Keep CMake quiet about a Cygwin issue

The CMake warning that is suppressed by this does not affect GROMACS
builds.

Change-Id: I6a19f55868b6363cb2be328286b4716659cc270b

Merge branch 'release-4-5-patches' into release-4-6

Change-Id: Id4fc52b784ac55c2e57669af391b8d2a70a56695

Merge "Resolving clang analyzer warnings in analysis" into release-4-6

Resolving clang analyzer warnings in analysis

Backporting fixes in selection and trajana from
  9b0c1478b93867550d4a126421bf85d35ad8c0e0

Conflicts:

src/gmxlib/selection/compiler.c
src/gmxlib/selection/selmethod.c
src/gromacs/fatalerror/gmxassert.h
src/gromacs/options/tests/abstractoptionstorage.cpp
src/programs/g_ana/g_ana.cpp

Change-Id: Ia3a74d5079b2e5dccac4499de9c4b65fdd3868e1

added correction for PME virial with non-linear virtual sites

With non-linear virtual site constructs the virial nees to be corrected
when it is not calculated from the particle coordinates and masses.
Currently this is only a problem for the PME mesh contribution.
This would lead to an incorrect virial and pressure. The error was small
for unordered systems, but could be significant for e.g. an all-atom
bilayer with hydrogens replaced by virtual sites. Fixes #908

Change-Id: I27a06019aa09734a14f3630ef296aaaacb9c5f0b

Warn when a value is unused

Change-Id: I306062c791daf41b0791d0609c666af1b2910524

fixed g_bar xvg reading issue

Due to a comparison between temperatures in single and double precision
g_bar could give a fatal error when processing xvg files.

Change-Id: Ie63187261b3f20f8e241bf606cc9f23fdd93ec2b

Removing unused code

Change-Id: Ic3eb509b2438c2dc5b022aed0974a400e75e8e02

Fix incorrect fclose usage

fclose should not be used files opened with
- ffopen
- xvgropen
- gmx_fio_fopen
- fflib_open

Change-Id: I18d8e506f2763812d8193a92c67f25028e450cc1

Merge "Warnings against equilibrating with P-R P-coupling" into release-4-5-patches

Merge "Fix typo in Clang CFlags" into release-4-5-patches

Warnings against equilibrating with P-R P-coupling

Hopefully this will inhibit some people from doing Parrinello-Rahman
pressure coupling during their equilibration stage. If there's a
legitimate reason for P-R to have volume changes greater than 1%
we can reconsider the verbosity and/or frequency of the warning.

Change-Id: Icaeafa2fad37dfaa4b63ba69be3a3f804d957e7c

Merge "Merge release-4-5-patches into release-4-6" into release-4-6

fixed increasing memory usage with SHAKE and DD

With domain decompostion SHAKE would be reinitialized every DD step.
This is now only done at initialization. Fixes #899

Change-Id: I4bcb162fa97271f602c5733ba04c7314456fcfa4

Removed qhop files from this branch.

Change-Id: I7b6038571caaf47313d3edcf7111b3d2d0a3420c

fixed incorrect table allocation

This caused problems after commit 553d4b13.
The bug was already in older versions, but was hidden due to
the over allocation for pointer alignment.

Change-Id: Ifc50ee66fd4e963aab20ff1231c90a472fd5c443

Merge release-4-5-patches into release-4-6

Change-Id: I5a7f695d49e3b7d1a5b4a0aeb099e5d3cc100faa

Merge "Clarified wording of documentation" into release-4-5-patches

Fix most errors found by cppcheck

While some are only style issues (e.g. include "" instead of <>),
it includes a couple of real bugs.
- Not correct handling of '\0' termination of string
- assignment in if statement
- using of variable before checking that it isn't null

Warnings from a couple of non-critical categories aren't fixed.

To run cppcheck (tested with latest 1.53):
cppcheck -j8 `find src -name '*.c' -o -name '*.cpp' -o -name '*.cu'` \
   -DHAVE_CONFIG_H -DSIZEOF_LONG_LONG_INT=8 -DSIZEOF_INT=4 -DLINUX -I include \
  --enable=style \
  --suppress=variableScope --suppress=unusedVariable --suppress=unreadVariable \
  --suppress=invalidscanf --suppress=unusedStructMember --suppress=sizeofCalculation \
  --suppress=unnecessaryForwardDeclaration --suppress=*:src/gmxlib/gmx_lapack/* \
  --suppress=*:src/contrib/*  --suppress=missingInclude \
  --template gcc  --inline-suppr

Change-Id: I1eabfce3e6cf8a6bed0f1282868667e290159279

Clarified wording of documentation

Change-Id: I16df3e9db672f86396a22479563dc394035af5ce

fixed appending OpenMP compiler option to flag list

GROMACS_C_FLAGS and GROMACS_CXX_FLAGS is treated as a string and
therefore appending to it using list append operation will result in an
extra semicolon.

Change-Id: I39f768b48bb8c029f8c7b21f0fc3c5e8c2309199

Changed cryptic error messages about disk quota

People who have never had a disk quota don't understand what "out
of quota" could possibly mean. They understand running out of
disk space much more often.

Change-Id: I988cd038e74cf4e914075a748f9f66038ab37882

Merge "mdrun -npme without PME or with PD now generates a fatal error" into release-4-6

Merge "Vibration spectra from velocity ACF or Normal modes." into release-4-6

mdrun -npme without PME or with PD now generates a fatal error

Change-Id: I887f2030e76a2932afd2989e17d7f14acefd18b1

Merge "More editorial fixes to tool documentation" into release-4-5-patches

Merge "Made description string for g_select -select clearer" into release-4-5-patches

Vibration spectra from velocity ACF or Normal modes.

The velocity autocorrelation function can be converted into
a vibrational spectrum by performing a fourier transform
and plotting the amplitude squared as a function of
frequency or wavenumber. This has now been automated in
g_velacc with the -os flag. Likewise, an option -os
has been added to the g_nmeig program to produce a spectrum
based on the harmonic approximation.

Change-Id: I7e3d669b9108e3a91856ff20688f6216a035bf4a

Activating more GCC warnings

- Removing very odd "else" in assert to remove empty body warning
- Ideally we wouldn't need to exclude:
   - missing-field-initializers: only first 4 fields of t_filenm
     are initialized in all places. Not easy to change.
   - sign-compare: used in several places. Not sure worth to fix.

Change-Id: I137482c8c0cbdd75b6406bc6c61cffdd45af2ad8

Fix typo in Clang CFlags

Change-Id: I4b03e585fd3f6f5d256478feef5038b2d7d13cba

Merge "Improve FindFFTW" into release-4-6

Merge "Improved cryptic error message" into release-4-5-patches

Merge "Improved warning note with DD and periodic molecules" into release-4-5-patches

More editorial fixes to tool documentation

Needed to add better treatment of <averages> and
|magnitudes| for LaTeX versions.

Change-Id: I5e8e3a8d685715af38117a762327d41a8bedf391

Editorial changes to tools documentation

Made LaTeX type-setting of maths in documentation of tools more
sophisticated, without changing the presentation of maths in
plain text mode.

Various spelling and typographical fixes also.

Change-Id: Ia0c13e1f2ba825911c6e0673af6b940aba9efa21

Make finite checks remember theirs results

* saves time on rerun of cmake
* makes cmake quiet on rerun

Change-Id: I2120d8f13d2b7c5baa52d5aa8f06d6eb9ca2bb26

Improved warning note with DD and periodic molecules

Change-Id: I3f7f201edcce1b70a360d0462e5103dfc144b2d7

Made description string for g_select -select clearer

New UNIX users wouldn't guess they had to enclose the selection
string in quotes to get around their shell.

Change-Id: I8dd7fdd237e4290ad830a168b4880739aa5dc425

Improved cryptic error message

Change-Id: Idd3f7a9ce36e155c61009b16da9a1f2b1a98e93e

Improve FindFFTW

* mark internally used variables INTERNAL
* make pkg-config search quiet on 2nd run to avoid confusion

Change-Id: I8b91adea1af8a14dffb88438fb2620df738f5a76

Merge "Minimal support for Clang" into release-4-5-patches

fixed g_anaeig incorrect multiple extremes, fixes #870

Change-Id: Id06f267c1da300f2a940778833ec35b85fecd3f1

added check for mass-weighting with g_anaeig -entropy, fixes #785

Change-Id: I7a1623ba7a7c7987078901876fb4717d8c0a7261

removed initial temperature with OpenMM, fixes #757

Change-Id: Ifc661ce3ece3dfcd2e4bad862392b96d64e70c18

Minimal support for Clang

Change-Id: I4c6fb22ef8b5d596527eb8fe0991c3c573be606c

[tools] Move common functions from gmx_sans.c to nsfactor.c

This commit moves common functions to gmx_sans and gmx_nse to nsfcator.c

Change-Id: Ifcf8e87203dd8721bb91807cd96d0e21f797b918
Signed-off-by: Alexey Shvetsov <alexxy@omrb.pnpi.spb.ru>

Merge branch 'release-4-5-patches' into release-4-6

Also fixed uninitialized variable warning in pdb2top.c with gcc 4.3.

Conflicts:
share/html/online/mdp_opt.html

Change-Id: I8e499a77a3ce71c2063f453348330e2aad38669b

fixed array index out of bounds in EM force trr storage

The partially implemented force copy was a leftover from a different way
of copying, which should never have been committed.

Change-Id: I026aa79158afcc327185f2f0cf6167d040eccede

use separate PME nodes with #nodes>18

PME nodes were used with #nodes>=10, but with 2D PME decomposition,
introduced already in 4.5, PME nodes are not faster with 12 or 16 nodes.

Change-Id: I9c5c8ac225df9d4b81f451074b74d35f002205ee

Merge "Minor formatting fixes to .mdp option documentation" into release-4-5-patches

Minor formatting fixes to .mdp option documentation

Change-Id: I71762463880e4cb650f680a2ed83125e450f003a

Merge remote-tracking branch 'gerrit/release-4-5-patches' into release-4-6

Conflicts:
admin/GerritBuild.bat

Change-Id: Ie04e9e5932aa2a154712a17efb03538648b4b817

added tpx file tag string

The tpx file now contains a tag, such that feature branches can increase
the tpx version without the risk of later having tpx files with the same
version number but different format which can not be distinguised on read.

Change-Id: I07e804133bb1e55de6f25fd355bd79a7cb2fe141

Fix pdb2gmx -cmap in corner cases

CMAP torsions are only generated when the atoms are from the same chain
(pdb2gmx -merge allows multiple chains to be present in the
moleculetype), and their residues are in residuetypes.dat as
"Protein".

Fixes #800, #885

Change-Id: Idc4f9a867beedc0be6a4c153606c0731eec86f5f

Fixed ABI warning for thread_mpi on gcc 4.6 for x86.

Atomic operation data structures are now padded to the size of a cache
line instead of aligned to the start of a cache line. This works just
as well, but eliminates a warning in gcc 4.6.

Change-Id: I8d00778dcfea41cdb518b083a9c0732e8d978c66

Merge "Use Git server (mirror) to avoid fetching problems" into release-4-5-patches

Use Git server (mirror) to avoid fetching problems

Change-Id: I144082bd01cf6a5f0583feac0e7877e8e1b163ff

Merge "changed default epsilon-rf value to 0" into release-4-6

changed default epsilon-rf value to 0

The default for epsilon-rf used to be 1, now it has been changed to infinity.
Infinity is a sensible value and gives 0 potential and force at the cut-off.

Change-Id: I3884085b687002edff4b4b96f21851e182268a55

Merge "Generate warnings for QMMM usage" into release-4-6

Generate warnings for QMMM usage

Change-Id: Ic6088f765cb0bfbb6f85e3da53421ef6a2167d94

Removed FFTw2 support

Change-Id: I04a702577ceb9f15ed9ec3fdc83bed0caef0fda1

Merge "Cmake: unite FindFFTW3* into one file" into release-4-6

Fixed an allocation issue with dH histograms.

Change-Id: Ifc6ed94d77664e3b3afb7fe8dbd1016444d86bc3

Merge changes If0c2f9d9,Ic2a8ad82 into release-4-5-patches

* changes:
  Fixed g_sham using more than three dimensions
  Integer multiplication that checks for overflow

fixed gmxcheck segv with tpr files without AdResS

Change-Id: If337862e4247b3ecada73f5d0cec20f27a0d9a50

Fixed g_sham using more than three dimensions

When the value given with g_sham -n was greater than 3, arrays used to
overflow and pick_minima() did not work. pick_minima() has been updated
to treat an arbitrary number of dimensions, but retains the particular
code for the two- and three-dimensional cases in the hope that these
are faster. The logic of the complex conditionals has hopefully been
made easier to follow without compromising performance with modern
compilers. Index variables are now of gmx_large_int_t type, as
high-dimensional cases can have large numbers of grid points very fast.

Change-Id: If0c2f9d9ceaf2b5c4c8b1a28a942fae8349fb600

Integer multiplication that checks for overflow

Integer multiplication overflows silently in C, and sometimes we need to
know when it would. For example, when computing the size of
multi-dimensional arrays, we can need to manage the case when the number
of dimensions and their size is too large to represent in the available
type.

Change-Id: Ic2a8ad820838cf314bb985511e483fc397ba35e2

Merge "Fixed bug with COM motion removal on part of the system" into release-4-5-patches

Cmake: unite FindFFTW3* into one file

* easier to maintain
* leave space to add support for fftw2
* leave space to add support for other modules like fftw_omp

Change-Id: I30e1f7a6fff26b924b29fe37a34a8ce5c5177153

renamed GMX_THREADS to GMX_THREAD_MPI

The aim is to better differentiate between the two types of threading:
OpenMP and thread-MPI.

Both CMake cache variable and preprocessor definitons were renamed.

Change-Id: Ie905a572cc9c02cacdcce6f80497a078131d253e

Merge remote-tracking branch 'gerrit/release-4-5-patches' into release-4-6

Change-Id: I13898484048384500e946fc4dddaaca3281a8819

Merge "fixed confusing nonbonded kernel pointer initialization" into release-4-6

Fixed bug with COM motion removal on part of the system

Center of mass motion removal on one group (not multiple) which is only part
of system would result in a tpr file with COM removal on the whole system.
Fixes #881

Change-Id: I181427bcad237045060000908f2c28d6e97c1ea4

Fix ICC on Windows Build

Fix NASM with NMake Generator

The workaround in parser.y may not be necessary any longer when this is
merged into master.

Change-Id: I810eec3fa4a13fd912da3832279e8ab47a62bdec

Merge remote-tracking branch 'gerrit/release-4-5-patches' into release-4-6

Change-Id: I825d441cf0d13d53ea8b460cb97e2e7775f2e8ce

Merge "Improved the accuracy of the sd1 integrator" into release-4-6

Merge "Append all ICC Performance flags only to Release Flags" into release-4-5-patches

Merge "Merges in threading branch as one squashed commit" into release-4-6

fixed confusing nonbonded kernel pointer initialization

Change-Id: I590c7c08d77491f05c798ccc4abf4de988b950f5

Merges in threading branch as one squashed commit

Implements OpenMP threading for PME

All PME (including SSE) code by Berk.
FFT, timer, and CPU affinity code by Roland.
Two bugfixes by Szilard.

Conflicts:
include/gmx_wallcycle.h
src/gmxlib/trxio.c
src/mdlib/Makefile.am
src/mdlib/fft5d.c
src/mdlib/gmx_wallcycle.c
src/tools/acf.test.c

Change-Id: I4bdd0e29caf135ba6b4cc7297640f29f0d136656

Merge "Revert "Fix ICC on Windows Build"" into release-4-5-patches

Improved the accuracy of the sd1 integrator

There was a small error in one of the coefficients of the sd1 integrator.
Correcting this makes sd1 as accurate as sd, unless constraints are used.

Change-Id: I2f71a80cc0487d92fd151c6ea4f243feb228626c

initialize adress_do_hybridpairs=FALSE by default

fr->adress_do_hybridpairs is evaluated in nonbonded.c:1028 even if
adress is actually off, so it could be used uninitialized, set it
to false for this case.

Change-Id: I6868e7ecc5c484c95483f5bc94f8205350e39028

Merge "g_sans - tool to compute SANS spectra" into release-4-6

Merge "Fixed error message to identify the correct file" into release-4-5-patches

Merge "Fixed trjconv -sub" into release-4-5-patches

Merge "Improved error message" into release-4-5-patches

Fixed trjconv -sub

If the index group contains 1..100, then clust->maxframe == 99,
and thus when frame == clust->maxframe it should be included
in the cluster.

Change-Id: I6e093d9f76d0952b31ea9ee1516bade9e9b4419b

Fixed error message to identify the correct file

When a frame exists, that file has *not* run out of frames.

Change-Id: Ief52d5a17ef73ab318f6d8e00d1f5acb71b69f52