{"action":"created","comment":{"url":"https://api.github.com/repos/geodynamics/aspect/pulls/comments/208062822","pull_request_review_id":143796662,"id":208062822,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIwODA2MjgyMg==","diff_hunk":"@@ -9937,6 +9937,29 @@ \\subsubsection{Benchmarks for operator splitting}\n \n The results shown here can be reproduced using the bash scripts \\texttt{run.sh} in the corresponding benchmark folders. \n \n+\\subsubsection{3D thermal convection benchmark}\n+ilabel{sec:benchmark-thermal-convection}\n+\n+\\textit{This section was contributed by L. Jeanniot and C. Thieulot.}\n+\n+We use an incompressible and isoviscous Rayleigh-Bénard thermal convection benchmark at infinite Prandtl number in a spherical shell to compare the results predicted by ASPECT with the previously benchmarked numerical code CitcomS. This benchmark follows the setup of Arrial et al. (2014) and Zhong et al. (2008). We explore the sensitivity of the initial and steady-state root mean square velocity $V_RMS$, average temperature $<T>$ and Nusselt number $Nu$ measured at both inner and outer boundaries to resolution for a Rayleigh number $Ra = 7000$ and $Ra = 70000$. \n+\n+The setup is a 3d spherical shell with a nondimensional length scale chosen as the approximate thickness of the mantle of the Earth (i.e. mantle thickness = 1). For low resolution models (here global refinement of 3), specifying Cells along circumferences at 12 allows better and quickier convergence to the solution. Angular momentum is here removed from the velocity field at the end of the Stokes solver step so that the net angular momentum is zero because for some geometries and certain boundary conditions the velocity field in not uniquely determined but contains free translations and/or rotations. \n+The dynamics of the fluid are entirely governed by the Rayleigh number $Ra$, which can be interpreted as a ratio of the destabilizing force due to the buoyancy of the heated fluid to the stabilizing force due to the the viscosity of the fluid and heat transfer by conduction. Therefore, we set here all the parameters which are parts of the Rayleigh formula to 1, except for the gravity component. The value given to the gravity magnitude therefore specifies the Rayleigh number value.","path":"doc/manual/manual.tex","position":12,"original_position":12,"commit_id":"05f9ce4d7f3032245047046b8a670ead3acb8e6d","original_commit_id":"05f9ce4d7f3032245047046b8a670ead3acb8e6d","user":{"login":"gassmoeller","id":7582930,"node_id":"MDQ6VXNlcjc1ODI5MzA=","avatar_url":"https://avatars0.githubusercontent.com/u/7582930?v=4","gravatar_id":"","url":"https://api.github.com/users/gassmoeller","html_url":"https://github.com/gassmoeller","followers_url":"https://api.github.com/users/gassmoeller/followers","following_url":"https://api.github.com/users/gassmoeller/following{/other_user}","gists_url":"https://api.github.com/users/gassmoeller/gists{/gist_id}","starred_url":"https://api.github.com/users/gassmoeller/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gassmoeller/subscriptions","organizations_url":"https://api.github.com/users/gassmoeller/orgs","repos_url":"https://api.github.com/users/gassmoeller/repos","events_url":"https://api.github.com/users/gassmoeller/events{/privacy}","received_events_url":"https://api.github.com/users/gassmoeller/received_events","type":"User","site_admin":false},"body":"`Therefore, we set here all the parameters which are parts of the Rayleigh formula to 1, except for the gravity component.` We do not usually call it `Rayleigh formula`. I would reformulate this to: `Therefore, we set all the parameters which are part of the Rayleigh number to 1, except for the gravity, which is then identical to the Rayleigh number.`","created_at":"2018-08-07T00:01:00Z","updated_at":"2018-08-07T00:05:28Z","html_url":"https://github.com/geodynamics/aspect/pull/2526#discussion_r208062822","pull_request_url":"https://api.github.com/repos/geodynamics/aspect/pulls/2526","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/geodynamics/aspect/pulls/comments/208062822"},"html":{"href":"https://github.com/geodynamics/aspect/pull/2526#discussion_r208062822"},"pull_request":{"href":"https://api.github.com/repos/geodynamics/aspect/pulls/2526"}}},"pull_request":{"url":"https://api.github.com/repos/geodynamics/aspect/pulls/2526","id":197888922,"node_id":"MDExOlB1bGxSZXF1ZXN0MTk3ODg4OTIy","html_url":"https://github.com/geodynamics/aspect/pull/2526","diff_url":"https://github.com/geodynamics/aspect/pull/2526.diff","patch_url":"https://github.com/geodynamics/aspect/pull/2526.patch","issue_url":"https://api.github.com/repos/geodynamics/aspect/issues/2526","number":2526,"state":"open","locked":false,"title":"[WIP] Benchmark 3D thermal convection in a spherical shell by Arrial et al. ","user":{"login":"LudovicJnnt","id":32137895,"node_id":"MDQ6VXNlcjMyMTM3ODk1","avatar_url":"https://avatars0.githubusercontent.com/u/32137895?v=4","gravatar_id":"","url":"https://api.github.com/users/LudovicJnnt","html_url":"https://github.com/LudovicJnnt","followers_url":"https://api.github.com/users/LudovicJnnt/followers","following_url":"https://api.github.com/users/LudovicJnnt/following{/other_user}","gists_url":"https://api.github.com/users/LudovicJnnt/gists{/gist_id}","starred_url":"https://api.github.com/users/LudovicJnnt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LudovicJnnt/subscriptions","organizations_url":"https://api.github.com/users/LudovicJnnt/orgs","repos_url":"https://api.github.com/users/LudovicJnnt/repos","events_url":"https://api.github.com/users/LudovicJnnt/events{/privacy}","received_events_url":"https://api.github.com/users/LudovicJnnt/received_events","type":"User","site_admin":false},"body":"Initial state of the thermal convection benchmark including parameter, source and include files ready to be reviewed. However, I have only  started writing a subsection in the manual as I need more test and figures with higher resolution to show. \r\n","created_at":"2018-06-28T00:39:05Z","updated_at":"2018-08-07T00:05:28Z","closed_at":null,"merged_at":null,"merge_commit_sha":"6c5fb56eb12a5c2f2709a312eff7a0d9bf311453","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/geodynamics/aspect/pulls/2526/commits","review_comments_url":"https://api.github.com/repos/geodynamics/aspect/pulls/2526/comments","review_comment_url":"https://api.github.com/repos/geodynamics/aspect/pulls/comments{/number}","comments_url":"https://api.github.com/repos/geodynamics/aspect/issues/2526/comments","statuses_url":"https://api.github.com/repos/geodynamics/aspect/statuses/05f9ce4d7f3032245047046b8a670ead3acb8e6d","head":{"label":"LudovicJnnt:thermal_convection_benchmark","ref":"thermal_convection_benchmark","sha":"05f9ce4d7f3032245047046b8a670ead3acb8e6d","user":{"login":"LudovicJnnt","id":32137895,"node_id":"MDQ6VXNlcjMyMTM3ODk1","avatar_url":"https://avatars0.githubusercontent.com/u/32137895?v=4","gravatar_id":"","url":"https://api.github.com/users/LudovicJnnt","html_url":"https://github.com/LudovicJnnt","followers_url":"https://api.github.com/users/LudovicJnnt/followers","following_url":"https://api.github.com/users/LudovicJnnt/following{/other_user}","gists_url":"https://api.github.com/users/LudovicJnnt/gists{/gist_id}","starred_url":"https://api.github.com/users/LudovicJnnt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LudovicJnnt/subscriptions","organizations_url":"https://api.github.com/users/LudovicJnnt/orgs","repos_url":"https://api.github.com/users/LudovicJnnt/repos","events_url":"https://api.github.com/users/LudovicJnnt/events{/privacy}","received_events_url":"https://api.github.com/users/LudovicJnnt/received_events","type":"User","site_admin":false},"repo":{"id":132438147,"node_id":"MDEwOlJlcG9zaXRvcnkxMzI0MzgxNDc=","name":"aspect","full_name":"LudovicJnnt/aspect","owner":{"login":"LudovicJnnt","id":32137895,"node_id":"MDQ6VXNlcjMyMTM3ODk1","avatar_url":"https://avatars0.githubusercontent.com/u/32137895?v=4","gravatar_id":"","url":"https://api.github.com/users/LudovicJnnt","html_url":"https://github.com/LudovicJnnt","followers_url":"https://api.github.com/users/LudovicJnnt/followers","following_url":"https://api.github.com/users/LudovicJnnt/following{/other_user}","gists_url":"https://api.github.com/users/LudovicJnnt/gists{/gist_id}","starred_url":"https://api.github.com/users/LudovicJnnt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LudovicJnnt/subscriptions","organizations_url":"https://api.github.com/users/LudovicJnnt/orgs","repos_url":"https://api.github.com/users/LudovicJnnt/repos","events_url":"https://api.github.com/users/LudovicJnnt/events{/privacy}","received_events_url":"https://api.github.com/users/LudovicJnnt/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/LudovicJnnt/aspect","description":"A parallel, extensible finite element code to simulate convection in both 2D and 3D models.","fork":true,"url":"https://api.github.com/repos/LudovicJnnt/aspect","forks_url":"https://api.github.com/repos/LudovicJnnt/aspect/forks","keys_url":"https://api.github.com/repos/LudovicJnnt/aspect/keys{/key_id}","collaborators_url":"https://api.github.com/repos/LudovicJnnt/aspect/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/LudovicJnnt/aspect/teams","hooks_url":"https://api.github.com/repos/LudovicJnnt/aspect/hooks","issue_events_url":"https://api.github.com/repos/LudovicJnnt/aspect/issues/events{/number}","events_url":"https://api.github.com/repos/LudovicJnnt/aspect/events","assignees_url":"https://api.github.com/repos/LudovicJnnt/aspect/assignees{/user}","branches_url":"https://api.github.com/repos/LudovicJnnt/aspect/branches{/branch}","tags_url":"https://api.github.com/repos/LudovicJnnt/aspect/tags","blobs_url":"https://api.github.com/repos/LudovicJnnt/aspect/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/LudovicJnnt/aspect/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/LudovicJnnt/aspect/git/refs{/sha}","trees_url":"https://api.github.com/repos/LudovicJnnt/aspect/git/trees{/sha}","statuses_url":"https://api.github.com/repos/LudovicJnnt/aspect/statuses/{sha}","languages_url":"https://api.github.com/repos/LudovicJnnt/aspect/languages","stargazers_url":"https://api.github.com/repos/LudovicJnnt/aspect/stargazers","contributors_url":"https://api.github.com/repos/LudovicJnnt/aspect/contributors","subscribers_url":"https://api.github.com/repos/LudovicJnnt/aspect/subscribers","subscription_url":"https://api.github.com/repos/LudovicJnnt/aspect/subscription","commits_url":"https://api.github.com/repos/LudovicJnnt/aspect/commits{/sha}","git_commits_url":"https://api.github.com/repos/LudovicJnnt/aspect/git/commits{/sha}","comments_url":"https://api.github.com/repos/LudovicJnnt/aspect/comments{/number}","issue_comment_url":"https://api.github.com/repos/LudovicJnnt/aspect/issues/comments{/number}","contents_url":"https://api.github.com/repos/LudovicJnnt/aspect/contents/{+path}","compare_url":"https://api.github.com/repos/LudovicJnnt/aspect/compare/{base}...{head}","merges_url":"https://api.github.com/repos/LudovicJnnt/aspect/merges","archive_url":"https://api.github.com/repos/LudovicJnnt/aspect/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/LudovicJnnt/aspect/downloads","issues_url":"https://api.github.com/repos/LudovicJnnt/aspect/issues{/number}","pulls_url":"https://api.github.com/repos/LudovicJnnt/aspect/pulls{/number}","milestones_url":"https://api.github.com/repos/LudovicJnnt/aspect/milestones{/number}","notifications_url":"https://api.github.com/repos/LudovicJnnt/aspect/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/LudovicJnnt/aspect/labels{/name}","releases_url":"https://api.github.com/repos/LudovicJnnt/aspect/releases{/id}","deployments_url":"https://api.github.com/repos/LudovicJnnt/aspect/deployments","created_at":"2018-05-07T09:29:16Z","updated_at":"2018-05-07T09:29:25Z","pushed_at":"2018-07-25T15:12:25Z","git_url":"git://github.com/LudovicJnnt/aspect.git","ssh_url":"git@github.com:LudovicJnnt/aspect.git","clone_url":"https://github.com/LudovicJnnt/aspect.git","svn_url":"https://github.com/LudovicJnnt/aspect","homepage":"https://aspect.geodynamics.org/","size":175924,"stargazers_count":0,"watchers_count":0,"language":"Gnuplot","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"other","name":"Other","spdx_id":null,"url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"geodynamics:master","ref":"master","sha":"8613ba4ef0684fffd331db29910a6d453ad84583","user":{"login":"geodynamics","id":4052461,"node_id":"MDEyOk9yZ2FuaXphdGlvbjQwNTI0NjE=","avatar_url":"https://avatars1.githubusercontent.com/u/4052461?v=4","gravatar_id":"","url":"https://api.github.com/users/geodynamics","html_url":"https://github.com/geodynamics","followers_url":"https://api.github.com/users/geodynamics/followers","following_url":"https://api.github.com/users/geodynamics/following{/other_user}","gists_url":"https://api.github.com/users/geodynamics/gists{/gist_id}","starred_url":"https://api.github.com/users/geodynamics/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geodynamics/subscriptions","organizations_url":"https://api.github.com/users/geodynamics/orgs","repos_url":"https://api.github.com/users/geodynamics/repos","events_url":"https://api.github.com/users/geodynamics/events{/privacy}","received_events_url":"https://api.github.com/users/geodynamics/received_events","type":"Organization","site_admin":false},"repo":{"id":19502452,"node_id":"MDEwOlJlcG9zaXRvcnkxOTUwMjQ1Mg==","name":"aspect","full_name":"geodynamics/aspect","owner":{"login":"geodynamics","id":4052461,"node_id":"MDEyOk9yZ2FuaXphdGlvbjQwNTI0NjE=","avatar_url":"https://avatars1.githubusercontent.com/u/4052461?v=4","gravatar_id":"","url":"https://api.github.com/users/geodynamics","html_url":"https://github.com/geodynamics","followers_url":"https://api.github.com/users/geodynamics/followers","following_url":"https://api.github.com/users/geodynamics/following{/other_user}","gists_url":"https://api.github.com/users/geodynamics/gists{/gist_id}","starred_url":"https://api.github.com/users/geodynamics/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geodynamics/subscriptions","organizations_url":"https://api.github.com/users/geodynamics/orgs","repos_url":"https://api.github.com/users/geodynamics/repos","events_url":"https://api.github.com/users/geodynamics/events{/privacy}","received_events_url":"https://api.github.com/users/geodynamics/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/geodynamics/aspect","description":"A parallel, extensible finite element code to simulate convection in both 2D and 3D models.","fork":false,"url":"https://api.github.com/repos/geodynamics/aspect","forks_url":"https://api.github.com/repos/geodynamics/aspect/forks","keys_url":"https://api.github.com/repos/geodynamics/aspect/keys{/key_id}","collaborators_url":"https://api.github.com/repos/geodynamics/aspect/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/geodynamics/aspect/teams","hooks_url":"https://api.github.com/repos/geodynamics/aspect/hooks","issue_events_url":"https://api.github.com/repos/geodynamics/aspect/issues/events{/number}","events_url":"https://api.github.com/repos/geodynamics/aspect/events","assignees_url":"https://api.github.com/repos/geodynamics/aspect/assignees{/user}","branches_url":"https://api.github.com/repos/geodynamics/aspect/branches{/branch}","tags_url":"https://api.github.com/repos/geodynamics/aspect/tags","blobs_url":"https://api.github.com/repos/geodynamics/aspect/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/geodynamics/aspect/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/geodynamics/aspect/git/refs{/sha}","trees_url":"https://api.github.com/repos/geodynamics/aspect/git/trees{/sha}","statuses_url":"https://api.github.com/repos/geodynamics/aspect/statuses/{sha}","languages_url":"https://api.github.com/repos/geodynamics/aspect/languages","stargazers_url":"https://api.github.com/repos/geodynamics/aspect/stargazers","contributors_url":"https://api.github.com/repos/geodynamics/aspect/contributors","subscribers_url":"https://api.github.com/repos/geodynamics/aspect/subscribers","subscription_url":"https://api.github.com/repos/geodynamics/aspect/subscription","commits_url":"https://api.github.com/repos/geodynamics/aspect/commits{/sha}","git_commits_url":"https://api.github.com/repos/geodynamics/aspect/git/commits{/sha}","comments_url":"https://api.github.com/repos/geodynamics/aspect/comments{/number}","issue_comment_url":"https://api.github.com/repos/geodynamics/aspect/issues/comments{/number}","contents_url":"https://api.github.com/repos/geodynamics/aspect/contents/{+path}","compare_url":"https://api.github.com/repos/geodynamics/aspect/compare/{base}...{head}","merges_url":"https://api.github.com/repos/geodynamics/aspect/merges","archive_url":"https://api.github.com/repos/geodynamics/aspect/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/geodynamics/aspect/downloads","issues_url":"https://api.github.com/repos/geodynamics/aspect/issues{/number}","pulls_url":"https://api.github.com/repos/geodynamics/aspect/pulls{/number}","milestones_url":"https://api.github.com/repos/geodynamics/aspect/milestones{/number}","notifications_url":"https://api.github.com/repos/geodynamics/aspect/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/geodynamics/aspect/labels{/name}","releases_url":"https://api.github.com/repos/geodynamics/aspect/releases{/id}","deployments_url":"https://api.github.com/repos/geodynamics/aspect/deployments","created_at":"2014-05-06T16:47:18Z","updated_at":"2018-08-06T23:41:13Z","pushed_at":"2018-08-06T23:41:03Z","git_url":"git://github.com/geodynamics/aspect.git","ssh_url":"git@github.com:geodynamics/aspect.git","clone_url":"https://github.com/geodynamics/aspect.git","svn_url":"https://github.com/geodynamics/aspect","homepage":"https://aspect.geodynamics.org/","size":178208,"stargazers_count":55,"watchers_count":55,"language":"Gnuplot","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":101,"mirror_url":null,"archived":false,"open_issues_count":210,"license":{"key":"other","name":"Other","spdx_id":null,"url":null,"node_id":"MDc6TGljZW5zZTA="},"forks":101,"open_issues":210,"watchers":55,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/geodynamics/aspect/pulls/2526"},"html":{"href":"https://github.com/geodynamics/aspect/pull/2526"},"issue":{"href":"https://api.github.com/repos/geodynamics/aspect/issues/2526"},"comments":{"href":"https://api.github.com/repos/geodynamics/aspect/issues/2526/comments"},"review_comments":{"href":"https://api.github.com/repos/geodynamics/aspect/pulls/2526/comments"},"review_comment":{"href":"https://api.github.com/repos/geodynamics/aspect/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/geodynamics/aspect/pulls/2526/commits"},"statuses":{"href":"https://api.github.com/repos/geodynamics/aspect/statuses/05f9ce4d7f3032245047046b8a670ead3acb8e6d"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"gassmoeller"}}