Small bug in earthquake scenario simulation with hazard-consistent downsampling

[migagf]

Hi,

I believe there's a small bug in the earthquake scenario simulation tool. Whenever a user picks a target period (Hazard Curve Sa Period) that corresponds to an integer (say 1 second), the hazard curve computation cannot run.
This is the callback:

IM_Calculator.get_im_exceedance_probility: error - intensity measure SA(1) does not match to ['PGA', 'SA(0.3)', 'SA(1.0)'].
The problem seems to be that an integer period that gets turned into a string without the decimal digit.
I was able to fix it on my local version, but my fix is not great (just added the '.0' whenever the period is an integer.

This is my code, starting from line 447 in HazardOcurrence.py.

if isinstance(period, int) or period.is_integer():
    im_name = f'SA({int(period)!s}.0)'
else:
    im_name = f'SA({period!s})'

Thanks!

Miguel

[fmckenna]

Miguel,

thanks. for simplicity i just added you changes to code base instead of changing the string compare.