Title: MMCif behavior when auth_seq_id is missing · Issue #775 · biojava/biojava · GitHub
Open Graph Title: MMCif behavior when auth_seq_id is missing · Issue #775 · biojava/biojava
X Title: MMCif behavior when auth_seq_id is missing · Issue #775 · biojava/biojava
Description: A discussion came up in PR #774 regarding the correct behavior when parsing an mmCif file without auth_seq_id. BioJava 4.2.11 requires the auth_seq_id column. This is a problem because it is optional according to the spec and omitted by ...
Open Graph Description: A discussion came up in PR #774 regarding the correct behavior when parsing an mmCif file without auth_seq_id. BioJava 4.2.11 requires the auth_seq_id column. This is a problem because it is option...
X Description: A discussion came up in PR #774 regarding the correct behavior when parsing an mmCif file without auth_seq_id. BioJava 4.2.11 requires the auth_seq_id column. This is a problem because it is option...
Opengraph URL: https://github.com/biojava/biojava/issues/775
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"MMCif behavior when auth_seq_id is missing","articleBody":"A discussion came up in PR #774 regarding the correct behavior when parsing an mmCif file without `auth_seq_id`.\r\n\r\nBioJava 4.2.11 requires the `auth_seq_id` column. This is a problem because it is optional according to the spec and omitted by PyMOL.\r\n\r\nIn b207d34 I added code to use the `label_seq_id` column for creating the ResidueNumber for each group if `auth_seq_id` is missing. There was some concern that this could lead to inconsistent residue numbers if some residues used '?' (defaulting to `label_`) while the rest used the `auth_` values specified. This worry is actually not justified due to *another* bug, which causes a NumberFormatException if '?' is used in that column.\r\n\r\n@josemduarte suggested only doing the `label_` fallback if ALL groups have null ResidueNumbers. This is probably the right solution, but it seems like such an edge case it might not be worth the hour it will take to fix it.","author":{"url":"https://github.com/sbliven","@type":"Person","name":"sbliven"},"datePublished":"2018-06-15T10:23:10.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":11},"url":"https://github.com/775/biojava/issues/775"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:347a9076-45bb-171c-b354-81a55dfda793 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | C730:25602F:18F0AB9:2174F52:6973F621 |
| html-safe-nonce | 7cdcddd7a561306bf6d372adf91d0369d34f4d5b4ff71f981658ccb1b1218ad1 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDNzMwOjI1NjAyRjoxOEYwQUI5OjIxNzRGNTI6Njk3M0Y2MjEiLCJ2aXNpdG9yX2lkIjoiNTc3NjUzMzQ2MjMxNjQ1NzI5IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | fea8a81b1133e3fbda28538241e66cc8b77d0b0438daca9884fb34694d568cd9 |
| hovercard-subject-tag | issue:332722573 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/biojava/biojava/775/issue_layout |
| twitter:image | https://opengraph.githubassets.com/2a5b5d5067dbe9e03da6ec5cd710e132122f1e964ac313930f64f82e520ccde6/biojava/biojava/issues/775 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/2a5b5d5067dbe9e03da6ec5cd710e132122f1e964ac313930f64f82e520ccde6/biojava/biojava/issues/775 |
| og:image:alt | A discussion came up in PR #774 regarding the correct behavior when parsing an mmCif file without auth_seq_id. BioJava 4.2.11 requires the auth_seq_id column. This is a problem because it is option... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | sbliven |
| hostname | github.com |
| expected-hostname | github.com |
| None | edd166dbd7a8c69304dd0f786a2ceae171d7a532d7f82516d7b971b6a831123e |
| turbo-cache-control | no-preview |
| go-import | github.com/biojava/biojava git https://github.com/biojava/biojava.git |
| octolytics-dimension-user_id | 237402 |
| octolytics-dimension-user_login | biojava |
| octolytics-dimension-repository_id | 9199359 |
| octolytics-dimension-repository_nwo | biojava/biojava |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 9199359 |
| octolytics-dimension-repository_network_root_nwo | biojava/biojava |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 7f2d62d773fd1f2c679e2ea93329a72fda65310a |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width