{"action":"created","issue":{"url":"https://api.github.com/repos/dmzuckerman/Sampling-Uncertainty/issues/48","repository_url":"https://api.github.com/repos/dmzuckerman/Sampling-Uncertainty","labels_url":"https://api.github.com/repos/dmzuckerman/Sampling-Uncertainty/issues/48/labels{/name}","comments_url":"https://api.github.com/repos/dmzuckerman/Sampling-Uncertainty/issues/48/comments","events_url":"https://api.github.com/repos/dmzuckerman/Sampling-Uncertainty/issues/48/events","html_url":"https://github.com/dmzuckerman/Sampling-Uncertainty/issues/48","id":351593535,"node_id":"MDU6SXNzdWUzNTE1OTM1MzU=","number":48,"title":"Catch all / Remaining Reviewer Comments","user":{"login":"dwsideriusNIST","id":5341350,"node_id":"MDQ6VXNlcjUzNDEzNTA=","avatar_url":"https://avatars2.githubusercontent.com/u/5341350?v=4","gravatar_id":"","url":"https://api.github.com/users/dwsideriusNIST","html_url":"https://github.com/dwsideriusNIST","followers_url":"https://api.github.com/users/dwsideriusNIST/followers","following_url":"https://api.github.com/users/dwsideriusNIST/following{/other_user}","gists_url":"https://api.github.com/users/dwsideriusNIST/gists{/gist_id}","starred_url":"https://api.github.com/users/dwsideriusNIST/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dwsideriusNIST/subscriptions","organizations_url":"https://api.github.com/users/dwsideriusNIST/orgs","repos_url":"https://api.github.com/users/dwsideriusNIST/repos","events_url":"https://api.github.com/users/dwsideriusNIST/events{/privacy}","received_events_url":"https://api.github.com/users/dwsideriusNIST/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"dwsideriusNIST","id":5341350,"node_id":"MDQ6VXNlcjUzNDEzNTA=","avatar_url":"https://avatars2.githubusercontent.com/u/5341350?v=4","gravatar_id":"","url":"https://api.github.com/users/dwsideriusNIST","html_url":"https://github.com/dwsideriusNIST","followers_url":"https://api.github.com/users/dwsideriusNIST/followers","following_url":"https://api.github.com/users/dwsideriusNIST/following{/other_user}","gists_url":"https://api.github.com/users/dwsideriusNIST/gists{/gist_id}","starred_url":"https://api.github.com/users/dwsideriusNIST/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dwsideriusNIST/subscriptions","organizations_url":"https://api.github.com/users/dwsideriusNIST/orgs","repos_url":"https://api.github.com/users/dwsideriusNIST/repos","events_url":"https://api.github.com/users/dwsideriusNIST/events{/privacy}","received_events_url":"https://api.github.com/users/dwsideriusNIST/received_events","type":"User","site_admin":false},"assignees":[{"login":"dwsideriusNIST","id":5341350,"node_id":"MDQ6VXNlcjUzNDEzNTA=","avatar_url":"https://avatars2.githubusercontent.com/u/5341350?v=4","gravatar_id":"","url":"https://api.github.com/users/dwsideriusNIST","html_url":"https://github.com/dwsideriusNIST","followers_url":"https://api.github.com/users/dwsideriusNIST/followers","following_url":"https://api.github.com/users/dwsideriusNIST/following{/other_user}","gists_url":"https://api.github.com/users/dwsideriusNIST/gists{/gist_id}","starred_url":"https://api.github.com/users/dwsideriusNIST/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dwsideriusNIST/subscriptions","organizations_url":"https://api.github.com/users/dwsideriusNIST/orgs","repos_url":"https://api.github.com/users/dwsideriusNIST/repos","events_url":"https://api.github.com/users/dwsideriusNIST/events{/privacy}","received_events_url":"https://api.github.com/users/dwsideriusNIST/received_events","type":"User","site_admin":false},{"login":"ajschult","id":7952924,"node_id":"MDQ6VXNlcjc5NTI5MjQ=","avatar_url":"https://avatars0.githubusercontent.com/u/7952924?v=4","gravatar_id":"","url":"https://api.github.com/users/ajschult","html_url":"https://github.com/ajschult","followers_url":"https://api.github.com/users/ajschult/followers","following_url":"https://api.github.com/users/ajschult/following{/other_user}","gists_url":"https://api.github.com/users/ajschult/gists{/gist_id}","starred_url":"https://api.github.com/users/ajschult/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ajschult/subscriptions","organizations_url":"https://api.github.com/users/ajschult/orgs","repos_url":"https://api.github.com/users/ajschult/repos","events_url":"https://api.github.com/users/ajschult/events{/privacy}","received_events_url":"https://api.github.com/users/ajschult/received_events","type":"User","site_admin":false},{"login":"drroe","id":10144183,"node_id":"MDQ6VXNlcjEwMTQ0MTgz","avatar_url":"https://avatars2.githubusercontent.com/u/10144183?v=4","gravatar_id":"","url":"https://api.github.com/users/drroe","html_url":"https://github.com/drroe","followers_url":"https://api.github.com/users/drroe/followers","following_url":"https://api.github.com/users/drroe/following{/other_user}","gists_url":"https://api.github.com/users/drroe/gists{/gist_id}","starred_url":"https://api.github.com/users/drroe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drroe/subscriptions","organizations_url":"https://api.github.com/users/drroe/orgs","repos_url":"https://api.github.com/users/drroe/repos","events_url":"https://api.github.com/users/drroe/events{/privacy}","received_events_url":"https://api.github.com/users/drroe/received_events","type":"User","site_admin":false},{"login":"agrossfield","id":13631218,"node_id":"MDQ6VXNlcjEzNjMxMjE4","avatar_url":"https://avatars3.githubusercontent.com/u/13631218?v=4","gravatar_id":"","url":"https://api.github.com/users/agrossfield","html_url":"https://github.com/agrossfield","followers_url":"https://api.github.com/users/agrossfield/followers","following_url":"https://api.github.com/users/agrossfield/following{/other_user}","gists_url":"https://api.github.com/users/agrossfield/gists{/gist_id}","starred_url":"https://api.github.com/users/agrossfield/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agrossfield/subscriptions","organizations_url":"https://api.github.com/users/agrossfield/orgs","repos_url":"https://api.github.com/users/agrossfield/repos","events_url":"https://api.github.com/users/agrossfield/events{/privacy}","received_events_url":"https://api.github.com/users/agrossfield/received_events","type":"User","site_admin":false},{"login":"ptmerz","id":20297930,"node_id":"MDQ6VXNlcjIwMjk3OTMw","avatar_url":"https://avatars1.githubusercontent.com/u/20297930?v=4","gravatar_id":"","url":"https://api.github.com/users/ptmerz","html_url":"https://github.com/ptmerz","followers_url":"https://api.github.com/users/ptmerz/followers","following_url":"https://api.github.com/users/ptmerz/following{/other_user}","gists_url":"https://api.github.com/users/ptmerz/gists{/gist_id}","starred_url":"https://api.github.com/users/ptmerz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ptmerz/subscriptions","organizations_url":"https://api.github.com/users/ptmerz/orgs","repos_url":"https://api.github.com/users/ptmerz/repos","events_url":"https://api.github.com/users/ptmerz/events{/privacy}","received_events_url":"https://api.github.com/users/ptmerz/received_events","type":"User","site_admin":false},{"login":"dmzuckerman","id":21345239,"node_id":"MDQ6VXNlcjIxMzQ1MjM5","avatar_url":"https://avatars2.githubusercontent.com/u/21345239?v=4","gravatar_id":"","url":"https://api.github.com/users/dmzuckerman","html_url":"https://github.com/dmzuckerman","followers_url":"https://api.github.com/users/dmzuckerman/followers","following_url":"https://api.github.com/users/dmzuckerman/following{/other_user}","gists_url":"https://api.github.com/users/dmzuckerman/gists{/gist_id}","starred_url":"https://api.github.com/users/dmzuckerman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dmzuckerman/subscriptions","organizations_url":"https://api.github.com/users/dmzuckerman/orgs","repos_url":"https://api.github.com/users/dmzuckerman/repos","events_url":"https://api.github.com/users/dmzuckerman/events{/privacy}","received_events_url":"https://api.github.com/users/dmzuckerman/received_events","type":"User","site_admin":false},{"login":"mangiapasta","id":31415825,"node_id":"MDQ6VXNlcjMxNDE1ODI1","avatar_url":"https://avatars1.githubusercontent.com/u/31415825?v=4","gravatar_id":"","url":"https://api.github.com/users/mangiapasta","html_url":"https://github.com/mangiapasta","followers_url":"https://api.github.com/users/mangiapasta/followers","following_url":"https://api.github.com/users/mangiapasta/following{/other_user}","gists_url":"https://api.github.com/users/mangiapasta/gists{/gist_id}","starred_url":"https://api.github.com/users/mangiapasta/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mangiapasta/subscriptions","organizations_url":"https://api.github.com/users/mangiapasta/orgs","repos_url":"https://api.github.com/users/mangiapasta/repos","events_url":"https://api.github.com/users/mangiapasta/events{/privacy}","received_events_url":"https://api.github.com/users/mangiapasta/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2018-08-17T13:24:55Z","updated_at":"2018-08-18T16:47:07Z","closed_at":null,"author_association":"COLLABORATOR","body":"Reviewer comments:\r\n\r\n\"Some pitfalls could be mentioned. Students often take the average from a trajectory and then the standard deviation and assume this is the \"error bar\"! They don't realize that the standard deviation here is just a measure of the fluctuations, which is just related to system size. It might be worth mentioning somewhere.\"\r\n\r\n\"The density of information contained in different sections can vary strongly. That makes the manuscript partially harder to read. While sections 1-5 are rather stretched out and new information is presented slowly, section 6 contains much more information in much less text. Moreover, while in sections 1-5 also rather basic concepts are introduced with care, in section 6 more advanced concepts are just introduced very quickly or not even explained at all. Examples for this are Voroni cells mentioned at the beginning of section 6.1 on page 12 or PCA mentioned on page 13. This makes section 6 partially hard to understand. Figure 6 helps to understand this section, but is not referenced in the text.\"\r\n\r\n\"p2: Maybe one should also include software/programming errors (see e.g. Ref 1, and a number of other recent references reporting \"bugs\" in heavily-used MD codes like GROMACS)\""},"comment":{"url":"https://api.github.com/repos/dmzuckerman/Sampling-Uncertainty/issues/comments/414071110","html_url":"https://github.com/dmzuckerman/Sampling-Uncertainty/issues/48#issuecomment-414071110","issue_url":"https://api.github.com/repos/dmzuckerman/Sampling-Uncertainty/issues/48","id":414071110,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNDA3MTExMA==","user":{"login":"agrossfield","id":13631218,"node_id":"MDQ6VXNlcjEzNjMxMjE4","avatar_url":"https://avatars3.githubusercontent.com/u/13631218?v=4","gravatar_id":"","url":"https://api.github.com/users/agrossfield","html_url":"https://github.com/agrossfield","followers_url":"https://api.github.com/users/agrossfield/followers","following_url":"https://api.github.com/users/agrossfield/following{/other_user}","gists_url":"https://api.github.com/users/agrossfield/gists{/gist_id}","starred_url":"https://api.github.com/users/agrossfield/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agrossfield/subscriptions","organizations_url":"https://api.github.com/users/agrossfield/orgs","repos_url":"https://api.github.com/users/agrossfield/repos","events_url":"https://api.github.com/users/agrossfield/events{/privacy}","received_events_url":"https://api.github.com/users/agrossfield/received_events","type":"User","site_admin":false},"created_at":"2018-08-18T16:47:07Z","updated_at":"2018-08-18T16:47:07Z","author_association":"COLLABORATOR","body":"In response to the third issue, I added a sentence to the scope section, saying we don't address bugs and implementation errors. \r\n\r\nI didn't add a reference to software best practices -- isn't there an article on that coming from the same workshop that led to this article?"}}
{"actor":{"display_login":"agrossfield"}}