gem / oq-engine

OpenQuake Engine: a software for Seismic Hazard and Risk Analysis
https://github.com/gem/oq-engine/#openquake-engine
GNU Affero General Public License v3.0
380 stars 276 forks source link

bindi_2011 missing DEFINED_FOR_REFERENCE_VELOCITY #6430

Closed hcrowley closed 1 year ago

hcrowley commented 3 years ago

The ground motion model "bindi_2011.py" does not have the term "DEFINED_FOR_REFERENCE_VELOCITY" and so it cannot be used with custom amplification functions. Can this definition be added?

rcgee commented 3 years ago

Hi Helen,

The DEFINED_FOR_REFERENCE_VELOCITY parameter was originally added only to GMPEs in the global mosaic, specifically those derived for rock conditions without built-in amplification functions/factors. This was needed to compute a global hazard map with uniform site conditions.

We haven’t yet worked on applying this parameter to the others GMPEs. We can certainly add this improvement to the list of things to do. Note that in many cases a reference velocity was not used in the development of the GMPEs, so I’m not sure it would always make sense to add it.

micheles commented 1 year ago

Closing since nobody protested for nearly 3 years.