{"action":"created","issue":{"url":"https://api.github.com/repos/vincentlab/PyFR/issues/70","labels_url":"https://api.github.com/repos/vincentlab/PyFR/issues/70/labels{/name}","comments_url":"https://api.github.com/repos/vincentlab/PyFR/issues/70/comments","events_url":"https://api.github.com/repos/vincentlab/PyFR/issues/70/events","html_url":"https://github.com/vincentlab/PyFR/pull/70","id":100738324,"number":70,"title":"Improve filtering for tensor product elements.","user":{"login":"FreddieWitherden","id":528611,"avatar_url":"https://avatars.githubusercontent.com/u/528611?v=3","gravatar_id":"","url":"https://api.github.com/users/FreddieWitherden","html_url":"https://github.com/FreddieWitherden","followers_url":"https://api.github.com/users/FreddieWitherden/followers","following_url":"https://api.github.com/users/FreddieWitherden/following{/other_user}","gists_url":"https://api.github.com/users/FreddieWitherden/gists{/gist_id}","starred_url":"https://api.github.com/users/FreddieWitherden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FreddieWitherden/subscriptions","organizations_url":"https://api.github.com/users/FreddieWitherden/orgs","repos_url":"https://api.github.com/users/FreddieWitherden/repos","events_url":"https://api.github.com/users/FreddieWitherden/events{/privacy}","received_events_url":"https://api.github.com/users/FreddieWitherden/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"pv101","id":2280961,"avatar_url":"https://avatars.githubusercontent.com/u/2280961?v=3","gravatar_id":"","url":"https://api.github.com/users/pv101","html_url":"https://github.com/pv101","followers_url":"https://api.github.com/users/pv101/followers","following_url":"https://api.github.com/users/pv101/following{/other_user}","gists_url":"https://api.github.com/users/pv101/gists{/gist_id}","starred_url":"https://api.github.com/users/pv101/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pv101/subscriptions","organizations_url":"https://api.github.com/users/pv101/orgs","repos_url":"https://api.github.com/users/pv101/repos","events_url":"https://api.github.com/users/pv101/events{/privacy}","received_events_url":"https://api.github.com/users/pv101/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-08-13T10:52:43Z","updated_at":"2015-09-21T14:00:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/vincentlab/PyFR/pulls/70","html_url":"https://github.com/vincentlab/PyFR/pull/70","diff_url":"https://github.com/vincentlab/PyFR/pull/70.diff","patch_url":"https://github.com/vincentlab/PyFR/pull/70.patch"},"body":"This resolves an issue around filtering tensor product elements that could result in the filtering being more aggressive than necessary."},"comment":{"url":"https://api.github.com/repos/vincentlab/PyFR/issues/comments/141990016","html_url":"https://github.com/vincentlab/PyFR/pull/70#issuecomment-141990016","issue_url":"https://api.github.com/repos/vincentlab/PyFR/issues/70","id":141990016,"user":{"login":"Aerojspark","id":8792917,"avatar_url":"https://avatars.githubusercontent.com/u/8792917?v=3","gravatar_id":"","url":"https://api.github.com/users/Aerojspark","html_url":"https://github.com/Aerojspark","followers_url":"https://api.github.com/users/Aerojspark/followers","following_url":"https://api.github.com/users/Aerojspark/following{/other_user}","gists_url":"https://api.github.com/users/Aerojspark/gists{/gist_id}","starred_url":"https://api.github.com/users/Aerojspark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Aerojspark/subscriptions","organizations_url":"https://api.github.com/users/Aerojspark/orgs","repos_url":"https://api.github.com/users/Aerojspark/repos","events_url":"https://api.github.com/users/Aerojspark/events{/privacy}","received_events_url":"https://api.github.com/users/Aerojspark/received_events","type":"User","site_admin":false},"created_at":"2015-09-21T14:00:57Z","updated_at":"2015-09-21T14:00:57Z","body":"I have tested this pull request with my NACA0021 simulation. Previously I used nsteps=50, cutoff=1, order=16 and alpha=36.0 for P3 simulation and it worked well. However, the simulation was crashed with this pull request even when I reduced nsteps as 1. I can adjust other coefficient but it may be too diffusive. I am wondering that we may not remove x^3 or y^3 modal coefficient for P3 quad/hex even with cutoff=0 filter.\r\n\r\nI found that Gassner also used modal filter for DG quad/hexa mesh but he applied 1D filter matrix in a tensor product fashion. I think we may need to test this pull request a little bit more."}}