nikosT / Gisola

Gisola: A High Performance Computing application for real-time Moment Tensor inversion
GNU General Public License v3.0
44 stars 12 forks source link

ISOLA 1024 Points #54

Open fanisvl opened 1 month ago

fanisvl commented 1 month ago

ref. https://github.com/nikosT/Gisola/discussions/53

Hello,

I made the necessary changes to set NUM_OF_TIME_SAMPLES from 8192 points to 1024.

I did some testing and inversions are indeed faster:

8192.log
INFO Function 'calculateInversions' executed 278.77 seconds
1024.log
INFO Function 'calculateInversions' executed 31.76 seconds

8192.log

1024.log

Do note that there's still some comments that reference 8192 points, in src/config.yaml

  .... 
  # For example, if 4.0<= Mag <=4.5, Gisola will use 245.76 sec as
  # inversion time window. Values, that can be used must be able
  # to be divided with 8192 and lead to not repeating decimals.
  ....
  # For example, 
  # if 4.0<= Mag <=4.5, Gisola will use 245.76 sec as inversion time window, 
  # leading to a time kvantum of 246.76/8192=0.03 sec. ....
  ....
nikosT commented 1 month ago

Hi @fanisvl, thanks for this work! The PR will be tested before merging. Could you please also add the respective values in configuration here in a new commit at the same PR?

fanisvl commented 1 month ago

Hi @fanisvl, thanks for this work! The PR will be tested before merging. Could you please also add the respective values in configuration here in a new commit at the same PR?

change Window config & comments for 1024 points

Let me know if I'm missing anything.