amaelkady / Updated-IMK-Models-for-OpenSEES

Documentation for the updated modified Ibarra-Medina-Krawinkler deterioration models (Bilinear, Peak-Oriented, Pinching). These models supersede those in OpenSEES where convergence problems are fixed.
GNU General Public License v3.0
4 stars 4 forks source link

IMKBilin OpenSees.exe tcl86t.dll error #1

Open jwberman opened 3 years ago

jwberman commented 3 years ago

Hi Ahmad, I took this off the main OpenSees thread as it probably just a Jeff issue. When I simply swap out the OpenSees.exe (V3.3.0) from the OpenSees website with your exe that has the updated IMK models I get a windows error indicating that the code execution cannot proceed because the file tcl86t.dll was not found. Do I need to do a different tcl install for you updated exe? Sorry for all the problems here and thanks a ton for your help. I'm planning to use these models in a class and need to get this sorted out so I can provide some clear instructions.

Jeff

amaelkady commented 3 years ago

There no bother at all; it is always good to catch bugs in the code. If you go back to my repo, you will find the tcl86t.dll (i just uploaded it). you only need to place it with the opensees exe.

jwberman commented 3 years ago

OK, thanks! The exe works now but the material is still having the same issue. Looks like this one was not fixed in the recent IMK update. You can refer back tot he opensees thread for the model parameters and load history.

amaelkady commented 3 years ago

alright then; I will fix it and get back to you (hopefully in couple of days at most).

amaelkady commented 3 years ago

@jwberman hi Jeff, I updated the executable; hopfully the bug is now fixed (see attached fig for the case you sent me). Please try it yourself and let me know debug .

jwberman commented 3 years ago

This appears to be working now!! Thanks so much!

amaelkady commented 3 years ago

That's good to know. I will submit the updates to M. Scott by end of week, pending further validations. If anything comes up meanwhile, please let me know.

jwberman commented 3 years ago

Hi Ahmad, Here is another set of parameters that is causing an incorrect response, although it is converging: mat_tag = mat_tag + 1; Ke=324.4; Up_pos=7.9; Upc_pos=52.6; Uu_pos=67.7; Fy_pos=1098.1; FmaxFy_pos=1.068; FresFy_pos=0.02; Up_neg=7.9; Upc_neg=52.6; Uu_neg=67.7; Fy_neg=1098.1; FmaxFy_neg=1.068; FresFy_neg=0.02; Lamda_S=68; Lamda_C=1000; Lamda_K=1000; c_S=1; c_C=1; c_K=1; D_pos=1; D_neg=1;

Loading: i_max_disp = [5 10 15 20 25 30 35 40 45 50]; % input displacements for cyclic cycles i_Ncycles = [2 2 2 2 2 2 2 2 2 2]; % number of cycles per input displacement

Response plot, the last half cycle is shooting up to what looks like the undeteriorated backbone: IMKBilinMistake

jwberman commented 3 years ago

Shoot, just realized I spelled your name wrong....sorry about that. Noted, Ahmed.

jwberman commented 3 years ago

Just realized I had the step size up way too high. Was at 0.5 (for the cycle amplitudes above). Ran it at 0.02 and it looks fine. Still maybe worth a check though.

amaelkady commented 3 years ago

Good! I will check it as well but yes, generally, the step size need to be reasonably small to correctly capture new excursions and load reversal transitions..

amaelkady commented 3 years ago

I updated the executable to fix this issue associated with large increments. Nonetheless, it is recommended to have a refined step increment (as small as possible) to get a better response with smooth transitions between branches.

jwberman commented 1 year ago

Hi Ahmed, Do you know if your fixes to IMK Bilin are in the latest OpenSeesPy?

Thanks, Jeff

amaelkady commented 1 year ago

@jwberman Hi Jeff, not really sure. I would ask M. Scott or Siliva. Please also note that in November, there has been a major update for all the IMK models which you can find on OpenSees master repo or in my GitHub repos. These models will show an echo message when called stating the version (currently, "Nov22")