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.

Sign up for GitHub

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

Jump to bottom

Misc for OgsTester #2257

Merged
merged 7 commits into from
Nov 6, 2018
Merged

Misc for OgsTester #2257

merged 7 commits into from
Nov 6, 2018

Conversation

endJunction
Copy link
Member

Extracted from OgsTester PR.

Few changes found during the OgsTester implementation. The extracted function in the VtuInterface is reused in OgsTester PR.

chleh
chleh reviewed Nov 6, 2018
View reviewed changes
Copy link
Collaborator

@chleh chleh left a comment

Choose a reason for hiding this comment

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

👍 after modified tests actually run.

@endJunction
Copy link
Member Author

endJunction commented Nov 6, 2018 •
edited
Loading

Hmmm.... Something is wrong there. Thinking aloud....
Of the three modified test the axisymmetric_pipe.prj is executed normally with Eigen, and it does not matter if the solid_density is there or not. But the hollow_sphere.prj (which is enabled for LIS only) does not run w/o solid_density.

Update: Got it now. There is another axisymmetric_pipe.prj

@endJunction
[T] Test postLIE only if LIE processes are build.
23462d4
@endJunction
[App] ogs; Capitalize CLI arguments' types.
4a71754
@endJunction
[MeL/IO] Extract vtu file name convers. for PETSc.
f39b338
@endJunction
[T] M/PressureBC; Add missing solid_density and g.
0b76d82
@endJunction
[T] M/SD; Cleanup duplicate axisymmetric_pipe.
5fa20e2 
Same test is in PressureBC/ directory; Moved reference result
and removed the duplicate prj/gml/vtu.
@endJunction
[T] M/SD; Cleanup almost duplicate axisymm. sphere
5cc9614 
Almost the same test in the PressureBC/ with different forces applied.
Move everything to PressureBC.
Vectorize sigma in reference result.
@endJunction
[T] SD/hollow sphere. Changes solver tols.
517c317 
Vectorize sigma in reference file.
Enable for Eigen builds too, update tolerance for sigma.
@endJunction
Copy link
Member Author

hollow_sphere test runs successfully with Eigen using CG/Diagonal linear solver.
Now all of the changed tests are Eigen enabled and do run (on my machine at least ;).

@bilke singularity1 hit a singularity ;(

@endJunction endJunction merged commit c542e53 into ufz:master Nov 6, 2018
@endJunction endJunction deleted the MiscForOgsTester branch November 6, 2018 19:57
@ogsbot
Copy link
Member

ogsbot commented Jun 19, 2020

OpenGeoSys development has been moved to GitLab.

See this pull request on GitLab.

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

@chleh chleh chleh left review comments

@bilke bilke bilke approved these changes

@wenqing wenqing wenqing approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@endJunction @ogsbot @bilke @wenqing @chleh