{"action":"closed","issue":{"url":"https://api.github.com/repos/mantidproject/mantid/issues/11966","labels_url":"https://api.github.com/repos/mantidproject/mantid/issues/11966/labels{/name}","comments_url":"https://api.github.com/repos/mantidproject/mantid/issues/11966/comments","events_url":"https://api.github.com/repos/mantidproject/mantid/issues/11966/events","html_url":"https://github.com/mantidproject/mantid/issues/11966","id":84729099,"number":11966,"title":"Fix CreateSampleWorkspace","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},"labels":[{"url":"https://api.github.com/repos/mantidproject/mantid/labels/Component:%20Framework","name":"Component: Framework","color":"009800"}],"state":"closed","locked":false,"assignee":{"login":"NickDraper","id":1017173,"avatar_url":"https://avatars.githubusercontent.com/u/1017173?v=3","gravatar_id":"","url":"https://api.github.com/users/NickDraper","html_url":"https://github.com/NickDraper","followers_url":"https://api.github.com/users/NickDraper/followers","following_url":"https://api.github.com/users/NickDraper/following{/other_user}","gists_url":"https://api.github.com/users/NickDraper/gists{/gist_id}","starred_url":"https://api.github.com/users/NickDraper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/NickDraper/subscriptions","organizations_url":"https://api.github.com/users/NickDraper/orgs","repos_url":"https://api.github.com/users/NickDraper/repos","events_url":"https://api.github.com/users/NickDraper/events{/privacy}","received_events_url":"https://api.github.com/users/NickDraper/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/mantidproject/mantid/milestones/3","html_url":"https://github.com/mantidproject/mantid/milestones/Release%203.5","labels_url":"https://api.github.com/repos/mantidproject/mantid/milestones/3/labels","id":1058985,"number":3,"title":"Release 3.5","description":"This is a placeholder in git for the [trac Release 3.5 milestone](http://trac.mantidproject.org/mantid/milestone/Release%203.5)","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":426,"closed_issues":804,"state":"open","created_at":"2015-04-08T18:44:37Z","updated_at":"2015-08-12T12:35:10Z","due_on":"2015-10-01T04:00:00Z","closed_at":null},"comments":1,"created_at":"2015-02-19T12:15:47Z","updated_at":"2015-08-12T12:35:10Z","closed_at":"2015-08-12T12:35:10Z","body":"This issue was originally [TRAC 11127](http://trac.mantidproject.org/mantid/ticket/11127)\n\nI've just had to fix CreateSampleWorkspace because the reference frame does not correspond with the acutal layout of the test instrument made.\n\nI now find that we have a property called \"BankPixelWidth\" on this algorithm. This doesn't actually control the pixel width. It controls the number of pixels! See the source code.\n\nThe usage examples are good. But It would be extremely helpful to see what the layout of the instrument really is. If you create a sample workspace around an instrument it is often useful to be able to calculate what the L1 & L2 distances are to support your calculations. There is no way of finding that out at present aside from inspecting the source code. I suggest having a schematic (including source, sample locations and detectors etc) so that people can see what sort of instrument and workspace they're getting back. \n\nIn summary:\n\n* Fix the pixel size issue one way or the other, but allow users to calculate size from number (vica versa) by including the Bank Width height somewhere, probably easiest in the documentation.\n* Add a schematic of the instrument layout to the documentation\n\n\n\nhttp://docs.mantidproject.org/nightly/algorithms/CreateSampleWorkspace-v1.html \n "}}