GitLab

Merge branch 'develop' into svn: using caches to speed up sc calculation (mainly relevant in gun calculations with many particles)

In OPAL several different TRACK's can performed within one simulation (does not hold for OPAL-cycl yet) this together with 
the restart capabilities calls for a delicate logic.

Different tracks, looks like: 

.....

TRACK, LINE= Injector, BEAM=beam1, MAXSTEPS=100, DT=1.0e-12, ZSTOP=0.14;  
 RUN, METHOD = "PARALLEL-T", BEAM = beam1, FIELDSOLVER = Fs1, DISTRIBUTION = Dist1;
ENDTRACK;

TRACK,LINE= Injector, BEAM=beam1, MAXSTEPS=50, DT=1.0e-11, ZSTOP=0.20;
 RUN, METHOD = "PARALLEL-T", BEAM=beam1, FIELDSOLVER=Fs1;
ENDTRACK;
STOP;

A run with the arguments -restart -1 will continue a track from the last dump in the h5 file. Other the from the last track will
at the moment certainly fail in the general case. 

Changes: 

- The h5 attribute TrackStep is not longer available anymore. Two new attributes: LocalTrackStep and GlobalTrackStep are introduced instead.

- LocalTrackStep refers to the track step in a track command (0 ... 99 and 0... 49 in teh above example), where the GlobalTrackStep is the 
accumulated number of steps i.e. 0 ... 149, shown in OPAL's screen output. Both local and global track steps are also written into the h5 file 
in order to properly restart the simulation.

- The PartBunch object has get/set and increment functions:

    inline void setGlobalTrackStep(long long n) {globalTrackStep_m = n;}
    inline long long getGlobalTrackStep() const {return globalTrackStep_m;}

    inline void setLocalTrackStep(long long n) {localTrackStep_m = n;}
    inline long long getLocalTrackStep() const {return localTrackStep_m;}
    inline void incLocalTrackStep() {localTrackStep_m++; globalTrackStep_m++;)

        
- In OPAL-cyc getTrackStep is replace by getLocalTrackStep assuming we only have one TRACK in the simulation.

- In the ParallelTTracker at the end of the main loop (in execute()) obsolet code is removed (cleanup monitors etc.)
  
For details see the diff :-)

Still some work todo: 

a) Need to remove: setLastStep in ParallelTTracker.cpp and in general

    getInstance()->getLastStep()
    getInstance()->getRestartStep() maybe not!

b) The get/set and increment functions could very well be in an other class!

c) ParallelPlate tests are broken but maybe also because of other changes need to fix this 
   together with Chuang and Achim

d) Make sure the restart works in OPAL-cycl we do not have yet a regression test for that.

Merge branch 'develop' into svn: more pretty printing; change variable names to comply with (unfortunately missing) coding style (Use descriptive names)

Merge branch 'develop' into svn: willfully breaking Scan-1 test; the first track starts with step 0 and so should the following tracks in a scan

Merge branch 'develop' into svn: fixing Scan-1 test. Interestingly this has nothing to do with any of the changes of yesterday, still the test was broken today but not before

- use forward declaration where possible
- remove unused includes

Merge branch 'develop' into svn: fixing long standing issue with number of steps performed with track

Merge branch 'develop' into svn: get rid of half of the remaining warnings in opal (without pragmas), disable the other half with pragmas;
 - move saving of maxSteps_m inside autophasing method
 - when using mpicxx not with its default compiler then the cmake file should still work. extract the absolute path to the underlying compiler and its name

Workaround, because doAutophase is changing the maximum number of steps we defined in the TRACK cmd.

Merge branch 'develop' into svn: fixing autophase by merging the correct traveling wave list into cavities_m

Merge branch 'develop' into svn: new ParallelTTracker, refactored from the old. Now default is to have units. Still lot of places that need to be cleaned up and/or fixed.

Add std:: to a bunch of string statements.