SeisSol / PUMGen

Mesh generation for SeisSol
BSD 3-Clause "New" or "Revised" License
7 stars 3 forks source link

up to new api 16: Sim_readLicenseFile -> SimLicense_start #38

Closed yannrichet closed 2 years ago

krenzland commented 2 years ago

I haven't checked the content of this PR yet but please either 1) rebase or 2) merge with squashing strategy. There are a lot of redundant commits. (If you are not sure what this means, I can rebase/merge for you)

Thomas-Ulrich commented 2 years ago

Hi, Thank you for this PR! Can you give more background about this PR (I guess this is about getting pumgen working at IRSN for Hugo). I am asking because, here at LMU, we are still able to run pumgen/master with 16.0-220101, so why bother changing? Also, how should we use the new feature option? I tried using:

srun /hppfs/work/pr63qo/di73yeq4/myLibs/PUMGen/build3/pumgen -s simmodsuite -l SimModelerLib.lic --analyseAR --xml $xml --sim_log simlog_${prefixXml}.dat  $prefix.smd ${prefixXml} -f acis,parasolid,discrete,discretemodeling,surface,volume,parallelmeshing,attributes

But then I got:

Mon Jan 31 15:07:44, Info:  Using SimModSuite
Warning: [0] No license for product (-1): acis
Mon Jan 31 15:07:46, Info:  Loading model
Mon Jan 31 15:07:56, Info:  Extracting cases
Mon Jan 31 15:07:56, Info:  surface smoothing option: surfaceSmoothingLevel surfaceSmoothingType surfaceFaceRotationLimit Snap 1   1   10   0
Mon Jan 31 15:07:56, Info:  volume smoothing option: volumeSmoothingLevel volumeSmoothingType 1   1
Mon Jan 31 15:07:56, Info:  Setting cases
Mon Jan 31 15:07:56, Warn:  SimModeler: [0] Code: 200 String: No license for feature attributes

Thomas.

Thomas-Ulrich commented 2 years ago

the PR has been rebased in cleaned in #42.