tmLQCD icon indicating copy to clipboard operation
tmLQCD copied to clipboard

commit 53db31 of deflation branch deleted a bunch of stuff

Open kostrzewa opened this issue 9 years ago • 9 comments

unfortunately a large number of things were removed during the merge of the deflation branch, in particular commit 53db31bff1d377816238945b127fdfccee97c741 caused a number of solvers to disappear...

kostrzewa avatar May 24 '16 17:05 kostrzewa

I think https://github.com/kostrzewa/tmLQCD/tree/mixed_cg_merge_hmc can serve as a guide on how to restore the lost functionality...

kostrzewa avatar May 24 '16 18:05 kostrzewa

It's not clear in the github diff, but it becomes clear when using git show

kostrzewa avatar May 24 '16 18:05 kostrzewa

Hmm, that's bad. Do you already know what to do?

On 24 May 2016 20:32:02 CEST, Bartosz Kostrzewa [email protected] wrote:

It's not clear in the github diff, but it becomes clear when using git show


You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/etmc/tmLQCD/issues/328#issuecomment-221355573

Carsten Urbach, www.carsten-urbach.eu

urbach avatar May 24 '16 20:05 urbach

Hmm, that's bad. Do you already know what to do?

My plan was to first restore the solver logic in invert_clover_eo.[c,h] and operator.c, which is where I first noticed it. Afterwards we will have to watch for further regressions and do further manual merges. I don't think that it would be possible, without a major investment of effort, to completely reconstruct what happened and how it affected everything. There were just too many changes in the deflation pull-request to make this practical...

kostrzewa avatar May 25 '16 07:05 kostrzewa

update: I meant invert_clover_eo.[c,h]

kostrzewa avatar May 25 '16 07:05 kostrzewa

unfortunately a large number of things were removed during the merge of the deflation branch, in particular commit 53db31bff1d377816238945b127fdfccee97c741 caused a number of solvers to disappear...

okay, I see that the RGMIXEDCG is no longer available in invert_clover_eo.c. What else?


You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

https://github.com/etmc/tmLQCD/issues/328

Carsten Urbach e-mail: [email protected] [email protected] Fon : +49 (0)228 73 2379 skype : carsten.urbach URL: http://www.carsten-urbach.eu

urbach avatar May 25 '16 08:05 urbach

at least the following in missing in invert_clover_eo.c in the logic

else if(solver_flag == RGMIXEDCG){
 iter = rg_mixed_cg_her(Odd_new, g_spinor_field[DUM_DERI], solver_params, max_iter, precision, rel_prec,
                                        VOLUME/2, &Qsw_pm_psi, &Qsw_pm_psi_32);
   Qm(Odd_new, Odd_new);
 }

urbach avatar May 25 '16 08:05 urbach

Well, for now it's only #329, but we'll have to see how things develop. Maybe other files were similarly affected...

kostrzewa avatar May 25 '16 08:05 kostrzewa

yes, I agree, we have to review carefully. I have a boring meeting starting at 11, where I might find some time...

Carsten Urbach e-mail: [email protected] [email protected] Fon : +49 (0)228 73 2379 skype : carsten.urbach URL: http://www.carsten-urbach.eu

urbach avatar May 25 '16 08:05 urbach