{"action":"created","issue":{"url":"https://api.github.com/repos/mantidproject/mantid/issues/14504","labels_url":"https://api.github.com/repos/mantidproject/mantid/issues/14504/labels{/name}","comments_url":"https://api.github.com/repos/mantidproject/mantid/issues/14504/comments","events_url":"https://api.github.com/repos/mantidproject/mantid/issues/14504/events","html_url":"https://github.com/mantidproject/mantid/pull/14504","id":117766431,"number":14504,"title":"Powder ISIS script fails when ExistingV is no","user":{"login":"shahrozahmed1","id":13132173,"avatar_url":"https://avatars.githubusercontent.com/u/13132173?v=3","gravatar_id":"","url":"https://api.github.com/users/shahrozahmed1","html_url":"https://github.com/shahrozahmed1","followers_url":"https://api.github.com/users/shahrozahmed1/followers","following_url":"https://api.github.com/users/shahrozahmed1/following{/other_user}","gists_url":"https://api.github.com/users/shahrozahmed1/gists{/gist_id}","starred_url":"https://api.github.com/users/shahrozahmed1/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shahrozahmed1/subscriptions","organizations_url":"https://api.github.com/users/shahrozahmed1/orgs","repos_url":"https://api.github.com/users/shahrozahmed1/repos","events_url":"https://api.github.com/users/shahrozahmed1/events{/privacy}","received_events_url":"https://api.github.com/users/shahrozahmed1/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/mantidproject/mantid/labels/Component:%20Diffraction","name":"Component: Diffraction","color":"009800"},{"url":"https://api.github.com/repos/mantidproject/mantid/labels/Group:%20Powder","name":"Group: Powder","color":"bfe5bf"},{"url":"https://api.github.com/repos/mantidproject/mantid/labels/Quality:%20System%20Tests","name":"Quality: System Tests","color":"c7def8"}],"state":"open","locked":false,"assignee":{"login":"FedeMPouzols","id":5257172,"avatar_url":"https://avatars.githubusercontent.com/u/5257172?v=3","gravatar_id":"","url":"https://api.github.com/users/FedeMPouzols","html_url":"https://github.com/FedeMPouzols","followers_url":"https://api.github.com/users/FedeMPouzols/followers","following_url":"https://api.github.com/users/FedeMPouzols/following{/other_user}","gists_url":"https://api.github.com/users/FedeMPouzols/gists{/gist_id}","starred_url":"https://api.github.com/users/FedeMPouzols/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FedeMPouzols/subscriptions","organizations_url":"https://api.github.com/users/FedeMPouzols/orgs","repos_url":"https://api.github.com/users/FedeMPouzols/repos","events_url":"https://api.github.com/users/FedeMPouzols/events{/privacy}","received_events_url":"https://api.github.com/users/FedeMPouzols/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/mantidproject/mantid/milestones/4","html_url":"https://github.com/mantidproject/mantid/milestones/Release%203.6","labels_url":"https://api.github.com/repos/mantidproject/mantid/milestones/4/labels","id":1106937,"number":4,"title":"Release 3.6","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":584,"closed_issues":606,"state":"open","created_at":"2015-05-12T15:36:23Z","updated_at":"2015-11-19T17:16:17Z","due_on":"2016-01-30T05:00:00Z","closed_at":null},"comments":2,"created_at":"2015-11-19T08:38:44Z","updated_at":"2015-11-19T18:34:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mantidproject/mantid/pulls/14504","html_url":"https://github.com/mantidproject/mantid/pull/14504","diff_url":"https://github.com/mantidproject/mantid/pull/14504.diff","patch_url":"https://github.com/mantidproject/mantid/pull/14504.patch"},"body":"fixes #14433 \r\n\r\n> * [x] merge #14426 before this\r\n\r\n**To Test**\r\n\r\n- Run `ISISPowderDiffractionHrpdTest.py` system test locally\r\n- Check all tests pass\r\n- Once the system test has passed, make sure the value of `ExistingV` in the pref file `mtd2.pref` hasn't changed to `yes` from `no`\r\n\r\n- You could also run the python script manually via Mantid using the system test script which is utilising  `mtd2.pref` preference file. \r\n\r\n- New parameter `Write_ExtV` has been included for `cry_focus.focus_all(..)` with default value of `True`, this will enable the user to disable the script to auto replace `ExistingV` value in pref file. \r\n"},"comment":{"url":"https://api.github.com/repos/mantidproject/mantid/issues/comments/158149092","html_url":"https://github.com/mantidproject/mantid/pull/14504#issuecomment-158149092","issue_url":"https://api.github.com/repos/mantidproject/mantid/issues/14504","id":158149092,"user":{"login":"FedeMPouzols","id":5257172,"avatar_url":"https://avatars.githubusercontent.com/u/5257172?v=3","gravatar_id":"","url":"https://api.github.com/users/FedeMPouzols","html_url":"https://github.com/FedeMPouzols","followers_url":"https://api.github.com/users/FedeMPouzols/followers","following_url":"https://api.github.com/users/FedeMPouzols/following{/other_user}","gists_url":"https://api.github.com/users/FedeMPouzols/gists{/gist_id}","starred_url":"https://api.github.com/users/FedeMPouzols/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FedeMPouzols/subscriptions","organizations_url":"https://api.github.com/users/FedeMPouzols/orgs","repos_url":"https://api.github.com/users/FedeMPouzols/repos","events_url":"https://api.github.com/users/FedeMPouzols/events{/privacy}","received_events_url":"https://api.github.com/users/FedeMPouzols/received_events","type":"User","site_admin":false},"created_at":"2015-11-19T18:34:30Z","updated_at":"2015-11-19T18:34:30Z","body":"@shahrozahmed1: Pylint complains about `global DIFF_PLACES` in `ISISPowderDiffractionHrpdTest.py`. Do you need that variable as a global?"}}