Project

General

Profile

Bug #1189

g_mindist and PBC

Added by Mark Abraham about 4 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Normal
Category:
analysis tools
Target version:
Affected version - extra info:
Affected version:
Difficulty:
uncategorized
Close

Description

g_mindist is broken with a .tpr that uses pbc=xy. See Mark's comment to https://gerrit.gromacs.org/2221

Associated revisions

Revision b72bb55d (diff)
Added by Berk Hess almost 3 years ago

Fix g_mindist with pbc=XY

Fixes #1189.

Change-Id: I7c8242df6f2644bebc49c13c86412f8bc1d37428

History

#1 Updated by Roland Schulz almost 3 years ago

  • Affected version set to 4.6

Should we for now just add a error message?

#2 Updated by Mark Abraham almost 3 years ago

  • Target version set to 4.6.6

Roland Schulz wrote:

Should we for now just add a error message?

Sure

#3 Updated by Gerrit Code Review Bot almost 3 years ago

Gerrit received a related patchset '1' for Issue #1189.
Uploader: Rossen Apostolov ()
Change-Id: I590bdaf23683dce42faab70637978474e4bb8822
Gerrit URL: https://gerrit.gromacs.org/3629

#4 Updated by Rossen Apostolov almost 3 years ago

  • Status changed from New to Fix uploaded
  • Assignee set to Rossen Apostolov

#5 Updated by Szilárd Páll almost 3 years ago

  • Category set to analysis tools

If the error message is considered a fix, I suggest creating a (related) issue with target 5.x that actually fixes the bug.

#6 Updated by Mark Abraham almost 3 years ago

I think the commit should Refs this issue, and we should clear the target until someone declares intent to fix it, per policy. Less work than opening and closing issues. If someone wants a fix, they need to do it, or ask normally. :-)

#7 Updated by Gerrit Code Review Bot almost 3 years ago

Gerrit received a related patchset '1' for Issue #1189.
Uploader: Berk Hess ()
Change-Id: I7c8242df6f2644bebc49c13c86412f8bc1d37428
Gerrit URL: https://gerrit.gromacs.org/3640

#8 Updated by Erik Lindahl almost 3 years ago

  • Status changed from Fix uploaded to Resolved

#9 Updated by Erik Lindahl almost 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF