-
Notifications
You must be signed in to change notification settings - Fork 276
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tl16 #4352
Conversation
Does this need backport to 3.3? @micheles |
This is a question for @mmpagani |
I don’t think it’s necessary. Recently they run calculation from master.
…On 11 Jan 2019, at 8:20, Michele Simionato wrote:
This is a question for @mmpagani
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#4352 (comment)
|
No, they don't. They are running a "feature frozen" nightly build which is 3.3 in practice, but they must be brought back to stable ASAP (the effort to keep them on unstable is too high, and too risky). |
LGTM on my part, if @mmpagani is happy. Please update the changelog. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good to me. Thanks Robin. I'll check with our INGV colleagues what they are exactly planning to use.
While preparing the new Italian National Hazard model, INGV noticed that some GMPEs in tusa_langer_2016.py mistakenly used frequency values as periods in the coefficient tables. This bug was present in the the epicentral models, but not the hypocentral model.
The values have been corrected in the GMPE file and in the verification tables.