Skip to content

Konstantin devel Wilson+Clover N_f=2+1 implementation#673

Open
kottnad wants to merge 4 commits intomasterfrom
konstantin_devel
Open

Konstantin devel Wilson+Clover N_f=2+1 implementation#673
kottnad wants to merge 4 commits intomasterfrom
konstantin_devel

Conversation

@kottnad
Copy link
Copy Markdown
Contributor

@kottnad kottnad commented Mar 27, 2026

Provides a working implementation of N_f=2+1 (plain) Wilson+Clover simulations.
Still lacks a GPU implementation of the force for the single flavor part though.

kottnad and others added 4 commits March 23, 2026 17:29
…ed explicit "single_flavor" flag in monomials to e.g. fix issue with double counting of trlog term and make the code more expressive in several places.
inv_param.dslash_type = QUDA_CLOVER_WILSON_DSLASH;
}
inv_param.matpc_type = QUDA_MATPC_EVEN_EVEN;
inv_param.matpc_type = QUDA_MATPC_EVEN_EVEN_ASYMMETRIC;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you test what happens when you set this to QUDA_MATPC_ODD_ODD_ASYMMETRIC ?

@kostrzewa
Copy link
Copy Markdown
Member

@kottnad thanks, this is looking great!

Not sure how to proceed w.r.t. the order of merging this or #664 first. Any order will require a certain level of conflict resolution.

@kostrzewa
Copy link
Copy Markdown
Member

@chaoos @mtaillefumier any thoughts on the best order of merging this or #664 ?

@chaoos
Copy link
Copy Markdown
Contributor

chaoos commented Mar 27, 2026

@kostrzewa If this is a quick and small thing, I'd merge it. Yes, it gives conflicts on the cmake branch, but else we have to suspend all merges until cmake is merged (which might still take a while?).

@kostrzewa
Copy link
Copy Markdown
Member

@kostrzewa If this is a quick and small thing, I'd merge it. Yes, it gives conflicts on the cmake branch, but else we have to suspend all merges until cmake is merged (which might still take a while?).

@chaoos compared to the cmake_support branch the number of changes here is small

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants