[cig-commits] [commit] devel, master: Update README_current_conclusions_from_Dimitri_about_the_bug.txt (1651cb3)

[cig_noreply at geodynamics.org]

Repository : https://github.com/geodynamics/specfem3d_globe

On branches: devel,master
Link       : https://github.com/geodynamics/specfem3d_globe/compare/bc58e579b3b0838a0968725a076f5904845437ca...be63f20cbb6f462104e949894dbe205d2398cd7f

>---------------------------------------------------------------

commit 1651cb3244eb484eaca79a46fea914ce4aa73c98
Author: Dimitri Komatitsch <komatits at users.noreply.github.com>
Date:   Thu Apr 10 13:50:04 2014 +0200

    Update README_current_conclusions_from_Dimitri_about_the_bug.txt


>---------------------------------------------------------------

1651cb3244eb484eaca79a46fea914ce4aa73c98
 .../README_current_conclusions_from_Dimitri_about_the_bug.txt           | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/IMPORTANT_BUG_benchmark_Ebru_Par_file/README_current_conclusions_from_Dimitri_about_the_bug.txt b/IMPORTANT_BUG_benchmark_Ebru_Par_file/README_current_conclusions_from_Dimitri_about_the_bug.txt
index 0eb45ad..0ba0743 100644
--- a/IMPORTANT_BUG_benchmark_Ebru_Par_file/README_current_conclusions_from_Dimitri_about_the_bug.txt
+++ b/IMPORTANT_BUG_benchmark_Ebru_Par_file/README_current_conclusions_from_Dimitri_about_the_bug.txt
@@ -7,7 +7,7 @@ runs are OK when attenuation is turned off in the Par_file; thus the bug is rela
 the problem (very likely) seems to be in the mesher because when not using mcmodel=medium to compile the mesher only and create the mesh, the results are then unchanged whether we compile the solver with mcmodel=medium or not; however, if we compile the mesher with mcmodel=medium instead of without, then the seismograms are (very) different
 
 attenuation with a 1D Earth model is OK (GJI 2002 attenuation benchmark is OK);
-note from Dimitri: confirmed also by this comment from last August "All 1D examples we have tested are fine (i.e. give identical seismograms with and without -mcmodel=medium -shared-intel), thus the bug is likely in the 3D routines."
+note from Dimitri: also confirmed by this comment from last August "All 1D examples we have tested are fine (i.e. give identical seismograms with and without -mcmodel=medium -shared-intel), thus the bug is likely in the 3D routines."
 
 thus the bug seems to be in the mesher, and come from attenuation in 3D models (or at least in the 3D model used by Ebru, which is s362ani); a good test could be to try another 3D model (e.g. s40rts) with attenuation, to see if the bug is still there. I think I remember from last August that the answer is yes (the bug is still there) but I am only 90% sure because it was several months ago; we should redo the test