[Biopython-dev] [Bug 2727] PDB.Bio: header should include CRYST1 information
bugzilla-daemon at portal.open-bio.org
bugzilla-daemon at portal.open-bio.org
Fri Jan 9 12:50:37 UTC 2009
http://bugzilla.open-bio.org/show_bug.cgi?id=2727
------- Comment #2 from biopython-bugzilla at maubp.freeserve.co.uk 2009-01-09 07:50 EST -------
Hopefully Bio.PDB's owner/maintainer Thomas Hamelryck can comment on this.
In the meantime, the code style seems to fit fine with the rest of
parse_pdb_header.py which is good. However, you have not updated the
parse_pdb_header function's docstring to include the new keys. Furthermore, it
would be nice to have the docstring describe the meaning of the cell, z-cell
and spacegroup entries you have introduced. I'm also curious about the default
values and their meanings.
Peter
--
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
More information about the Biopython-dev
mailing list