{"action":"created","issue":{"url":"https://api.github.com/repos/mickypaganini/lwtnn/issues/1","repository_url":"https://api.github.com/repos/mickypaganini/lwtnn","labels_url":"https://api.github.com/repos/mickypaganini/lwtnn/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/mickypaganini/lwtnn/issues/1/comments","events_url":"https://api.github.com/repos/mickypaganini/lwtnn/issues/1/events","html_url":"https://github.com/mickypaganini/lwtnn/pull/1","id":153714584,"number":1,"title":"Merge to recent RNN changes, some cleanup","user":{"login":"dguest","id":1979581,"avatar_url":"https://avatars.githubusercontent.com/u/1979581?v=3","gravatar_id":"","url":"https://api.github.com/users/dguest","html_url":"https://github.com/dguest","followers_url":"https://api.github.com/users/dguest/followers","following_url":"https://api.github.com/users/dguest/following{/other_user}","gists_url":"https://api.github.com/users/dguest/gists{/gist_id}","starred_url":"https://api.github.com/users/dguest/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dguest/subscriptions","organizations_url":"https://api.github.com/users/dguest/orgs","repos_url":"https://api.github.com/users/dguest/repos","events_url":"https://api.github.com/users/dguest/events{/privacy}","received_events_url":"https://api.github.com/users/dguest/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-05-09T07:02:12Z","updated_at":"2016-05-09T08:25:10Z","closed_at":"2016-05-09T08:21:13Z","pull_request":{"url":"https://api.github.com/repos/mickypaganini/lwtnn/pulls/1","html_url":"https://github.com/mickypaganini/lwtnn/pull/1","diff_url":"https://github.com/mickypaganini/lwtnn/pull/1.diff","patch_url":"https://github.com/mickypaganini/lwtnn/pull/1.patch"},"body":"Some things from my RNN branch didn't make it into your GRU changes, and I also tried to cut down the duplicated code the GRUs added where it was easy. If these changes work with your branch I can merge your other pull request."},"comment":{"url":"https://api.github.com/repos/mickypaganini/lwtnn/issues/comments/217804675","html_url":"https://github.com/mickypaganini/lwtnn/pull/1#issuecomment-217804675","issue_url":"https://api.github.com/repos/mickypaganini/lwtnn/issues/1","id":217804675,"user":{"login":"mickypaganini","id":6855965,"avatar_url":"https://avatars.githubusercontent.com/u/6855965?v=3","gravatar_id":"","url":"https://api.github.com/users/mickypaganini","html_url":"https://github.com/mickypaganini","followers_url":"https://api.github.com/users/mickypaganini/followers","following_url":"https://api.github.com/users/mickypaganini/following{/other_user}","gists_url":"https://api.github.com/users/mickypaganini/gists{/gist_id}","starred_url":"https://api.github.com/users/mickypaganini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mickypaganini/subscriptions","organizations_url":"https://api.github.com/users/mickypaganini/orgs","repos_url":"https://api.github.com/users/mickypaganini/repos","events_url":"https://api.github.com/users/mickypaganini/events{/privacy}","received_events_url":"https://api.github.com/users/mickypaganini/received_events","type":"User","site_admin":false},"created_at":"2016-05-09T08:25:10Z","updated_at":"2016-05-09T08:25:10Z","body":"Ok, merged your stuff into mine. That should have automatically updated the other pull request right?\r\n- I like the idea of startwith('1.0')\r\n- I can modify the name to 'sort_by'. Is the point of having the list of variable names and the variable used for sorting just for the user's information? We really can put no check in place on whether the user is actually sorting tracks by the right variable and feeding in inputs with the correct variable order."}}