{"action":"created","issue":{"url":"https://api.github.com/repos/broadinstitute/hellbender/issues/604","labels_url":"https://api.github.com/repos/broadinstitute/hellbender/issues/604/labels{/name}","comments_url":"https://api.github.com/repos/broadinstitute/hellbender/issues/604/comments","events_url":"https://api.github.com/repos/broadinstitute/hellbender/issues/604/events","html_url":"https://github.com/broadinstitute/hellbender/pull/604","id":91805280,"number":604,"title":"Change CompareSAMs to obey validation stringency; re-enable two integ…","user":{"login":"cmnbroad","id":10062863,"avatar_url":"https://avatars.githubusercontent.com/u/10062863?v=3","gravatar_id":"","url":"https://api.github.com/users/cmnbroad","html_url":"https://github.com/cmnbroad","followers_url":"https://api.github.com/users/cmnbroad/followers","following_url":"https://api.github.com/users/cmnbroad/following{/other_user}","gists_url":"https://api.github.com/users/cmnbroad/gists{/gist_id}","starred_url":"https://api.github.com/users/cmnbroad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmnbroad/subscriptions","organizations_url":"https://api.github.com/users/cmnbroad/orgs","repos_url":"https://api.github.com/users/cmnbroad/repos","events_url":"https://api.github.com/users/cmnbroad/events{/privacy}","received_events_url":"https://api.github.com/users/cmnbroad/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"droazen","id":798637,"avatar_url":"https://avatars.githubusercontent.com/u/798637?v=3","gravatar_id":"","url":"https://api.github.com/users/droazen","html_url":"https://github.com/droazen","followers_url":"https://api.github.com/users/droazen/followers","following_url":"https://api.github.com/users/droazen/following{/other_user}","gists_url":"https://api.github.com/users/droazen/gists{/gist_id}","starred_url":"https://api.github.com/users/droazen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/droazen/subscriptions","organizations_url":"https://api.github.com/users/droazen/orgs","repos_url":"https://api.github.com/users/droazen/repos","events_url":"https://api.github.com/users/droazen/events{/privacy}","received_events_url":"https://api.github.com/users/droazen/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-06-29T13:37:04Z","updated_at":"2015-07-08T15:17:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/broadinstitute/hellbender/pulls/604","html_url":"https://github.com/broadinstitute/hellbender/pull/604","diff_url":"https://github.com/broadinstitute/hellbender/pull/604.diff","patch_url":"https://github.com/broadinstitute/hellbender/pull/604.patch"},"body":"…ration tests. The fix for the original bug (CompareSAMs not obeying stringency) is a one line fix in CompareSAMs. The two BQSR integration tests referenced in the issue use a different code path and required a different fix (assuming that relaxing the stringency is the right thing to do in those cases).\r\n\r\nI also added a new CompareSAMs integration test and changed the CompareSAMs tool to return  result of the comparison.\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/broadinstitute/hellbender/issues/comments/119621771","html_url":"https://github.com/broadinstitute/hellbender/pull/604#issuecomment-119621771","issue_url":"https://api.github.com/repos/broadinstitute/hellbender/issues/604","id":119621771,"user":{"login":"droazen","id":798637,"avatar_url":"https://avatars.githubusercontent.com/u/798637?v=3","gravatar_id":"","url":"https://api.github.com/users/droazen","html_url":"https://github.com/droazen","followers_url":"https://api.github.com/users/droazen/followers","following_url":"https://api.github.com/users/droazen/following{/other_user}","gists_url":"https://api.github.com/users/droazen/gists{/gist_id}","starred_url":"https://api.github.com/users/droazen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/droazen/subscriptions","organizations_url":"https://api.github.com/users/droazen/orgs","repos_url":"https://api.github.com/users/droazen/repos","events_url":"https://api.github.com/users/droazen/events{/privacy}","received_events_url":"https://api.github.com/users/droazen/received_events","type":"User","site_admin":false},"created_at":"2015-07-08T15:17:09Z","updated_at":"2015-07-08T15:17:09Z","body":"Before we review, would you mind rebasing this onto the latest master (in case substantial changes are needed)? Thanks!"}}