Project

General

Profile

Bug #2557

make_gromos_rtp.py has a sytax error

Added by Logan Dahlquist 3 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
preprocessing (pdb2gmx,grompp)
Target version:
Affected version - extra info:
2018.2
Affected version:
Difficulty:
simple
Close

Description

The script make_gromos_rtp.py has a sytax error and is missing a ',' on line 126. The line reads

print >> sys.stderr "Could not find string",string,"in list of length",len(list)

normally, however the correct syntax is

print >> sys.stderr, "Could not find string",string,"in list of length",len(list)

by adding the comma after sys.stderr. While this wasn't a huge issue to fix, I hope that it can be updated so that others who download the package do not run into the same issue.

Thanks,
Logan

Associated revisions

Revision 72d0d53f (diff)
Added by Kevin Boyd about 2 months ago

Fixed syntax error in make_gromos_rtp.py

Didn't affect the resulting rtp if everything else was in order,
just prevented an error message.

Refs #2557

Change-Id: I5289db159b62abde31103841c33d2b90f4a7717e

History

#1 Updated by Gerrit Code Review Bot about 2 months ago

Gerrit received a related patchset '1' for Issue #2557.
Uploader: Kevin Boyd ()
Change-Id: gromacs~release-2018~I5289db159b62abde31103841c33d2b90f4a7717e
Gerrit URL: https://gerrit.gromacs.org/8116

#2 Updated by Mark Abraham about 2 months ago

  • Description updated (diff)
  • Category set to preprocessing (pdb2gmx,grompp)
  • Status changed from New to Fix uploaded
  • Assignee deleted (Logan Dahlquist)
  • Target version set to 2018.3

#3 Updated by Paul Bauer about 1 month ago

  • Status changed from Fix uploaded to Closed

Also available in: Atom PDF