Change Kshell sorting precision to full precision#5411
Change Kshell sorting precision to full precision#5411ye-luo wants to merge 1 commit intoQMCPACK:developfrom
Conversation
|
It's going to be a bunch of work for me to unwrap all my dependent changes from my fix and encapsulation of kcontainer. Just finish your review of my work, I didn't ask you to play coding golf with this and I've been held up for a very long time in review. My performance is not evaluated based on how small a change you can eventually figure out how to make. |
We can probably use git trick to simply revert the change #5389 included in #5406. I can facilitate the work. Are there any others PRs or branches affected by #5389? |
PDoakORNL
left a comment
There was a problem hiding this comment.
I do not support this reversion.
This PR is left pending as it was. Will revise and clean up once your estimates get in. |
|
Is this going anywhere? |
|
@jtkrogel the changes is PR is still crucial to keep minimal code with mixed precision support. |
Proposed changes
Having
modk2in full precision seems fixing kshell counting issue.fixes #5384
What type(s) of changes does this code introduce?
Does this introduce a breaking change?
What systems has this change been tested on?
epyc-serer
Checklist