Update N2Method.py - Githubissues

GEMScienceTools / rmtk

The Risk Modeller’s Toolkit prototype code.

GNU Affero General Public License v3.0

12 stars 14 forks source link

Update N2Method.py #108

Open ccasotto opened 8 years ago

ccasotto commented 8 years ago

1 - Bug in definition of parameter q (elastic acceleration demand at yielding Sa,e(Ty) over yielding strength Say) in N2 method q=Sa,e(Ty)/Say According to Fajfar [2000]

2- The modification for Sdi > 3*demand_Sd_Ty at line 57 and 58, this is not present in the original paper

Fajfar [2000] "a nonlinear analysis method for performance based seismic design", Earthquake Spectra, 16(3), http://30let.ikpir.com/data/bibliografije/att/1085537.fulltext.pdf

raoanirudh commented 8 years ago

LGTM :+1:

ccasotto commented 8 years ago

what about removing line 57 and 58?

raoanirudh commented 8 years ago

@VSilva, any particular cases where these two lines are required in the N2 method?

if Sdi > 3*demand_Sd_Ty:
    Sdi = demand_Sd_Ty