{"action":"created","issue":{"url":"https://api.github.com/repos/dealii/dealii/issues/1554","labels_url":"https://api.github.com/repos/dealii/dealii/issues/1554/labels{/name}","comments_url":"https://api.github.com/repos/dealii/dealii/issues/1554/comments","events_url":"https://api.github.com/repos/dealii/dealii/issues/1554/events","html_url":"https://github.com/dealii/dealii/issues/1554","id":105389811,"number":1554,"title":"FEValues and FEFaceValues should have symmetric Hessians","user":{"login":"davydden","id":8023934,"avatar_url":"https://avatars.githubusercontent.com/u/8023934?v=3","gravatar_id":"","url":"https://api.github.com/users/davydden","html_url":"https://github.com/davydden","followers_url":"https://api.github.com/users/davydden/followers","following_url":"https://api.github.com/users/davydden/following{/other_user}","gists_url":"https://api.github.com/users/davydden/gists{/gist_id}","starred_url":"https://api.github.com/users/davydden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davydden/subscriptions","organizations_url":"https://api.github.com/users/davydden/orgs","repos_url":"https://api.github.com/users/davydden/repos","events_url":"https://api.github.com/users/davydden/events{/privacy}","received_events_url":"https://api.github.com/users/davydden/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-09-08T14:13:26Z","updated_at":"2015-09-08T14:55:05Z","closed_at":null,"body":"subj.\r\n\r\nI guess the code was written before `SymmetricTensor` was available."},"comment":{"url":"https://api.github.com/repos/dealii/dealii/issues/comments/138591614","html_url":"https://github.com/dealii/dealii/issues/1554#issuecomment-138591614","issue_url":"https://api.github.com/repos/dealii/dealii/issues/1554","id":138591614,"user":{"login":"bangerth","id":7504421,"avatar_url":"https://avatars.githubusercontent.com/u/7504421?v=3","gravatar_id":"","url":"https://api.github.com/users/bangerth","html_url":"https://github.com/bangerth","followers_url":"https://api.github.com/users/bangerth/followers","following_url":"https://api.github.com/users/bangerth/following{/other_user}","gists_url":"https://api.github.com/users/bangerth/gists{/gist_id}","starred_url":"https://api.github.com/users/bangerth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bangerth/subscriptions","organizations_url":"https://api.github.com/users/bangerth/orgs","repos_url":"https://api.github.com/users/bangerth/repos","events_url":"https://api.github.com/users/bangerth/events{/privacy}","received_events_url":"https://api.github.com/users/bangerth/received_events","type":"User","site_admin":false},"created_at":"2015-09-08T14:55:05Z","updated_at":"2015-09-08T14:55:05Z","body":"Yes, correct. I thought the same this morning when seeing your change to the cosine function. It's true that the code predates the existence of the SymmetricTensor class. We should eventually fix this.\r\n\r\nOf course one could then argue that we need a TotallySymmetricTensor class also for third and higher derivatives so that `t[i][j][k] = t[m][n][p]` where `(mnp)` is any permutation (not necessarily of positive order) or `(ijk)`. But that's beyond my will to add right now :-)"}}