There are some mistakes at the example of Periodic hill on OpenFOAM: User Guide v1906

Summary

At the web site, Periodic hills describtion function is wrong.

Steps to reproduce

open the website [https://www.openfoam.com/documentation/guides/latest/doc/verification-validation-turbulent-periodic-hill.html#sec-verification-validation-turbulent-periodic-hill-overview] At the Mesh Table, you will see hills described by the function:

as the first equation could be large than 1, the function should be corrected as:

and the number in the table of c2 should be -1.016116352781×10−1 rather than 1.016116352781×10−1

Example case

What is the current bug behaviour?

What is the expected correct behavior?

Relevant logs and/or images

Environment information

• OpenFOAM version :
• Operating system :
• Hardware info :
• Compiler :

Possible fixes

changed the description

By Xu Zhaoyue on 2019-09-10T04:12:24 (imported from GitLab project)

changed the description

By Xu Zhaoyue on 2019-09-10T04:15:29 (imported from GitLab project)

@andy @kuti

Thanks for your report - we've made the change locally and will update the web version soon

By Andrew Heather on 2019-09-13T08:49:46 (imported from GitLab project)

Page updated on the website

By Andrew Heather on 2019-11-12T13:59:22 (imported from GitLab project)

closed

By Andrew Heather on 2019-11-12T13:59:23 (imported from GitLab project)

Hi @Prashant !

Currently, on master in https://develop.openfoam.com/Development/openfoam/-/blob/master/tutorials/incompressible/pimpleFoam/LES/periodicHill/transient/system/blockMeshDict

Line 62: if (x >= 0 && x < 9) { profile[i].y() = 28 + 6.775070969851E-03*x*x - 2.124527775800E-03*x*x*x; } So the minis still missing in the code, and the hill height locally exceeds 28! Or am I missing something .

Also, since a_1 is defined as 28 in the table in the documentation (i.e. not scaled with the hill height), shouldn't the min in the documentation be with respect to 28 as the base value, rather than 1?

Best, Timofey