{"action":"reopened","issue":{"url":"https://api.github.com/repos/trilinos/Trilinos/issues/41","labels_url":"https://api.github.com/repos/trilinos/Trilinos/issues/41/labels{/name}","comments_url":"https://api.github.com/repos/trilinos/Trilinos/issues/41/comments","events_url":"https://api.github.com/repos/trilinos/Trilinos/issues/41/events","html_url":"https://github.com/trilinos/Trilinos/issues/41","id":120063881,"number":41,"title":"Tpetra::CrsMatrix::getLocalDiagCopy: Thread-parallelize diagonal extraction","user":{"login":"mhoemmen","id":926747,"avatar_url":"https://avatars.githubusercontent.com/u/926747?v=3","gravatar_id":"","url":"https://api.github.com/users/mhoemmen","html_url":"https://github.com/mhoemmen","followers_url":"https://api.github.com/users/mhoemmen/followers","following_url":"https://api.github.com/users/mhoemmen/following{/other_user}","gists_url":"https://api.github.com/users/mhoemmen/gists{/gist_id}","starred_url":"https://api.github.com/users/mhoemmen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhoemmen/subscriptions","organizations_url":"https://api.github.com/users/mhoemmen/orgs","repos_url":"https://api.github.com/users/mhoemmen/repos","events_url":"https://api.github.com/users/mhoemmen/events{/privacy}","received_events_url":"https://api.github.com/users/mhoemmen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-12-03T00:02:56Z","updated_at":"2015-12-03T18:13:59Z","closed_at":null,"body":"@trilinos/tpetra Performance tests with Nalu discovered that Ifpack2::Relaxation Jacobi setup had a sequential section in CrsMatrix::getLocalDiagCopy (the two-argument version, though it's easy enough to optimize the one-argument version too).  Nalu developers experimented with putting a parallel_for loop around diagonal extraction in getLocalDiagCopy, and it worked great."}}