Skip to content

Stockholm#8

Open
qianglise wants to merge 5 commits intofrafrom
stockholm
Open

Stockholm#8
qianglise wants to merge 5 commits intofrafrom
stockholm

Conversation

@qianglise
Copy link
Contributor

first commit of the user case (based on OF 12) of wind comfort study of stockholm city from SLB analys

@qianglise qianglise mentioned this pull request Mar 3, 2026
@qianglise qianglise linked an issue Mar 3, 2026 that may be closed by this pull request


At this stage, one could modify various boundary/inflow conditions if necessary.
See section `Boundary conditions <https://qianglise.github.io/openfoamold/handson/#boundary-conditions>`__
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we keep this link or turn it into something on the enccs github?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it should be somehting on enccs github, I forgot to change the links.


Next step, run the solver with the default setup of the domain decomposition and turbulence modelling.
See sections `Parallelization <https://qianglise.github.io/openfoamold/handson/#parallelization>`__ and
`Turbulence modelling <https://qianglise.github.io/openfoamold/handson/#turbulence-modelling>`__
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it should be somehting on enccs github, I forgot to change the links.

Copy link
Contributor

@ffrancesco94 ffrancesco94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good for me apart from the fact that the files should be renamed (I guess OF>=11 has some fallback that it understands the old name too). If the case was tested also with OF>=11 we can merge for me after the files have been renamed!

- nutkRoughWallFunction


To speficy flow direction, one need to change the entry ``flowDir`` in the file *0/include/ABLConditions*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

specify

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be physicalProperties according to the new OF conventions, no?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This instead is momentumTransport, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create a nicer usecase

2 participants