search

ufz / ogs5

OpenGeoSys THM/C simulator version 5
https://www.opengeosys.org/ogs-5
Other
39 stars 97 forks source link

Upgrade numdiff and use gcc 6.2 #127

Closed wenqing closed 5 years ago

wenqing commented 5 years ago

This PR upgrades numdiff and uses gcc 6.2 on jenkins.

There are two benchmarks are marked as failed in the recent PRs

     85 - NB-M/pressure/Sphere_elastic/m_sphere_elastic (Failed)
    164 - YS-RWPT/2DGrains/2d_grains (Failed)

Benchmark 85 has a special BC of normal traction, and Benchmark 164 is about random walk.

On Jenkins, loading numdiff downgrades the gcc from 6.2 to 4.8. For the two failed benchmarks, the codes compiled with gcc 4.8 and 6.2, respectively, give results with a slight difference. The reference results were generated by using the code compiled with gcc 6.2, while Jenkin uses gcc 4.8 after running cli.sh. This makes the two benchmarks failed.

The following figures shows the displacement magnitude by the codes compiled with gcc 4.8, and 6.2, respectively: comp1

bilke commented 5 years ago

Awesome Wenqing, thanks for figuring this out! The build is marked as unstable because there are new compiler warnings because of the newer compiler. I will take of this next week. Thanks a lot!