Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid damping values error #19

Closed
geoport opened this issue Oct 17, 2023 · 3 comments
Closed

Invalid damping values error #19

geoport opened this issue Oct 17, 2023 · 3 comments

Comments

@geoport
Copy link

geoport commented Oct 17, 2023

For a soil layer with the following properties I get "ValueError: The provided damping values must be between [0, 100]." error.
Thk : 7 m
Vs : 300 m/s
Damping : 2%
Density : 1500 kg/m3

@jsh9
Copy link
Collaborator

jsh9 commented Oct 17, 2023

Hi, could you provide more details for me to reproduce this issue? For example, the code and/or data files you were using.

Thanks!

@geoport
Copy link
Author

geoport commented Oct 31, 2023

You can try to run Pipeline_05_Nonlinear_Method.ipynb with the following vs_profile

vs_matrix = np.array([[7, 300, 0.01, 1500, 1]])
vs_profile = Vs_Profile(vs_matrix,add_halfspace=True)

after running

sim_results = nl_sim.run(
      show_fig=True, 
      save_txt=False, 
      remove_sim_dir=True,
  )

it throws ValueError: The provided damping values must be between [0, 100]. error. I know how to disable this error but I wonder if I should.

@xia-fr
Copy link
Collaborator

xia-fr commented Sep 24, 2024

Hello, this has been addressed in #22 .

From the notes:
For nonlinear simulations, the dummy curves for ggmax and xi are now created separately from their corresponding calibrated parameters. The values contained in these dummy curves are not consequential for the simulations, but previously both the ggmax and xi curves were generated off of only the ggmax parameters. This would occasionally trigger a ValueError related to unrealistic damping values, since the ggmax parameters are only calibrated to guarantee the shape of the ggmax curve.

@xia-fr xia-fr closed this as completed Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants