aaspip / pyekfmm

A python package for 3D fast-marching-based traveltime calculation and its applications in seismology
GNU General Public License v3.0
51 stars 11 forks source link

Velocity model does not plot correctly. #6

Closed andresp-wave closed 6 months ago

andresp-wave commented 6 months ago

First, thanks for developing this great tool !

I believe that the plot3d function does not work for all cases. If I'm not wrong, it always assumes that the spatial sampling is 0.01. "L52 X, Y, Z = np.meshgrid(np.arange(nx)0.01, np.arange(ny)0.01, np.arange(nz)*0.01)" For example in "test_pyekfmm_raytrainc3d.py" when I change the spatial sampling to 0.1 the size of the velocity model should change to 30( in the depth axis) but it remains to the same value of 3 km.

chenyk1990 commented 6 months ago

Hi, Andres,

Please try again. It should be fixed. Let me know if you have any problems. The example is at https://github.com/aaspip/pyekfmm/blob/main/demos/test_pyekfmm_raytracing3d.py

Best, YC

On Mon, Apr 22, 2024 at 5:48 PM Andres Felipe Pena Castro < @.***> wrote:

First, thanks for developing this great tool !

I believe that the plot3d function does not work for all cases. If I'm not wrong, it always assumes that the spatial sampling is 0.01. "L52 X, Y, Z = np.meshgrid(np.arange(nx)0.01, np.arange(ny)0.01, np.arange(nz)*0.01)" For example in "test_pyekfmm_raytrainc3d.py" when I change the spatial sampling to 0.1 the size of the velocity model should change to 30( in the depth axis) but it remains to the same value of 3 km.

— Reply to this email directly, view it on GitHub https://github.com/aaspip/pyekfmm/issues/6, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADHCWEV2K5Z2D4JD4G2RWYDY6WHUXAVCNFSM6AAAAABGTURHNSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TONJYHAZTKMY . You are receiving this because you are subscribed to this thread.Message ID: @.***>

andresp-wave commented 6 months ago

Hi Yangkang,

I just tested and it works. Thanks ! I'll let you know if I encounter any other issues using pyekfmm.

Best, Andres