{"action":"created","issue":{"url":"https://api.github.com/repos/bigdatagenomics/adam/issues/727","labels_url":"https://api.github.com/repos/bigdatagenomics/adam/issues/727/labels{/name}","comments_url":"https://api.github.com/repos/bigdatagenomics/adam/issues/727/comments","events_url":"https://api.github.com/repos/bigdatagenomics/adam/issues/727/events","html_url":"https://github.com/bigdatagenomics/adam/pull/727","id":94637623,"number":727,"title":"Update MdTag parsing for CIGAR N","user":{"login":"arahuja","id":455755,"avatar_url":"https://avatars.githubusercontent.com/u/455755?v=3","gravatar_id":"","url":"https://api.github.com/users/arahuja","html_url":"https://github.com/arahuja","followers_url":"https://api.github.com/users/arahuja/followers","following_url":"https://api.github.com/users/arahuja/following{/other_user}","gists_url":"https://api.github.com/users/arahuja/gists{/gist_id}","starred_url":"https://api.github.com/users/arahuja/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arahuja/subscriptions","organizations_url":"https://api.github.com/users/arahuja/orgs","repos_url":"https://api.github.com/users/arahuja/repos","events_url":"https://api.github.com/users/arahuja/events{/privacy}","received_events_url":"https://api.github.com/users/arahuja/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-07-13T04:39:16Z","updated_at":"2015-07-20T18:59:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bigdatagenomics/adam/pulls/727","html_url":"https://github.com/bigdatagenomics/adam/pull/727","diff_url":"https://github.com/bigdatagenomics/adam/pull/727.diff","patch_url":"https://github.com/bigdatagenomics/adam/pull/727.patch"},"body":"Fixes #726 \r\n\r\nThis fixes the MdTag parsing to respect N operators in the CIGAR.  It turned to out be trickier than I expected, so if this isn't a case we want to support or want to handle it differently, I'd be fine with that.\r\n\r\nI added 2 tests for that case as well but removed one test that required an error if an MDTag did not end with a digit - is that a valid constraint?  I couldn't figure out how it would affect parsing or make it ambiguous?"},"comment":{"url":"https://api.github.com/repos/bigdatagenomics/adam/issues/comments/122988981","html_url":"https://github.com/bigdatagenomics/adam/pull/727#issuecomment-122988981","issue_url":"https://api.github.com/repos/bigdatagenomics/adam/issues/727","id":122988981,"user":{"login":"arahuja","id":455755,"avatar_url":"https://avatars.githubusercontent.com/u/455755?v=3","gravatar_id":"","url":"https://api.github.com/users/arahuja","html_url":"https://github.com/arahuja","followers_url":"https://api.github.com/users/arahuja/followers","following_url":"https://api.github.com/users/arahuja/following{/other_user}","gists_url":"https://api.github.com/users/arahuja/gists{/gist_id}","starred_url":"https://api.github.com/users/arahuja/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arahuja/subscriptions","organizations_url":"https://api.github.com/users/arahuja/orgs","repos_url":"https://api.github.com/users/arahuja/repos","events_url":"https://api.github.com/users/arahuja/events{/privacy}","received_events_url":"https://api.github.com/users/arahuja/received_events","type":"User","site_admin":false},"created_at":"2015-07-20T18:59:13Z","updated_at":"2015-07-20T18:59:13Z","body":"@fnothaft Updated to handle the `S` operator as well, as that was missing before.  I still did not add back in throwing an error an missing digit to end the MDTag, as I can't think of case where the reference positions would be wrong do that.  I can add it in, but it doesn't seem necessary.\r\n\r\nI want to add some more test cases as well anyways"}}