Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

simplified marin test for rans3p #168

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

simplified marin test for rans3p #168

wants to merge 23 commits into from

Conversation

cekees
Copy link
Member

@cekees cekees commented Feb 9, 2018

I added the test in the 3d/marin3p folder. It depends on erdc/proteus#683 and is in need of several improvements:

  • add Context options to marin.py (e.g. he, genMesh, openTop,...)
  • add 4 pressure gauges and 4 line gauges to extract data and replace the included pvpython extractMarinHeight.py and extractMarinPressure.py. Since those scripts pull from the xdmf, the extracted data are dependent on the user's selection of output times, and the pararview scripts don't seem to work as-is for rans3p's mixed FEM spaces anyway.
  • check and enable the openTop option--right now this is running with closed boundaries
  • check that things are OK in parallel and do a grid refinement study comparing to the experimental data, which is included in the *.dat files.

/cc @arnsong @manuel-quezada @wacyyang @westb @Giovanni-Cozzuto-1989 @Pedrohrw

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