{"action":"created","issue":{"url":"https://api.github.com/repos/FHIBioGroup/fafoom-dev/issues/5","repository_url":"https://api.github.com/repos/FHIBioGroup/fafoom-dev","labels_url":"https://api.github.com/repos/FHIBioGroup/fafoom-dev/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/FHIBioGroup/fafoom-dev/issues/5/comments","events_url":"https://api.github.com/repos/FHIBioGroup/fafoom-dev/issues/5/events","html_url":"https://github.com/FHIBioGroup/fafoom-dev/pull/5","id":300802472,"number":5,"title":"Merge FAFOOM v2","user":{"login":"mahrossi","id":6743106,"avatar_url":"https://avatars2.githubusercontent.com/u/6743106?v=4","gravatar_id":"","url":"https://api.github.com/users/mahrossi","html_url":"https://github.com/mahrossi","followers_url":"https://api.github.com/users/mahrossi/followers","following_url":"https://api.github.com/users/mahrossi/following{/other_user}","gists_url":"https://api.github.com/users/mahrossi/gists{/gist_id}","starred_url":"https://api.github.com/users/mahrossi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mahrossi/subscriptions","organizations_url":"https://api.github.com/users/mahrossi/orgs","repos_url":"https://api.github.com/users/mahrossi/repos","events_url":"https://api.github.com/users/mahrossi/events{/privacy}","received_events_url":"https://api.github.com/users/mahrossi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-02-27T21:37:50Z","updated_at":"2018-02-27T22:09:28Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/FHIBioGroup/fafoom-dev/pulls/5","html_url":"https://github.com/FHIBioGroup/fafoom-dev/pull/5","diff_url":"https://github.com/FHIBioGroup/fafoom-dev/pull/5.diff","patch_url":"https://github.com/FHIBioGroup/fafoom-dev/pull/5.patch"},"body":"This is everything Dmitrii has done so far (and really essentially only Dmitrii coded everything and Carsten and I just discussed ideas and the architecture with him). \r\n\r\nThey include quite substantial changes deep inside FAFOOM. Some of them are the \"independence\" from RDkit in the code, inclusion of translational and rotational degrees of freedom, protomeric degree of freedom, and many many changes to the routines. Only pieces of the original implementation are still there. Testcases are provided. We suggest that any new developments are branched and based on this new master branch. I will leave the pull request here for a couple of days in case anyone wants to give a look."},"comment":{"url":"https://api.github.com/repos/FHIBioGroup/fafoom-dev/issues/comments/369044473","html_url":"https://github.com/FHIBioGroup/fafoom-dev/pull/5#issuecomment-369044473","issue_url":"https://api.github.com/repos/FHIBioGroup/fafoom-dev/issues/5","id":369044473,"user":{"login":"mahrossi","id":6743106,"avatar_url":"https://avatars2.githubusercontent.com/u/6743106?v=4","gravatar_id":"","url":"https://api.github.com/users/mahrossi","html_url":"https://github.com/mahrossi","followers_url":"https://api.github.com/users/mahrossi/followers","following_url":"https://api.github.com/users/mahrossi/following{/other_user}","gists_url":"https://api.github.com/users/mahrossi/gists{/gist_id}","starred_url":"https://api.github.com/users/mahrossi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mahrossi/subscriptions","organizations_url":"https://api.github.com/users/mahrossi/orgs","repos_url":"https://api.github.com/users/mahrossi/repos","events_url":"https://api.github.com/users/mahrossi/events{/privacy}","received_events_url":"https://api.github.com/users/mahrossi/received_events","type":"User","site_admin":false},"created_at":"2018-02-27T22:09:27Z","updated_at":"2018-02-27T22:09:27Z","author_association":"OWNER","body":"Hi Mateusz,\n\nGiven that the original FAFOOM is still alive in another completely\nseparate repository, there isn't a massive need to review every line.\nGeneral comments and improvements, or even testing the examples, would be\nmore than welcome of course. However, detailed testing\ncan even be done after the merge. Let us know what you can do until Friday\nand if you see any showstoppers (for us stuff works! :) ).\n\n\nOn Tue, Feb 27, 2018 at 10:48 PM, Mateusz Marianski <\nnotifications@github.com> wrote:\n\n> When do you want to commit? I want to read the code before.\n> I will try to review it this week but, being engaged in multiple tasks of\n> spurious scientific importance, I can't promise.\n>\n> On Tue, Feb 27, 2018 at 4:45 PM, Mariana Rossi <notifications@github.com>\n> wrote:\n>\n> > Actually I started reviewing and I have a comment :) .pyc and .out files\n> > should not be pushed into the repo. We can talk about how to remove them.\n> >\n> > —\n> > You are receiving this because your review was requested.\n> > Reply to this email directly, view it on GitHub\n> > <https://github.com/FHIBioGroup/fafoom-dev/pull/5#issuecomment-369037356\n> >,\n> > or mute the thread\n> > <https://github.com/notifications/unsubscribe-auth/\n> ALS4ElFPGlAQHQrCPY0o4FGe0uJxsu6yks5tZHeBgaJpZM4SVr7t>\n> > .\n> >\n>\n> —\n> You are receiving this because you authored the thread.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/FHIBioGroup/fafoom-dev/pull/5#issuecomment-369038279>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/AGbkQlw0kkA1zVhdG20n5UCsco7p-igtks5tZHg-gaJpZM4SVr7t>\n> .\n>\n"}}
{"actor":{"display_login":"mahrossi"}}