oq-engine icon indicating copy to clipboard operation
oq-engine copied to clipboard

Adding new CAV and AI models

Open emthompson-usgs opened this issue 11 months ago • 3 comments

Campbell and Bozorgnia (2019) give new models for CAV and IA. I noticed that the previous model for CAV (Cambel and Bozorgnia, 2010) is included in the CampbellBozorgnia2008 class as separate coefficients. We are interested in contributing implementations of the new 2019 models for CAV and IA, but unsure how to structure it. Should we make a new module with separate classes for CAV and IA? This would deviate from how the 2010 model is implemented. Any guidance on this would be great. Thanks.

emthompson-usgs avatar Mar 19 '24 15:03 emthompson-usgs

Dear Eric, many thanks for your message. If I am not mistaken Campbell and Bozorgnia 2019 use the same functional form of CB14 and add coefficients for these two additional IMTs. If this is correct, I would follow the same approach used for CB08. It's not the cleanest but it facilitates the use of this GMM for computing various metrics simultaneously.

mmpagani avatar Mar 22 '24 07:03 mmpagani

Sounds good. That's what we will do.

emthompson-usgs avatar Mar 22 '24 14:03 emthompson-usgs

Hi @mmpagani,

In doing this, we realized that the horizontal component for the 2019 metrics is different than those in 2014. So, I think what we should do is add the IA and CAV coefficients to the CampbellBozorgnia2014 table, and then create a CampbellBozorgnia2019 subclass that changes the following class attributes:

DEFINED_FOR_INTENSITY_MEASURE_COMPONENT=const.IMC.GEOMETRIC_MEAN
DEFINED_FOR_INTENSITY_MEASURE_TYPES={CAV, IA}

Let us know if this makes sense to you. Thanks.

emthompson-usgs avatar Mar 28 '24 15:03 emthompson-usgs

PR #9535 was merged, so I think this issue can be closed.

micheles avatar Sep 24 '24 06:09 micheles

Unless I'm missing something, the DEFINED_FOR_INTENSITY_MEASURE_COMPONENT attribute is set incorrectly in the merge.

emthompson-usgs avatar Sep 24 '24 20:09 emthompson-usgs

The attribute is now set correctly, lets try to close this again.

micheles avatar Sep 27 '24 04:09 micheles