{"action":"created","issue":{"url":"https://api.github.com/repos/choderalab/perses/issues/81","labels_url":"https://api.github.com/repos/choderalab/perses/issues/81/labels{/name}","comments_url":"https://api.github.com/repos/choderalab/perses/issues/81/comments","events_url":"https://api.github.com/repos/choderalab/perses/issues/81/events","html_url":"https://github.com/choderalab/perses/pull/81","id":130438086,"number":81,"title":"Allow specification of desired mutations in metadata","user":{"login":"juliebehr","id":6343082,"avatar_url":"https://avatars.githubusercontent.com/u/6343082?v=3","gravatar_id":"","url":"https://api.github.com/users/juliebehr","html_url":"https://github.com/juliebehr","followers_url":"https://api.github.com/users/juliebehr/followers","following_url":"https://api.github.com/users/juliebehr/following{/other_user}","gists_url":"https://api.github.com/users/juliebehr/gists{/gist_id}","starred_url":"https://api.github.com/users/juliebehr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/juliebehr/subscriptions","organizations_url":"https://api.github.com/users/juliebehr/orgs","repos_url":"https://api.github.com/users/juliebehr/repos","events_url":"https://api.github.com/users/juliebehr/events{/privacy}","received_events_url":"https://api.github.com/users/juliebehr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-02-01T18:10:23Z","updated_at":"2016-02-01T23:03:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/choderalab/perses/pulls/81","html_url":"https://github.com/choderalab/perses/pull/81","diff_url":"https://github.com/choderalab/perses/pull/81.diff","patch_url":"https://github.com/choderalab/perses/pull/81.patch"},"body":"Calling the list of mutations ```metadata['mutation_library']``` but this is distinct from ```metadata['sequence_library']``` which will be used to define things like peptides, probably in a separate class."},"comment":{"url":"https://api.github.com/repos/choderalab/perses/issues/comments/178244883","html_url":"https://github.com/choderalab/perses/pull/81#issuecomment-178244883","issue_url":"https://api.github.com/repos/choderalab/perses/issues/81","id":178244883,"user":{"login":"jchodera","id":3656088,"avatar_url":"https://avatars.githubusercontent.com/u/3656088?v=3","gravatar_id":"","url":"https://api.github.com/users/jchodera","html_url":"https://github.com/jchodera","followers_url":"https://api.github.com/users/jchodera/followers","following_url":"https://api.github.com/users/jchodera/following{/other_user}","gists_url":"https://api.github.com/users/jchodera/gists{/gist_id}","starred_url":"https://api.github.com/users/jchodera/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jchodera/subscriptions","organizations_url":"https://api.github.com/users/jchodera/orgs","repos_url":"https://api.github.com/users/jchodera/repos","events_url":"https://api.github.com/users/jchodera/events{/privacy}","received_events_url":"https://api.github.com/users/jchodera/received_events","type":"User","site_admin":false},"created_at":"2016-02-01T23:03:09Z","updated_at":"2016-02-01T23:03:09Z","body":"Looks good!  The only issue I spotted is that unless we also want to have to specify the wild-type residue for each resid we want to allow mutations for, we probably want to have the constructor add the current residue name at that resid to the list of `allowed_mutations` at that site. \r\n\r\nFor example, currently, if we have `ALA-VAL-LEU` and specify `allowed_mutations = [ ('1', ['GLY']) ]`, it won't let us go back to `ALA`---it will just change it to `GLY` and be done with it.  Not sure if this is a feature or a bug.\r\n\r\nOtherwise, good to merge!"}}