possible bug in fix_atom/swap

Hi,

Please reply to the list.

The up to date code can be seen here:
https://github.com/lammps/lammps/blob/master/src/MC/fix_atom_swap.cpp#L361

You should redo "make yes-mc" after pulling.

Also, I didn't commit that change :slight_smile:

Pierre

yes ,
well ... no commit no exist…
Indeed i now see it in the svn.

yinon

yinon,

it would be *extremely* helpful, if you would post less cryptic
e-mails. in particular,

- always report which version of LAMMPS _exactly_ this is in reference
to and what platform (OS, architecture, compiler) you are running on.

- when you suspect a problem in the code, please provide an easy to
run and complete input deck that can reproduce the problem, and
explain what is seen and what should be seen instead

- always verify if the problem persists with the latest development
version from the svn or (by preference) git repository.

- if you suggest a modification to the code, please provide a proper
patch with the proposed modification relative to the latest
development sources. the output of "git diff" or "svn diff" would be a
good starting point. when using the standalone diff program, please
use the -u or -c option to produce a diff file with some context,
which makes it infinitely easier to be merged in case there are some
unreleased local modifications

- when describing what you are looking at and what you conclude,
please remember, that nobody on the list can look over your shoulder,
so be specific and use complete sentences.

in total, the less people here will have to guess, the better are your
chances that people with take you seriously and will consider what you
have to say.

thanks for your understanding,
      axel.

ok
all is well (in this respect) with the latest version of lammps.