{"action":"created","comment":{"url":"https://api.github.com/repos/mantidproject/mantid/pulls/comments/58073353","id":58073353,"diff_hunk":"@@ -4,7 +4,7 @@\n #include \"MantidKernel/System.h\"\n #include \"MantidQtAPI/AlgorithmRunner.h\"\n #include \"MantidQtCustomInterfaces/Reflectometry/IReflPresenter.h\"\n-#include \"MantidQtCustomInterfaces/Reflectometry/ReflCommandBase.h\"\n+#include \"MantidQtCustomInterfaces/Reflectometry/ReflCommand.h\"","path":"MantidQt/CustomInterfaces/inc/MantidQtCustomInterfaces/Reflectometry/ReflMainView.h","position":5,"original_position":5,"commit_id":"d1174f7e758384035c521249d825773725f2b512","original_commit_id":"d1174f7e758384035c521249d825773725f2b512","user":{"login":"OwenArnold","id":1109536,"avatar_url":"https://avatars.githubusercontent.com/u/1109536?v=3","gravatar_id":"","url":"https://api.github.com/users/OwenArnold","html_url":"https://github.com/OwenArnold","followers_url":"https://api.github.com/users/OwenArnold/followers","following_url":"https://api.github.com/users/OwenArnold/following{/other_user}","gists_url":"https://api.github.com/users/OwenArnold/gists{/gist_id}","starred_url":"https://api.github.com/users/OwenArnold/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OwenArnold/subscriptions","organizations_url":"https://api.github.com/users/OwenArnold/orgs","repos_url":"https://api.github.com/users/OwenArnold/repos","events_url":"https://api.github.com/users/OwenArnold/events{/privacy}","received_events_url":"https://api.github.com/users/OwenArnold/received_events","type":"User","site_admin":false},"body":"You could actually avoid the include here and foward declare a lot of this instead. Unfortunately that would mean that you couldn't use your `typedefs`, but it would help decouple your compilation units.","created_at":"2016-03-31T15:23:47Z","updated_at":"2016-03-31T15:23:47Z","html_url":"https://github.com/mantidproject/mantid/pull/15670#discussion_r58073353","pull_request_url":"https://api.github.com/repos/mantidproject/mantid/pulls/15670","_links":{"self":{"href":"https://api.github.com/repos/mantidproject/mantid/pulls/comments/58073353"},"html":{"href":"https://github.com/mantidproject/mantid/pull/15670#discussion_r58073353"},"pull_request":{"href":"https://api.github.com/repos/mantidproject/mantid/pulls/15670"}}},"pull_request":{"url":"https://api.github.com/repos/mantidproject/mantid/pulls/15670","id":62878586,"html_url":"https://github.com/mantidproject/mantid/pull/15670","diff_url":"https://github.com/mantidproject/mantid/pull/15670.diff","patch_url":"https://github.com/mantidproject/mantid/pull/15670.patch","issue_url":"https://api.github.com/repos/mantidproject/mantid/issues/15670","number":15670,"state":"open","locked":false,"title":"Refactor ReflMainView and ReflMainViewPresenter","user":{"login":"raquelalvarezbanos","id":8956985,"avatar_url":"https://avatars.githubusercontent.com/u/8956985?v=3","gravatar_id":"","url":"https://api.github.com/users/raquelalvarezbanos","html_url":"https://github.com/raquelalvarezbanos","followers_url":"https://api.github.com/users/raquelalvarezbanos/followers","following_url":"https://api.github.com/users/raquelalvarezbanos/following{/other_user}","gists_url":"https://api.github.com/users/raquelalvarezbanos/gists{/gist_id}","starred_url":"https://api.github.com/users/raquelalvarezbanos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raquelalvarezbanos/subscriptions","organizations_url":"https://api.github.com/users/raquelalvarezbanos/orgs","repos_url":"https://api.github.com/users/raquelalvarezbanos/repos","events_url":"https://api.github.com/users/raquelalvarezbanos/events{/privacy}","received_events_url":"https://api.github.com/users/raquelalvarezbanos/received_events","type":"User","site_admin":false},"body":"This is the first step in #14732.\r\n- `ReflMainView` has been split into two different classes: `ReflMainView` and `ReflTableView`. `ReflTableView` has the functionality related to the processing table: options to open/save a table, add/delete rows, etc, even the \"Reflectometry UI Options\" have been transferred to `ReflTableView` as they refer to the processing table.\r\n- Each class has its own presenter, and each presenter has its own unit tests:\r\n - `ReflMainView` -> `ReflMainViewPresenter` -> `ReflMainViewPresenterTest`\r\n - `ReflTableView` -> `ReflTableViewPresenter` -> `ReflTableViewPresenterTest`\r\n- Files defining the mock objects have also been split:\r\n - `ReflMainViewMockObjects`\r\n - `ReflTableViewMockObjects`\r\n\r\nThe following diagram tries to represent the new structure of the interface (apologies if it does not follow the standards, but I hope it is clear enough):\r\n\r\n![newdiagram](https://cloud.githubusercontent.com/assets/8956985/13775106/a5cf2bc0-ea9a-11e5-83f1-758d170b535e.png)\r\n\r\n- The left section represents reflectometry related stuff, which includes the abstract presenter `IReflPresenter` defining the notification interface, the concrete presenter `ReflMainViewPresenter`, which does all the UI logic, and interacts with the abstract view, `ReflMainView`. \r\n- The concrete view, `QtReflMainView` has a component `QReflTableView`. This is now a new widget independent from `QtReflMainView` (although some communication between them is needed).\r\n- The right section reseprents all the things that should be unrelated to reflectometry (at the moment, they are still related to this specific area, but they will be generalized in the next issue). Essentially it consists of an abstract view, `ReflTableView`, that is updated by a `ReflTableViewPresenter` implementing an abstract `IReflTablePresenter`.\r\n\r\n**To test:**\r\n\r\n1. Please consider the above diagram and the following comments and make sure I am following the correct approach:\r\n\r\n- Some changes are visible to users: most of the funcionality has been moved to `ReflTable` (by `ReflTable` I mean both the view and presenter). This means that most of the options in the menus have been removed and moved to the tool bar. Actually, only the `Slit Calculator` remains in the `Reflectometry` menu bar.\r\n- The existing progress bar now belongs to `ReflTable`. I've added a new progress bar in the `Search Runs` section which will report progress when transferring runs. The original progress bar will only report processing. One thing to consider is that now we have two different `ReflProgress` classes defined in each presenter. See [here](https://github.com/mantidproject/mantid/blob/14921_Refactor_ReflMainView_and_ReflMainViewPresenter/MantidQt/CustomInterfaces/src/Reflectometry/ReflMainViewPresenter.cpp#L28-L52) and [here](https://github.com/mantidproject/mantid/blob/14921_Refactor_ReflMainView_and_ReflMainViewPresenter/MantidQt/CustomInterfaces/src/Reflectometry/ReflTableViewPresenter.cpp#L44-L68). Should we merge them into a single class and move it to a separate header that can be used by both presenters? If so, in which directory or subdirectory should this class be?\r\n- Some communication between `QtReflMainView` and `QReflTableView` is needed, specifically in the following cases:\r\n - When transferring runs to the table, `QtReflMainView` needs to get from its presenter the set of runs to transfer and then send them to `QReflTableView` so they can be added to the table. This is done [here](https://github.com/mantidproject/mantid/blob/14921_Refactor_ReflMainView_and_ReflMainViewPresenter/MantidQt/CustomInterfaces/src/Reflectometry/QtReflMainView.cpp#L161-L164).\r\n - Instrument combo boxes need to be synchronized, whenever one changes, the other should be updated accordingly. This is done [here](https://github.com/mantidproject/mantid/blob/14921_Refactor_ReflMainView_and_ReflMainViewPresenter/MantidQt/CustomInterfaces/src/Reflectometry/QtReflMainView.cpp#L52-L58)\r\n - `QReflTableView` needs to run python code, for instance when the user loads a TBL file or plots a row. As it is not a `UserSubWindow` anymore, but a `MantidWidget`, when the signal `runAsPythonScript(const QString &, bool)` is emitted, it has to be re-emitted by `QtReflMainView`. This is done [here](https://github.com/mantidproject/mantid/blob/14921_Refactor_ReflMainView_and_ReflMainViewPresenter/MantidQt/CustomInterfaces/src/Reflectometry/QtReflMainView.cpp#L60-L62)\r\n\r\n2. Code review\r\n3. Please make sure all the existing functionality still works (transferring runs, processing, importing/exporting a TBL, plotting rows and groups). Most of it is tested in unit tests.\r\n\r\nFixes #14921 \r\n\r\nRelease notes have been updated.\r\n\r\n---\r\n\r\n#### Reviewer ####\r\n\r\nPlease comment on the following ([full description](http://www.mantidproject.org/Individual_Ticket_Testing)):\r\n\r\n##### Code Review #####\r\n\r\n- [ ] Is the code of an acceptable quality?\r\n- [ ] Does the code conform to the coding standards? Is it well structured with small focussed classes/methods/functions?\r\n- [ ] Are there unit/system tests in place? Are the unit tests small and test the a class in isolation?\r\n- [ ] If there are changes in the release notes then do they describe the changes appropriately?\r\n\r\n##### Functional Tests #####\r\n\r\n- [ ] Do changes function as described? Add comments below that describe the tests performed?\r\n- [ ] How do the changes handle unexpected situations, e.g. bad input?\r\n- [ ] Has the relevant documentation been added/updated?\r\n- [ ] Is user-facing documentation written in a user-friendly manner?\r\n- [ ] Has developer documentation been updated if required?\r\n\r\n- Does everything look good? Comment with the ship it emoji but don't merge. A member of `@mantidproject/gatekeepers` will take care of it.\r\n","created_at":"2016-03-15T09:49:58Z","updated_at":"2016-03-31T15:23:47Z","closed_at":null,"merged_at":null,"merge_commit_sha":"211c26766036e7a2f701269dda6aa5f18b9500a6","assignee":{"login":"OwenArnold","id":1109536,"avatar_url":"https://avatars.githubusercontent.com/u/1109536?v=3","gravatar_id":"","url":"https://api.github.com/users/OwenArnold","html_url":"https://github.com/OwenArnold","followers_url":"https://api.github.com/users/OwenArnold/followers","following_url":"https://api.github.com/users/OwenArnold/following{/other_user}","gists_url":"https://api.github.com/users/OwenArnold/gists{/gist_id}","starred_url":"https://api.github.com/users/OwenArnold/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OwenArnold/subscriptions","organizations_url":"https://api.github.com/users/OwenArnold/orgs","repos_url":"https://api.github.com/users/OwenArnold/repos","events_url":"https://api.github.com/users/OwenArnold/events{/privacy}","received_events_url":"https://api.github.com/users/OwenArnold/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/mantidproject/mantid/milestones/5","html_url":"https://github.com/mantidproject/mantid/milestones/Release%203.7","labels_url":"https://api.github.com/repos/mantidproject/mantid/milestones/5/labels","id":1106939,"number":5,"title":"Release 3.7","description":"","creator":{"login":"peterfpeterson","id":404003,"avatar_url":"https://avatars.githubusercontent.com/u/404003?v=3","gravatar_id":"","url":"https://api.github.com/users/peterfpeterson","html_url":"https://github.com/peterfpeterson","followers_url":"https://api.github.com/users/peterfpeterson/followers","following_url":"https://api.github.com/users/peterfpeterson/following{/other_user}","gists_url":"https://api.github.com/users/peterfpeterson/gists{/gist_id}","starred_url":"https://api.github.com/users/peterfpeterson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peterfpeterson/subscriptions","organizations_url":"https://api.github.com/users/peterfpeterson/orgs","repos_url":"https://api.github.com/users/peterfpeterson/repos","events_url":"https://api.github.com/users/peterfpeterson/events{/privacy}","received_events_url":"https://api.github.com/users/peterfpeterson/received_events","type":"User","site_admin":false},"open_issues":587,"closed_issues":455,"state":"open","created_at":"2015-05-12T15:38:13Z","updated_at":"2016-03-31T14:08:30Z","due_on":"2016-06-05T23:00:00Z","closed_at":null},"commits_url":"https://api.github.com/repos/mantidproject/mantid/pulls/15670/commits","review_comments_url":"https://api.github.com/repos/mantidproject/mantid/pulls/15670/comments","review_comment_url":"https://api.github.com/repos/mantidproject/mantid/pulls/comments{/number}","comments_url":"https://api.github.com/repos/mantidproject/mantid/issues/15670/comments","statuses_url":"https://api.github.com/repos/mantidproject/mantid/statuses/d1174f7e758384035c521249d825773725f2b512","head":{"label":"mantidproject:14921_Refactor_ReflMainView_and_ReflMainViewPresenter","ref":"14921_Refactor_ReflMainView_and_ReflMainViewPresenter","sha":"d1174f7e758384035c521249d825773725f2b512","user":{"login":"mantidproject","id":671496,"avatar_url":"https://avatars.githubusercontent.com/u/671496?v=3","gravatar_id":"","url":"https://api.github.com/users/mantidproject","html_url":"https://github.com/mantidproject","followers_url":"https://api.github.com/users/mantidproject/followers","following_url":"https://api.github.com/users/mantidproject/following{/other_user}","gists_url":"https://api.github.com/users/mantidproject/gists{/gist_id}","starred_url":"https://api.github.com/users/mantidproject/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mantidproject/subscriptions","organizations_url":"https://api.github.com/users/mantidproject/orgs","repos_url":"https://api.github.com/users/mantidproject/repos","events_url":"https://api.github.com/users/mantidproject/events{/privacy}","received_events_url":"https://api.github.com/users/mantidproject/received_events","type":"Organization","site_admin":false},"repo":{"id":2594116,"name":"mantid","full_name":"mantidproject/mantid","owner":{"login":"mantidproject","id":671496,"avatar_url":"https://avatars.githubusercontent.com/u/671496?v=3","gravatar_id":"","url":"https://api.github.com/users/mantidproject","html_url":"https://github.com/mantidproject","followers_url":"https://api.github.com/users/mantidproject/followers","following_url":"https://api.github.com/users/mantidproject/following{/other_user}","gists_url":"https://api.github.com/users/mantidproject/gists{/gist_id}","starred_url":"https://api.github.com/users/mantidproject/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mantidproject/subscriptions","organizations_url":"https://api.github.com/users/mantidproject/orgs","repos_url":"https://api.github.com/users/mantidproject/repos","events_url":"https://api.github.com/users/mantidproject/events{/privacy}","received_events_url":"https://api.github.com/users/mantidproject/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/mantidproject/mantid","description":"Main repository for Mantid code","fork":false,"url":"https://api.github.com/repos/mantidproject/mantid","forks_url":"https://api.github.com/repos/mantidproject/mantid/forks","keys_url":"https://api.github.com/repos/mantidproject/mantid/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mantidproject/mantid/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mantidproject/mantid/teams","hooks_url":"https://api.github.com/repos/mantidproject/mantid/hooks","issue_events_url":"https://api.github.com/repos/mantidproject/mantid/issues/events{/number}","events_url":"https://api.github.com/repos/mantidproject/mantid/events","assignees_url":"https://api.github.com/repos/mantidproject/mantid/assignees{/user}","branches_url":"https://api.github.com/repos/mantidproject/mantid/branches{/branch}","tags_url":"https://api.github.com/repos/mantidproject/mantid/tags","blobs_url":"https://api.github.com/repos/mantidproject/mantid/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mantidproject/mantid/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mantidproject/mantid/git/refs{/sha}","trees_url":"https://api.github.com/repos/mantidproject/mantid/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mantidproject/mantid/statuses/{sha}","languages_url":"https://api.github.com/repos/mantidproject/mantid/languages","stargazers_url":"https://api.github.com/repos/mantidproject/mantid/stargazers","contributors_url":"https://api.github.com/repos/mantidproject/mantid/contributors","subscribers_url":"https://api.github.com/repos/mantidproject/mantid/subscribers","subscription_url":"https://api.github.com/repos/mantidproject/mantid/subscription","commits_url":"https://api.github.com/repos/mantidproject/mantid/commits{/sha}","git_commits_url":"https://api.github.com/repos/mantidproject/mantid/git/commits{/sha}","comments_url":"https://api.github.com/repos/mantidproject/mantid/comments{/number}","issue_comment_url":"https://api.github.com/repos/mantidproject/mantid/issues/comments{/number}","contents_url":"https://api.github.com/repos/mantidproject/mantid/contents/{+path}","compare_url":"https://api.github.com/repos/mantidproject/mantid/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mantidproject/mantid/merges","archive_url":"https://api.github.com/repos/mantidproject/mantid/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mantidproject/mantid/downloads","issues_url":"https://api.github.com/repos/mantidproject/mantid/issues{/number}","pulls_url":"https://api.github.com/repos/mantidproject/mantid/pulls{/number}","milestones_url":"https://api.github.com/repos/mantidproject/mantid/milestones{/number}","notifications_url":"https://api.github.com/repos/mantidproject/mantid/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mantidproject/mantid/labels{/name}","releases_url":"https://api.github.com/repos/mantidproject/mantid/releases{/id}","deployments_url":"https://api.github.com/repos/mantidproject/mantid/deployments","created_at":"2011-10-17T19:07:38Z","updated_at":"2016-03-29T09:12:44Z","pushed_at":"2016-03-31T15:01:34Z","git_url":"git://github.com/mantidproject/mantid.git","ssh_url":"git@github.com:mantidproject/mantid.git","clone_url":"https://github.com/mantidproject/mantid.git","svn_url":"https://github.com/mantidproject/mantid","homepage":"www.mantidproject.org","size":403320,"stargazers_count":40,"watchers_count":40,"language":"C++","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":29,"mirror_url":null,"open_issues_count":1508,"forks":29,"open_issues":1508,"watchers":40,"default_branch":"master"}},"base":{"label":"mantidproject:master","ref":"master","sha":"d154e462e4ee2c660f7c50c7906f031968109cb5","user":{"login":"mantidproject","id":671496,"avatar_url":"https://avatars.githubusercontent.com/u/671496?v=3","gravatar_id":"","url":"https://api.github.com/users/mantidproject","html_url":"https://github.com/mantidproject","followers_url":"https://api.github.com/users/mantidproject/followers","following_url":"https://api.github.com/users/mantidproject/following{/other_user}","gists_url":"https://api.github.com/users/mantidproject/gists{/gist_id}","starred_url":"https://api.github.com/users/mantidproject/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mantidproject/subscriptions","organizations_url":"https://api.github.com/users/mantidproject/orgs","repos_url":"https://api.github.com/users/mantidproject/repos","events_url":"https://api.github.com/users/mantidproject/events{/privacy}","received_events_url":"https://api.github.com/users/mantidproject/received_events","type":"Organization","site_admin":false},"repo":{"id":2594116,"name":"mantid","full_name":"mantidproject/mantid","owner":{"login":"mantidproject","id":671496,"avatar_url":"https://avatars.githubusercontent.com/u/671496?v=3","gravatar_id":"","url":"https://api.github.com/users/mantidproject","html_url":"https://github.com/mantidproject","followers_url":"https://api.github.com/users/mantidproject/followers","following_url":"https://api.github.com/users/mantidproject/following{/other_user}","gists_url":"https://api.github.com/users/mantidproject/gists{/gist_id}","starred_url":"https://api.github.com/users/mantidproject/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mantidproject/subscriptions","organizations_url":"https://api.github.com/users/mantidproject/orgs","repos_url":"https://api.github.com/users/mantidproject/repos","events_url":"https://api.github.com/users/mantidproject/events{/privacy}","received_events_url":"https://api.github.com/users/mantidproject/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/mantidproject/mantid","description":"Main repository for Mantid code","fork":false,"url":"https://api.github.com/repos/mantidproject/mantid","forks_url":"https://api.github.com/repos/mantidproject/mantid/forks","keys_url":"https://api.github.com/repos/mantidproject/mantid/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mantidproject/mantid/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mantidproject/mantid/teams","hooks_url":"https://api.github.com/repos/mantidproject/mantid/hooks","issue_events_url":"https://api.github.com/repos/mantidproject/mantid/issues/events{/number}","events_url":"https://api.github.com/repos/mantidproject/mantid/events","assignees_url":"https://api.github.com/repos/mantidproject/mantid/assignees{/user}","branches_url":"https://api.github.com/repos/mantidproject/mantid/branches{/branch}","tags_url":"https://api.github.com/repos/mantidproject/mantid/tags","blobs_url":"https://api.github.com/repos/mantidproject/mantid/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mantidproject/mantid/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mantidproject/mantid/git/refs{/sha}","trees_url":"https://api.github.com/repos/mantidproject/mantid/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mantidproject/mantid/statuses/{sha}","languages_url":"https://api.github.com/repos/mantidproject/mantid/languages","stargazers_url":"https://api.github.com/repos/mantidproject/mantid/stargazers","contributors_url":"https://api.github.com/repos/mantidproject/mantid/contributors","subscribers_url":"https://api.github.com/repos/mantidproject/mantid/subscribers","subscription_url":"https://api.github.com/repos/mantidproject/mantid/subscription","commits_url":"https://api.github.com/repos/mantidproject/mantid/commits{/sha}","git_commits_url":"https://api.github.com/repos/mantidproject/mantid/git/commits{/sha}","comments_url":"https://api.github.com/repos/mantidproject/mantid/comments{/number}","issue_comment_url":"https://api.github.com/repos/mantidproject/mantid/issues/comments{/number}","contents_url":"https://api.github.com/repos/mantidproject/mantid/contents/{+path}","compare_url":"https://api.github.com/repos/mantidproject/mantid/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mantidproject/mantid/merges","archive_url":"https://api.github.com/repos/mantidproject/mantid/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mantidproject/mantid/downloads","issues_url":"https://api.github.com/repos/mantidproject/mantid/issues{/number}","pulls_url":"https://api.github.com/repos/mantidproject/mantid/pulls{/number}","milestones_url":"https://api.github.com/repos/mantidproject/mantid/milestones{/number}","notifications_url":"https://api.github.com/repos/mantidproject/mantid/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mantidproject/mantid/labels{/name}","releases_url":"https://api.github.com/repos/mantidproject/mantid/releases{/id}","deployments_url":"https://api.github.com/repos/mantidproject/mantid/deployments","created_at":"2011-10-17T19:07:38Z","updated_at":"2016-03-29T09:12:44Z","pushed_at":"2016-03-31T15:01:34Z","git_url":"git://github.com/mantidproject/mantid.git","ssh_url":"git@github.com:mantidproject/mantid.git","clone_url":"https://github.com/mantidproject/mantid.git","svn_url":"https://github.com/mantidproject/mantid","homepage":"www.mantidproject.org","size":403320,"stargazers_count":40,"watchers_count":40,"language":"C++","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":29,"mirror_url":null,"open_issues_count":1508,"forks":29,"open_issues":1508,"watchers":40,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/mantidproject/mantid/pulls/15670"},"html":{"href":"https://github.com/mantidproject/mantid/pull/15670"},"issue":{"href":"https://api.github.com/repos/mantidproject/mantid/issues/15670"},"comments":{"href":"https://api.github.com/repos/mantidproject/mantid/issues/15670/comments"},"review_comments":{"href":"https://api.github.com/repos/mantidproject/mantid/pulls/15670/comments"},"review_comment":{"href":"https://api.github.com/repos/mantidproject/mantid/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/mantidproject/mantid/pulls/15670/commits"},"statuses":{"href":"https://api.github.com/repos/mantidproject/mantid/statuses/d1174f7e758384035c521249d825773725f2b512"}}}}