Project

General

Profile

Task #2391

TPI status

Added by Aleksei Iupinov 29 days ago. Updated 1 day ago.

Status:
Accepted
Priority:
Low
Assignee:
Category:
-
Target version:
Difficulty:
uncategorized
Close

Description

While trying to refactor away some annoying atom count parameter in the PME code, I ran into TPI logic again, and realized it's not covered by tests.
While trying to throw together a test, I ran into "TPI does not work (yet) with the Verlet cut-off scheme" message.
Without at least a basic TPI sanity test, I doubt this feature will survive much longer...


Subtasks

Bug #2416: TPI producing invalid results with nsteps too smallFix uploadedBerk Hess

Related issues

Related to GROMACS - Task #1971: Removing buggy features vs. keeping workflows New

History

#1 Updated by Aleksei Iupinov 29 days ago

  • Related to Task #1971: Removing buggy features vs. keeping workflows added

#2 Updated by Berk Hess 29 days ago

  • Status changed from New to Accepted

Indeed.
Do you already have a test set up? We can add it with the group scheme and later convert it to Verlet.
I should fix up the nbnxn pair search for TPI, that's actually easier than the group scheme code.

#3 Updated by Aleksei Iupinov 29 days ago

No, I don't, I've only tried adding some more mdp parameters to the PmeTest.
Maybe you could provide the mdrun input/arguments?

#4 Updated by Gerrit Code Review Bot 1 day ago

Gerrit received a related patchset '1' for Issue #2391.
Uploader: Aleksei Iupinov ()
Change-Id: gromacs~master~I41b584f4753b9b0e126256a94c1304f0a870aa73
Gerrit URL: https://gerrit.gromacs.org/7596

#5 Updated by Aleksei Iupinov 1 day ago

Here's one test with files provided by Berk.

There's more to test here - tpic, PME..

Also available in: Atom PDF