use freq_gradsmooth to guide setting time window duration

[wliuphd]

Updates to generate travel times properly when using the Dirac source time function.

[bjorn2]

Hi Wei,
Just one question. It looks as if you have added a frequency input parameter freq_gradsmooth, that is used to modify the source frequency in the eikonal solver. That is fine, I can see why that is needed, but why the name "gradsmooth" ? It does not seem to have anything to do with smoothing the gradient.
/Bjorn

[wliuphd]

Bjorn,

The parameter freq_gradsmooth was originally used to calculate spatial scales needed for smoothing gradients. It is also used for gauging time window durations here when source frequency can be subject to different scales and changes form different source time functions.

[andersp]

Hi Wei, I think Bjorn wants you to rename this variable to better indicate its current usage. Thanks, Anders From: Wei Liu ***@***.***> Date: Monday, November 8, 2021 at 8:55 AM To: geodynamics/sw4 ***@***.***> Cc: Petersson, Anders ***@***.***>, Assign ***@***.***> Subject: Re: [geodynamics/sw4] use freq_gradsmooth to guide setting time window duration (PR #102) Bjorn, The parameter freq_gradsmooth was originally used to calculate spatial scales needed for smoothing gradients. It is also used for gauging time window durations here when source frequency can be subject to different scales and changes form different source time functions. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub<https://urldefense.us/v3/__https:/github.com/geodynamics/sw4/pull/102*issuecomment-963361793__;Iw!!G2kpM7uM-TzIFchu!g6JoNf0I4Oc65WHscCfdK994dVpWdtrYNuX4O1W0Hf4AiZLiKminaQvDNC4EcKatYao$>, or unsubscribe<https://urldefense.us/v3/__https:/github.com/notifications/unsubscribe-auth/ACECFBZTBWJAB7OUZPEZXK3UK76IJANCNFSM5HEW5TOQ__;!!G2kpM7uM-TzIFchu!g6JoNf0I4Oc65WHscCfdK994dVpWdtrYNuX4O1W0Hf4AiZLiKminaQvDNC4El_PqcEU$>. Triage notifications on the go with GitHub Mobile for iOS<https://urldefense.us/v3/__https:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!G2kpM7uM-TzIFchu!g6JoNf0I4Oc65WHscCfdK994dVpWdtrYNuX4O1W0Hf4AiZLiKminaQvDNC4ExhU1hL0$> or Android<https://urldefense.us/v3/__https:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!G2kpM7uM-TzIFchu!g6JoNf0I4Oc65WHscCfdK994dVpWdtrYNuX4O1W0Hf4AiZLiKminaQvDNC4EHtRfUL8$>.

[wliuphd]

How about renaming freq_gradsmooth to freq_peakpower?

[andersp]

@wliuphd That name makes more sense to me.

[wliuphd]

@bjorn2 , I have updated PR with renaming the parameter. Can you merge it since I don't have write access.