GFDRR / thinkhazard_processing

Think Hazard: Overcome Risk - Processing module
0 stars 1 forks source link

unique threshold across units #27

Closed fvanderbiest closed 8 years ago

fvanderbiest commented 8 years ago

In thinkhazard_processing.yaml, thresholds are currently specific to a raster unit. They should not be IMO.

Instead, we should be able to internally convert from one unit to another (eg: 1m = 100 cm). This way, there will be a unique threshold value per (hazard_type, local/global).

pgiraud commented 8 years ago

Also related to https://github.com/GFDRR/thinkhazard/issues/374

fvanderbiest commented 8 years ago

Discussing this with @NicolasT-brgm ...

We agreed to say that:

SA stands for "spectral acceleration", see http://earthquake.usgs.gov/hazards/about/technical.php#sa

MMI stands for "Modified Mercalli Intensity" see https://en.wikipedia.org/wiki/Mercalli_intensity_scale

fvanderbiest commented 8 years ago

In addition, unit validation should take place during harvesting step, not during processing step. As a result, this code should also validate units.

fvanderbiest commented 8 years ago

To date, on GeoNode, datasets advertise these units:

Should the tool be able to handle other units ? Should the thresholds differ when the data unit changes (eg for EQ)? Or should we apply a "normalizing ratio" to convert from one unit to an other one, for which the threshold is set ?

fvanderbiest commented 8 years ago

Updated today:

fvanderbiest commented 8 years ago

@stufraser1 confirms that 1 PGA-g = 980.665 PGA-gal

100 PGA-g-percent = 1 PGA-g-decimal = 1 PGA-g

g and g-dec shouldn't both be used. I suggested amending the label 'g' to 'g-decimal' in Geonode only to distinguish it from g-percent

PGA can be defined as:

or

An earthquake can cause acceleration > 1 g, so decimal can for example exceed 1.5g, or 150 %g

fvanderbiest commented 8 years ago

Document where the thresholds for each hazard type can be found: https://docs.google.com/document/d/1fdbKi3OmqKIAslq7_N9tlo_JadsK6WF59Cwju71MnFM/edit#

fvanderbiest commented 8 years ago

OK, leaving this as a "nice to have" improvement for now, after discussion with @pgiraud