-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
_struct_conn group missing #4
Comments
We currently rely on mmcif files and the chemical component dictionary to get bond information. We felt that the metal bond given in the archive are currently not good enough, so we did not include them. |
do please include them. Let the user decide if they want them or not. On Mon, Apr 25, 2016 at 11:06 AM, Alexander Rose notifications@github.com
Robert M. Hanson If nature does not answer first what we want, -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900 |
The Chimera team also expressed interest to have those bonds. |
The question boils down to whether we absolutely stick to clean and consistent data for whole-archive analyses or allow some data (like "metal bonds") that is only good for a number of structures and then filter it out at the API level. |
Just notice that although we have all the bonding between amino acids and all the bonding within groups, we are missing the very important ones in _struct_conn, particularly in 1BLU
I believe this may be because the "metalc" type is being skipped. This is an important type to include.
metalc1 metalc ? B SF4 . FE1 ? ? ? 1_555 A CYS 53 SG ? ? A SF4 101 A CYS 53 1_555 ? ? ? ? ? ? ? 2.189 ?
same for 1hho:
metalc1 metalc ? A HIS 87 NE2 ? ? ? 1_555 D HEM . FE ? ? A HIS 87 A HEM 143 1_555 ? ? ? ? ? ? ? 1.937 ?
It is critical that we have these.
The text was updated successfully, but these errors were encountered: