search

SPECFEM / specfem3d

SPECFEM3D_Cartesian simulates acoustic (fluid), elastic (solid), coupled acoustic/elastic, poroelastic or seismic wave propagation in any type of conforming mesh of hexahedra (structured or not).
GNU General Public License v3.0
403 stars 225 forks source link

VTK version #1473

Open kenichi-tsuda opened 2 years ago

kenichi-tsuda commented 2 years ago

Dear People

My name is Kenichi Tsuda using SEPCFEM3D for several years. I have a question about the version of the code to support VTK.

Actually, I was using VTK-5.10.1 for several years to generate movie data. When I used it on May, it worked. And I have tried it today, it did not work because of the version of VTK. For the later version of VTK-6, the compilation of mesh2vtu did not work. On the other hand, the compilation of mesh2vtu for VTK-5 did not work on Redhat7

I'm very appreciated to have any advice and suggestions for this issue. (If you need any more clarifications, please let me know)

Regards

Kenichi Tsuda (kenichi.tsuda@shimz.co.jp)

danielpeter commented 2 years ago

hi Kenichi-san,

could you provide the error output? and is it a compilation or linking problem? it's likely a VTK installation issue. it works when i test here with VTK-6.2 on ubuntu.

kenichi-tsuda commented 2 years ago

Dear Daniel san

Thanks for your e-mail. What I've got when J have compiled. I assume the problem looks coming from the vtk library version installed on our machine.

@.*** mesh2vtu]$ make g++ -O3 -I/opt/seismo-util/include -I/opt/seismo-util/include/vtk -Wno-deprecated -c mesh2vtu.cxx -o mesh2vtu.o mesh2vtu.cxx:39:31: Critical Error: vtk/vtkFloatArray.h: No fiilr or directory

include <vtk/vtkFloatArray.h>

                                                   ^

I'm wondering even I could install the package of vtk library for our OS (Redhat 7). I'm very appreciated to have any tips to solve for this problem.

Regards

Kenichi Tsuda

差出人: daniel peter @.> 送信日時: 2021年11月24日 23:19 宛先: geodynamics/specfem3d @.> CC: 津田　健一 @.>; Author @.> 件名: Re: [geodynamics/specfem3d] VTK version (Issue #1473)

hi Kenichi-san,

could you provide the error output? and is it a compilation or linking problem? it's likely a VTK installation issue. it works when i test here with VTK-6.2 on ubuntu.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/geodynamics/specfem3d/issues/1473#issuecomment-977921583, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACAVTPV36LG7CE4FFL3MWOTUNTX7DANCNFSM5IVXZJ7A. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

danielpeter commented 2 years ago

it looks like you're using a very old version of the mesh2vtu.cxx file which still uses 

#include <vtk/vtkFloatArray.h>
..

please try to update to the most recent devel version where you will find the updated mesh2vtu files in directory:

SPECFEM3D/utils/Visualization/Paraview/mesh2vtu

the Makefile has been amended as well, so instead of looking in the /opt/seismo-util/ directory, it first tries to find the current VTK libraries on your system by making use of a cmake-statement cmake --find-package -DNAME:STRING=VTK .. hopefully this will solve the installation problem.

also note that you can directly convert the binary mesh files to VTK-files without the need of a VTK library installation by compiling and using the xcombine_vol_data_vtk binary:

make xcombine_vol_data_vtk

i'll see if i can make a commit soon to add a VTU binary file format to reduced file sizes a bit in a similar way.