{"action":"created","comment":{"url":"https://api.github.com/repos/vincentlab/PyFR/pulls/comments/29614715","id":29614715,"diff_hunk":"@@ -44,6 +57,41 @@ def __init__(self, *args, **kwargs):\n         tplargs = dict(ndims=self.ndims, nvars=self.nvars, rsolver=rsolver,\n                        visc_corr=visc_corr, c=self._tpl_c)\n \n+        # Generate the additional kernels, view matrices for artificial viscosity\n+        self.shock_capturing = self.cfg.get('solver', 'shock-capturing', 'none')\n+        if self.shock_capturing == 'artificial-viscosity':\n+            be, lhs, rhs = args[0], args[1], args[2]","path":"pyfr/solvers/navstokes/inters.py","position":41,"original_position":41,"commit_id":"fd0deef66c9fd92fd44bf0f3ea5bb82955d52de5","original_commit_id":"fd0deef66c9fd92fd44bf0f3ea5bb82955d52de5","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},"body":"Do not use `*args` or `**kwargs`.  This will break if we ever choose to pass some arguments by key.","created_at":"2015-05-04T19:28:10Z","updated_at":"2015-05-04T19:28:10Z","html_url":"https://github.com/vincentlab/PyFR/pull/46#discussion_r29614715","pull_request_url":"https://api.github.com/repos/vincentlab/PyFR/pulls/46","_links":{"self":{"href":"https://api.github.com/repos/vincentlab/PyFR/pulls/comments/29614715"},"html":{"href":"https://github.com/vincentlab/PyFR/pull/46#discussion_r29614715"},"pull_request":{"href":"https://api.github.com/repos/vincentlab/PyFR/pulls/46"}}},"pull_request":{"url":"https://api.github.com/repos/vincentlab/PyFR/pulls/46","id":33979770,"html_url":"https://github.com/vincentlab/PyFR/pull/46","diff_url":"https://github.com/vincentlab/PyFR/pull/46.diff","patch_url":"https://github.com/vincentlab/PyFR/pull/46.patch","issue_url":"https://api.github.com/repos/vincentlab/PyFR/issues/46","number":46,"state":"open","locked":false,"title":"Added Persson's artificial viscosity (AIAA 2006-0112).","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},"body":"I have added Persson's artificial viscosity scheme (AIAA 2006-0112). It requires three tuning parameters, which is defined in [solver-avis] section.\r\n\r\namu0 : maximum amount of artificial viscosity ~ h/p\r\ns0 : cut-off of the smooth indicator ~ 1/p^4\r\nkappa : range of the smooth indicator ~ empirically large\r\n\r\nTo get a proper values of them, you need to some (or many) trial-and-errors. \r\n\r\nThis scheme is only working for Navier-Stokes system. If you want to simulate inviscid shock wave  flow, you should impose viscosity (mu in [solver] section) as 0.0.","created_at":"2015-04-23T17:51:36Z","updated_at":"2015-05-04T19:28:10Z","closed_at":null,"merged_at":null,"merge_commit_sha":"3a6a9c11923e5dc46a698360ccab2d51a1c204cb","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/vincentlab/PyFR/pulls/46/commits","review_comments_url":"https://api.github.com/repos/vincentlab/PyFR/pulls/46/comments","review_comment_url":"https://api.github.com/repos/vincentlab/PyFR/pulls/comments{/number}","comments_url":"https://api.github.com/repos/vincentlab/PyFR/issues/46/comments","statuses_url":"https://api.github.com/repos/vincentlab/PyFR/statuses/fd0deef66c9fd92fd44bf0f3ea5bb82955d52de5","head":{"label":"Aerojspark:develop","ref":"develop","sha":"fd0deef66c9fd92fd44bf0f3ea5bb82955d52de5","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},"repo":{"id":24884060,"name":"PyFR","full_name":"Aerojspark/PyFR","owner":{"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},"private":false,"html_url":"https://github.com/Aerojspark/PyFR","description":"PyFR","fork":true,"url":"https://api.github.com/repos/Aerojspark/PyFR","forks_url":"https://api.github.com/repos/Aerojspark/PyFR/forks","keys_url":"https://api.github.com/repos/Aerojspark/PyFR/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Aerojspark/PyFR/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Aerojspark/PyFR/teams","hooks_url":"https://api.github.com/repos/Aerojspark/PyFR/hooks","issue_events_url":"https://api.github.com/repos/Aerojspark/PyFR/issues/events{/number}","events_url":"https://api.github.com/repos/Aerojspark/PyFR/events","assignees_url":"https://api.github.com/repos/Aerojspark/PyFR/assignees{/user}","branches_url":"https://api.github.com/repos/Aerojspark/PyFR/branches{/branch}","tags_url":"https://api.github.com/repos/Aerojspark/PyFR/tags","blobs_url":"https://api.github.com/repos/Aerojspark/PyFR/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Aerojspark/PyFR/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Aerojspark/PyFR/git/refs{/sha}","trees_url":"https://api.github.com/repos/Aerojspark/PyFR/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Aerojspark/PyFR/statuses/{sha}","languages_url":"https://api.github.com/repos/Aerojspark/PyFR/languages","stargazers_url":"https://api.github.com/repos/Aerojspark/PyFR/stargazers","contributors_url":"https://api.github.com/repos/Aerojspark/PyFR/contributors","subscribers_url":"https://api.github.com/repos/Aerojspark/PyFR/subscribers","subscription_url":"https://api.github.com/repos/Aerojspark/PyFR/subscription","commits_url":"https://api.github.com/repos/Aerojspark/PyFR/commits{/sha}","git_commits_url":"https://api.github.com/repos/Aerojspark/PyFR/git/commits{/sha}","comments_url":"https://api.github.com/repos/Aerojspark/PyFR/comments{/number}","issue_comment_url":"https://api.github.com/repos/Aerojspark/PyFR/issues/comments{/number}","contents_url":"https://api.github.com/repos/Aerojspark/PyFR/contents/{+path}","compare_url":"https://api.github.com/repos/Aerojspark/PyFR/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Aerojspark/PyFR/merges","archive_url":"https://api.github.com/repos/Aerojspark/PyFR/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Aerojspark/PyFR/downloads","issues_url":"https://api.github.com/repos/Aerojspark/PyFR/issues{/number}","pulls_url":"https://api.github.com/repos/Aerojspark/PyFR/pulls{/number}","milestones_url":"https://api.github.com/repos/Aerojspark/PyFR/milestones{/number}","notifications_url":"https://api.github.com/repos/Aerojspark/PyFR/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Aerojspark/PyFR/labels{/name}","releases_url":"https://api.github.com/repos/Aerojspark/PyFR/releases{/id}","created_at":"2014-10-07T09:22:28Z","updated_at":"2015-05-03T17:54:36Z","pushed_at":"2015-05-03T17:54:36Z","git_url":"git://github.com/Aerojspark/PyFR.git","ssh_url":"git@github.com:Aerojspark/PyFR.git","clone_url":"https://github.com/Aerojspark/PyFR.git","svn_url":"https://github.com/Aerojspark/PyFR","homepage":"","size":1890,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"develop"}},"base":{"label":"vincentlab:develop","ref":"develop","sha":"109e0773506bcc7c75cc1478b6da8b59d9632c25","user":{"login":"vincentlab","id":3252468,"avatar_url":"https://avatars.githubusercontent.com/u/3252468?v=3","gravatar_id":"","url":"https://api.github.com/users/vincentlab","html_url":"https://github.com/vincentlab","followers_url":"https://api.github.com/users/vincentlab/followers","following_url":"https://api.github.com/users/vincentlab/following{/other_user}","gists_url":"https://api.github.com/users/vincentlab/gists{/gist_id}","starred_url":"https://api.github.com/users/vincentlab/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vincentlab/subscriptions","organizations_url":"https://api.github.com/users/vincentlab/orgs","repos_url":"https://api.github.com/users/vincentlab/repos","events_url":"https://api.github.com/users/vincentlab/events{/privacy}","received_events_url":"https://api.github.com/users/vincentlab/received_events","type":"Organization","site_admin":false},"repo":{"id":7652671,"name":"PyFR","full_name":"vincentlab/PyFR","owner":{"login":"vincentlab","id":3252468,"avatar_url":"https://avatars.githubusercontent.com/u/3252468?v=3","gravatar_id":"","url":"https://api.github.com/users/vincentlab","html_url":"https://github.com/vincentlab","followers_url":"https://api.github.com/users/vincentlab/followers","following_url":"https://api.github.com/users/vincentlab/following{/other_user}","gists_url":"https://api.github.com/users/vincentlab/gists{/gist_id}","starred_url":"https://api.github.com/users/vincentlab/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vincentlab/subscriptions","organizations_url":"https://api.github.com/users/vincentlab/orgs","repos_url":"https://api.github.com/users/vincentlab/repos","events_url":"https://api.github.com/users/vincentlab/events{/privacy}","received_events_url":"https://api.github.com/users/vincentlab/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/vincentlab/PyFR","description":"PyFR","fork":false,"url":"https://api.github.com/repos/vincentlab/PyFR","forks_url":"https://api.github.com/repos/vincentlab/PyFR/forks","keys_url":"https://api.github.com/repos/vincentlab/PyFR/keys{/key_id}","collaborators_url":"https://api.github.com/repos/vincentlab/PyFR/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/vincentlab/PyFR/teams","hooks_url":"https://api.github.com/repos/vincentlab/PyFR/hooks","issue_events_url":"https://api.github.com/repos/vincentlab/PyFR/issues/events{/number}","events_url":"https://api.github.com/repos/vincentlab/PyFR/events","assignees_url":"https://api.github.com/repos/vincentlab/PyFR/assignees{/user}","branches_url":"https://api.github.com/repos/vincentlab/PyFR/branches{/branch}","tags_url":"https://api.github.com/repos/vincentlab/PyFR/tags","blobs_url":"https://api.github.com/repos/vincentlab/PyFR/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/vincentlab/PyFR/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/vincentlab/PyFR/git/refs{/sha}","trees_url":"https://api.github.com/repos/vincentlab/PyFR/git/trees{/sha}","statuses_url":"https://api.github.com/repos/vincentlab/PyFR/statuses/{sha}","languages_url":"https://api.github.com/repos/vincentlab/PyFR/languages","stargazers_url":"https://api.github.com/repos/vincentlab/PyFR/stargazers","contributors_url":"https://api.github.com/repos/vincentlab/PyFR/contributors","subscribers_url":"https://api.github.com/repos/vincentlab/PyFR/subscribers","subscription_url":"https://api.github.com/repos/vincentlab/PyFR/subscription","commits_url":"https://api.github.com/repos/vincentlab/PyFR/commits{/sha}","git_commits_url":"https://api.github.com/repos/vincentlab/PyFR/git/commits{/sha}","comments_url":"https://api.github.com/repos/vincentlab/PyFR/comments{/number}","issue_comment_url":"https://api.github.com/repos/vincentlab/PyFR/issues/comments{/number}","contents_url":"https://api.github.com/repos/vincentlab/PyFR/contents/{+path}","compare_url":"https://api.github.com/repos/vincentlab/PyFR/compare/{base}...{head}","merges_url":"https://api.github.com/repos/vincentlab/PyFR/merges","archive_url":"https://api.github.com/repos/vincentlab/PyFR/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/vincentlab/PyFR/downloads","issues_url":"https://api.github.com/repos/vincentlab/PyFR/issues{/number}","pulls_url":"https://api.github.com/repos/vincentlab/PyFR/pulls{/number}","milestones_url":"https://api.github.com/repos/vincentlab/PyFR/milestones{/number}","notifications_url":"https://api.github.com/repos/vincentlab/PyFR/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/vincentlab/PyFR/labels{/name}","releases_url":"https://api.github.com/repos/vincentlab/PyFR/releases{/id}","created_at":"2013-01-16T19:53:24Z","updated_at":"2015-05-04T19:12:43Z","pushed_at":"2015-05-04T19:12:43Z","git_url":"git://github.com/vincentlab/PyFR.git","ssh_url":"git@github.com:vincentlab/PyFR.git","clone_url":"https://github.com/vincentlab/PyFR.git","svn_url":"https://github.com/vincentlab/PyFR","homepage":"","size":4068,"stargazers_count":18,"watchers_count":18,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":24,"mirror_url":null,"open_issues_count":4,"forks":24,"open_issues":4,"watchers":18,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/vincentlab/PyFR/pulls/46"},"html":{"href":"https://github.com/vincentlab/PyFR/pull/46"},"issue":{"href":"https://api.github.com/repos/vincentlab/PyFR/issues/46"},"comments":{"href":"https://api.github.com/repos/vincentlab/PyFR/issues/46/comments"},"review_comments":{"href":"https://api.github.com/repos/vincentlab/PyFR/pulls/46/comments"},"review_comment":{"href":"https://api.github.com/repos/vincentlab/PyFR/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/vincentlab/PyFR/pulls/46/commits"},"statuses":{"href":"https://api.github.com/repos/vincentlab/PyFR/statuses/fd0deef66c9fd92fd44bf0f3ea5bb82955d52de5"}}}}