{"action":"closed","issue":{"url":"https://api.github.com/repos/JTFouquier/ghost-tree/issues/14","labels_url":"https://api.github.com/repos/JTFouquier/ghost-tree/issues/14/labels{/name}","comments_url":"https://api.github.com/repos/JTFouquier/ghost-tree/issues/14/comments","events_url":"https://api.github.com/repos/JTFouquier/ghost-tree/issues/14/events","html_url":"https://github.com/JTFouquier/ghost-tree/issues/14","id":65133320,"number":14,"title":"ghost-tree filter-alignment-positions works, but unit testing is challenging...","user":{"login":"JTFouquier","id":7920357,"avatar_url":"https://avatars.githubusercontent.com/u/7920357?v=3","gravatar_id":"","url":"https://api.github.com/users/JTFouquier","html_url":"https://github.com/JTFouquier","followers_url":"https://api.github.com/users/JTFouquier/followers","following_url":"https://api.github.com/users/JTFouquier/following{/other_user}","gists_url":"https://api.github.com/users/JTFouquier/gists{/gist_id}","starred_url":"https://api.github.com/users/JTFouquier/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JTFouquier/subscriptions","organizations_url":"https://api.github.com/users/JTFouquier/orgs","repos_url":"https://api.github.com/users/JTFouquier/repos","events_url":"https://api.github.com/users/JTFouquier/events{/privacy}","received_events_url":"https://api.github.com/users/JTFouquier/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-03-30T04:43:19Z","updated_at":"2015-04-03T16:54:42Z","closed_at":"2015-04-03T16:54:42Z","body":"@ebolyen, @jairideout, @johnchase:  we worked on filter.py (filter-alignment-positions) in AZ, but I just moved it into 3 functions and got rid of the workaround (aln = aln.omit_gap_positions(1.0 - np.finfo(float).eps)) after skbio was updated.  \r\n\r\nhttps://github.com/JTFouquier/ghost-tree/blob/master/ghosttree/filter.py\r\n\r\n@ebolyen said it would be difficult to test.  o. m. g. \r\n\r\nIs it safe to say that enough of this script uses skbio that I don't really need to unit test it?  :)  If I need to test it, any tips?  I have tried unit testing this so many times.  \r\n\r\nThanks! \r\n"}}