-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
merge DDalphaAMG_nd branch into etmc/tmLQCD/master #423
Merged
Merged
Changes from 78 commits
Commits
Show all changes
104 commits
Select commit
Hold shift + click to select a range
5b0ba74
Working interface to DDalphaAMG TM2p1p1
sbacchio 7883d32
Merge branch 'master' of github.com:Finkenrath/tmLQCD into DDalphaAMG_nd
sbacchio 2976c3a
Updating to MGMMSND
sbacchio 4a6a69d
Bug fix
sbacchio 63407e7
Merge branch 'master' of github.com:Finkenrath/tmLQCD into DDalphaAMG_nd
sbacchio 9da5d6d
Merge branch 'master' of github.com:Finkenrath/tmLQCD into DDalphaAMG_nd
sbacchio 8ba0bd3
Optimized smoother iterations for MG 1+1
sbacchio ccad26e
Added mixed approach with DDalphaAMG and CGMMS. Use the parameter MGN…
sbacchio e6bab07
Checked correction term for the rational approximation. Improved the …
sbacchio c4510cf
Fixed reweighting_factor.c in order to work for reweighting the corre…
sbacchio 04a8b9d
Merged Finkerath/master
sbacchio 57d375a
Added estimation of next order correction in monomial/ndratcor_monomi…
sbacchio 6be1fc9
Improved computation of heatbath with MG. TESTED.
sbacchio c1b8bdb
Making DDalphaAMG working for inverting doublet operator within inver…
sbacchio f34eda7
Master merged
sbacchio 800e96f
Fixed some merging issues
sbacchio 7edd999
Fixed checking for ndratcor_monomial heatbath. Added documentation
sbacchio c7d3874
Added mms_squared_solver_prec to solver_pm_t structure. Defined its u…
sbacchio 3a1afd8
Using different tolerances for the shifts in the rational approximati…
sbacchio dfc9789
Add flag SPERIMENTAL around new changes in rat monomial
sbacchio 1c7b982
Merge pull request #9 from Finkenrath/tol_array
sbacchio 8434a00
removing spurious changings
sbacchio 5f01c73
Bug fix: cg_mms wasn't using relative residual for the shifts.
sbacchio fd0ca8c
Bug fix in the scaling of the operator in the rational approximation
sbacchio af9e3a5
Enable initial guess in rg_mixed_cg_her_nd
sbacchio 78dcd46
Rename parameter from mg_no_shift to mg_mms_mass.
sbacchio c8d3e3d
Using initial guess for rgmixedCG in mms solver
sbacchio 5188b6d
Add convert_even_to_lexic functions
sbacchio 2c77d9e
Add Laplacian intial guess for MG and RG-MIXED-CG when are used as MM…
sbacchio 44e2527
Bug fixes and restoring MGNumberOfShifts
sbacchio 462c9fa
Add initial guess to single inversions done in ndrat heatbath
sbacchio b897761
Comment not needed test
sbacchio 90cb924
Merge branch 'hotfix' of github.com:Finkenrath/tmLQCD into DDalphaAMG_nd
sbacchio 0d57268
Bug fix in allocation introduced by previous commit.
sbacchio 15c6edb
Add full clover-improved operator
sbacchio c6d8d46
Add function square_and_max for spinor
sbacchio 1b2977f
Add min and relative norm to square_and_minmax function (renamed from…
sbacchio 6157c1b
Bug fix in freeing memory
sbacchio a8e88e4
Changed square_and_max to square_and_minmax
sbacchio 0b5adda
Add choice of mms solver for the RHMC with Wilson
sbacchio 4361344
Merge branch 'master' of github.com:Finkenrath/tmLQCD into DDalphaAMG_nd
sbacchio d739949
Move initial guess to separate function
sbacchio b0d4aaa
Add inverson for single operator in monomial
sbacchio e0b2653
Removing mms_squared_solver_prec. Not needed at the end.
sbacchio 0e5c2b6
Done FIXME
sbacchio 777edf4
Missed declaration of solve_mms_nd_plus
sbacchio 5d7ef13
Done FIXME part 2
sbacchio 578bfbb
Removing spurious changings
sbacchio 4d61d9b
Fixing indentation and removing spurios changings
sbacchio a244d08
Replacing tab with spaces
sbacchio 0c00d7c
Add initial guess from CGMMS
sbacchio 6b36878
Replacing flag OMP with correct one TM_USE_OMP
sbacchio 15b58ec
Add SWAP macro to global
sbacchio 8e3bc1f
Improving printing in initial guess
sbacchio 986aad7
Bug fixes and some additional improvements
sbacchio d99e87a
Add variant for shifted operator.
sbacchio 1930301
Merge branch 'master' of github.com:etmc/tmLQCD into DDalphaAMG_nd
sbacchio e7d64b0
Enabling the use of QphiX
sbacchio 7388975
Restoring residual check
sbacchio b5f3c01
Running MG with DDalphaAMG as standard
sbacchio 2835afa
Adding time printing to cg_her_nd
sbacchio 5d50f9b
Adding clover operator to nddetratio
sbacchio 9f2b568
Enabling multiple measurements for CORRELATOR
sbacchio 94f1f9a
Adding NDCLOVERDETRATIO to monomial
sbacchio 5d8d466
Some changes to read_input
sbacchio e64478e
Replacing MGTEST with debug_level > 2
sbacchio 22f557f
Adding extra restoresu3
Finkenrath 86a49aa
Making exposu3 and restoresu3 function similar to openQCD
Finkenrath 867b24a
Adding reweighting option for testing the restore of su3
sbacchio c46d5ef
Merge branch 'master' of github.com:etmc/tmLQCD into DDalphaAMG_nd
sbacchio 43b6963
Merge branch 'integrator' of github.com:Finkenrath/tmLQCD into DDalph…
sbacchio ed55735
Adding extra restoresu3
Finkenrath da53162
Solving undefined reference in case of compilation without DDalphaAMG
sbacchio 94b4494
Making exposu3 thread safe
sbacchio d80023c
Merge remote-tracking branch 'etmc/master' into DDalphaAMG_nd
kostrzewa 551b949
in update_gauge, init_exposu3 must be called OUTSIDE of the parallel …
kostrzewa b323134
Merge pull request #12 from kostrzewa/DDalphaAMG_nd_fix_thread_safe
sbacchio 6287103
Merge branch 'DDalphaAMG_nd' of https://github.com/Finkenrath/tmLQCD …
kostrzewa b134082
Bug Fix, fix in expo.c for 2tr(X)>1, fix in ndratcor_monomial.c the e…
Finkenrath d206e97
Hack for allowing the multishift inversion using QphiX AND DDalphaAMG…
sbacchio 4bae01c
Merge branch 'DDalphaAMG_nd_merge_etmc_master' of github.com:etmc/tmL…
sbacchio aa2de0a
Removing space after new line in macro
sbacchio 040d7d8
Merge branch 'master' of github.com:etmc/tmLQCD into DDalphaAMG_nd_me…
kostrzewa 35fb270
in parsing the DDalphaAMG input parameters, clarify the intent when p…
kostrzewa 825f3bb
Fixed typo
sbacchio 7a52cb9
Updated documentation of DDalphaAMG with new parameters.
sbacchio 27445a9
Fixing read_input
sbacchio 92beb93
Giving init_exposu3 a better place in the world.
sbacchio 5d65fec
Adding solver parameter for initial guess
sbacchio 011bf6b
Adding return of -1 in case of non-cnvergence
sbacchio 8b03779
Adding kappa2 for NDDETRATMONOMIAL,NDCLDETRATMONOMIAL
sbacchio 527f13c
just typos
sbacchio 770abe1
removing CGMMS from MSOLVER
sbacchio fe4bef4
introduce multi-threaded update_momenta_fg
kostrzewa 52bd747
Merge pull request #13 from kostrzewa/DDalphaAMG_nd_update_momenta_fg…
sbacchio fe296d6
remove a commented out line in update_momenta_fg
kostrzewa e997266
use fatal_error in monomial_solve.c
kostrzewa 0ebb357
linkage of addup_ddummy was wrong
kostrzewa 9a5f4ad
monomial_solve: perform residual check from debug level 2 onwards
kostrzewa 0329089
Revert "monomial_solve: perform residual check from debug level 2 onw…
kostrzewa 876cc61
remove 'restoresu3_flag' from meas/correlators.c
kostrzewa 1b65403
some illegal memory accesses in meas/correlators
kostrzewa 9a05fc4
Merge remote-tracking branch 'origin/master' into DDalphaAMG_nd_merge…
kostrzewa 42356fc
remove restoresu3_flag
kostrzewa File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Finkenrath @sbacchio
There is no default set for this anywhere, if I'm not mistaken. This should be done in read_input as for the other global input paramters.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was just a check for computing the violation of the special unitary of the links due to the bug in restore su3.
We applied reweighed the action of some old ensembles for testing that difference in the energy wasn't significant.
All the tests have been passed and this change can be removed if you wish.
The full change is visible in commit 867b24a
867b24a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's okay to leave in, but then a default needs to be set for the parameter. Since it's just a check, perhaps it should be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that there was no bug in restoresu3. The existing routine was a valid procedure, but the extended approach is of course somewhat better in finite precision.