Skip to content

Port to preCICE v2 and new OpenFOAM config#70

Merged
MakisH merged 27 commits into
masterfrom
develop
Feb 12, 2020
Merged

Port to preCICE v2 and new OpenFOAM config#70
MakisH merged 27 commits into
masterfrom
develop

Conversation

@BenjaminRodenberg
Copy link
Copy Markdown
Contributor

@BenjaminRodenberg BenjaminRodenberg commented Feb 11, 2020

For more details, see:
#71 (changes specific to preCICE v2)
#69 (changes specific to the new OpenFOAM config)

BenjaminRodenberg and others added 27 commits November 29, 2019 18:06
Recent versions of OpenFOAM require pcorrFinal to also be set in fvSolution.
Tested with OpenFOAM 5 (oldest supported) and OpenFOAM v1912 (latest).
If it is not present, deal.II complains. We also provide a (non-empty)
output directory in the cylinderFlap_2D tutorial.

[ci skip]
Port OpenFOAM tutorials to new OpenFOAM adapter config dictionary
Copy link
Copy Markdown
Member

@MakisH MakisH left a comment

Choose a reason for hiding this comment

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

This actually already integrates the new OpenFOAM adapter configuration format, so it should be two PRs. Other than that, it's ok.

@MakisH MakisH self-assigned this Feb 11, 2020
@MakisH MakisH changed the title Port to preCICE v2 Port to preCICE v2 and new OpenFOAM config Feb 12, 2020
@MakisH MakisH changed the title Port to preCICE v2 and new OpenFOAM config Port to new OpenFOAM config Feb 12, 2020
@MakisH MakisH changed the title Port to new OpenFOAM config Port to preCICE v2 and new OpenFOAM config Feb 12, 2020
Copy link
Copy Markdown
Member

@MakisH MakisH left a comment

Choose a reason for hiding this comment

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

I opened a separate, summary PR about changes related to preCICE v2 in #71, for documentation reasons.

I also created the tag https://github.com/precice/tutorials/tree/precice-v1.6 for anyone looking for the older tutorials.

After merging that, we can also merge this one.

@MakisH MakisH merged commit 32e1041 into master Feb 12, 2020
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.

4 participants