{"action":"created","issue":{"url":"https://api.github.com/repos/CASL/Futility/issues/138","repository_url":"https://api.github.com/repos/CASL/Futility","labels_url":"https://api.github.com/repos/CASL/Futility/issues/138/labels{/name}","comments_url":"https://api.github.com/repos/CASL/Futility/issues/138/comments","events_url":"https://api.github.com/repos/CASL/Futility/issues/138/events","html_url":"https://github.com/CASL/Futility/pull/138","id":364901180,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE4OTc3ODEx","number":138,"title":"Add new getAll procedures for parallel vectors","user":{"login":"rks171","id":2204328,"node_id":"MDQ6VXNlcjIyMDQzMjg=","avatar_url":"https://avatars2.githubusercontent.com/u/2204328?v=4","gravatar_id":"","url":"https://api.github.com/users/rks171","html_url":"https://github.com/rks171","followers_url":"https://api.github.com/users/rks171/followers","following_url":"https://api.github.com/users/rks171/following{/other_user}","gists_url":"https://api.github.com/users/rks171/gists{/gist_id}","starred_url":"https://api.github.com/users/rks171/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rks171/subscriptions","organizations_url":"https://api.github.com/users/rks171/orgs","repos_url":"https://api.github.com/users/rks171/repos","events_url":"https://api.github.com/users/rks171/events{/privacy}","received_events_url":"https://api.github.com/users/rks171/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-09-28T13:59:35Z","updated_at":"2018-09-28T14:32:13Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/CASL/Futility/pulls/138","html_url":"https://github.com/CASL/Futility/pull/138","diff_url":"https://github.com/CASL/Futility/pull/138.diff","patch_url":"https://github.com/CASL/Futility/pull/138.patch"},"body":"Description:\r\nThe original getAll procedure will not work for a parallel vector because it\r\nwill cause an out of bounds error in PETSc.  To get all values at once, the user\r\nneeds to pass in the global indices of the values to get and they can only\r\npass  in indices that belong to the proc.  I added a new procedure, 'getSelected',\r\nwhich can be used to do this, clarified that getAll should not be used for parallel\r\nvectors, added tests for getSelected, and then added a new set of parallel vector\r\ntests."},"comment":{"url":"https://api.github.com/repos/CASL/Futility/issues/comments/425454872","html_url":"https://github.com/CASL/Futility/pull/138#issuecomment-425454872","issue_url":"https://api.github.com/repos/CASL/Futility/issues/138","id":425454872,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNTQ1NDg3Mg==","user":{"login":"stimpsonsg","id":15932146,"node_id":"MDQ6VXNlcjE1OTMyMTQ2","avatar_url":"https://avatars1.githubusercontent.com/u/15932146?v=4","gravatar_id":"","url":"https://api.github.com/users/stimpsonsg","html_url":"https://github.com/stimpsonsg","followers_url":"https://api.github.com/users/stimpsonsg/followers","following_url":"https://api.github.com/users/stimpsonsg/following{/other_user}","gists_url":"https://api.github.com/users/stimpsonsg/gists{/gist_id}","starred_url":"https://api.github.com/users/stimpsonsg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stimpsonsg/subscriptions","organizations_url":"https://api.github.com/users/stimpsonsg/orgs","repos_url":"https://api.github.com/users/stimpsonsg/repos","events_url":"https://api.github.com/users/stimpsonsg/events{/privacy}","received_events_url":"https://api.github.com/users/stimpsonsg/received_events","type":"User","site_admin":false},"created_at":"2018-09-28T14:32:13Z","updated_at":"2018-09-28T14:32:13Z","author_association":"CONTRIBUTOR","body":"Yeah, if nlocal isn't set, it would make it nearly impossible to for the vector to know.  Let's pass on that.  I don't think we're using getAll for anything substantial anyway.\r\n\r\nLet's add new errors as DBC."}}
{"actor":{"display_login":"stimpsonsg"}}