{"action":"created","issue":{"url":"https://api.github.com/repos/geodynamics/aspect/issues/567","labels_url":"https://api.github.com/repos/geodynamics/aspect/issues/567/labels{/name}","comments_url":"https://api.github.com/repos/geodynamics/aspect/issues/567/comments","events_url":"https://api.github.com/repos/geodynamics/aspect/issues/567/events","html_url":"https://github.com/geodynamics/aspect/pull/567","id":99511495,"number":567,"title":"remove an unwanted loop","user":{"login":"jdannberg","id":7517347,"avatar_url":"https://avatars.githubusercontent.com/u/7517347?v=3","gravatar_id":"","url":"https://api.github.com/users/jdannberg","html_url":"https://github.com/jdannberg","followers_url":"https://api.github.com/users/jdannberg/followers","following_url":"https://api.github.com/users/jdannberg/following{/other_user}","gists_url":"https://api.github.com/users/jdannberg/gists{/gist_id}","starred_url":"https://api.github.com/users/jdannberg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdannberg/subscriptions","organizations_url":"https://api.github.com/users/jdannberg/orgs","repos_url":"https://api.github.com/users/jdannberg/repos","events_url":"https://api.github.com/users/jdannberg/events{/privacy}","received_events_url":"https://api.github.com/users/jdannberg/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-08-06T19:53:52Z","updated_at":"2015-08-18T17:28:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/geodynamics/aspect/pulls/567","html_url":"https://github.com/geodynamics/aspect/pull/567","diff_url":"https://github.com/geodynamics/aspect/pull/567.diff","patch_url":"https://github.com/geodynamics/aspect/pull/567.patch"},"body":"I think we do not need this loop (as we already do the loop over all quadrature points within the individual averaging functions), and in the case of the averaging schemes that do not generate one average for all of the points in one cell, but calculate position-dependent, different values for every point, this even generates different results (because we average several times). \r\nPlease correct me if I am wrong!"},"comment":{"url":"https://api.github.com/repos/geodynamics/aspect/issues/comments/132288676","html_url":"https://github.com/geodynamics/aspect/pull/567#issuecomment-132288676","issue_url":"https://api.github.com/repos/geodynamics/aspect/issues/567","id":132288676,"user":{"login":"tjhei","id":1531285,"avatar_url":"https://avatars.githubusercontent.com/u/1531285?v=3","gravatar_id":"","url":"https://api.github.com/users/tjhei","html_url":"https://github.com/tjhei","followers_url":"https://api.github.com/users/tjhei/followers","following_url":"https://api.github.com/users/tjhei/following{/other_user}","gists_url":"https://api.github.com/users/tjhei/gists{/gist_id}","starred_url":"https://api.github.com/users/tjhei/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tjhei/subscriptions","organizations_url":"https://api.github.com/users/tjhei/orgs","repos_url":"https://api.github.com/users/tjhei/repos","events_url":"https://api.github.com/users/tjhei/events{/privacy}","received_events_url":"https://api.github.com/users/tjhei/received_events","type":"User","site_admin":false},"created_at":"2015-08-18T17:28:43Z","updated_at":"2015-08-18T17:28:43Z","body":"Makes sense. We could move the if inside the ```average()``` function though.\r\n\r\nPlease add a comment and fix what the tester is complaining about."}}