search

SPECFEM / specfem2d

SPECFEM2D simulates forward and adjoint seismic wave propagation in two-dimensional acoustic, (an)elastic, poroelastic or coupled acoustic-(an)elastic-poroelastic media, with Convolution PML absorbing conditions.
GNU General Public License v3.0
204 stars 147 forks source link

specfem2d nightly build has failed #507

Closed buildbot-princeton closed 8 years ago

buildbot-princeton commented 8 years ago

See build 161.

luet commented 8 years ago

I think the problem is with

EXAMPLES/axisymmetric_case_AXISYM_option/run_this_example.sh

the line

NPROC=`grep nproc DATA/Par_file | cut -d = -f 2 | cut -d \# -f 1 | tr -d ' '`

does not return anything because lower case nproc is not in the Par_file. Replacing nproc by NPROC should fix the problem.

komatits commented 8 years ago

Thanks! Fixed. (changed grep to grep -i)

Let me cc Daniel because I think he fixed all the scripts yesterday thus let us doublecheck.

Best, Dimitri.

On 12/15/2015 03:13 PM, David Luet wrote:

I think the problem is with

|EXAMPLES/axisymmetric_case_AXISYM_option/run_this_example.sh |

the line

|NPROC=grep nproc DATA/Par_file | cut -d = -f 2 | cut -d \# -f 1 | tr -d ' ' |

does not return anything because lower case |nproc| is not in the |Par_file|. Replacing |nproc| by |NPROC| should fix the problem.

— Reply to this email directly or view it on GitHub https://github.com/geodynamics/specfem2d/issues/507#issuecomment-164776196.

Dimitri Komatitsch CNRS Research Director (DR CNRS), Laboratory of Mechanics and Acoustics, UPR 7051, Marseille, France http://komatitsch.free.fr

danielpeter commented 8 years ago

yes, the variable in the Par_file is now NPROC (uppercase letters), and identical with the 3d version. i changed the grep commands in the run scripts accordingly in PR #508

komatits commented 8 years ago

Thanks! the problem is the failed nightly build was probably elsewhere then, because in that case both grep NPROC and grep -i NPROC should have worked.

Dimitri.

On 12/15/2015 07:51 PM, daniel peter wrote:

yes, the variable in the Par_file is now NPROC (uppercase letters), and identical with the 3d version. i changed the grep commands in the run scripts accordingly in PR #508 https://github.com/geodynamics/specfem2d/pull/508

— Reply to this email directly or view it on GitHub https://github.com/geodynamics/specfem2d/issues/507#issuecomment-164853840.

Dimitri Komatitsch CNRS Research Director (DR CNRS), Laboratory of Mechanics and Acoustics, UPR 7051, Marseille, France http://komatitsch.free.fr

QuLogic commented 8 years ago

The nightly build failed before @danielpeter opened #508.