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

Responses are blown up for simulations considering both 3D Velocity Model and topography when the damping type is Rayleigh damping #1

Open
Clifford-Yen opened this issue May 17, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@Clifford-Yen
Copy link
Owner

Clifford-Yen commented May 17, 2024

When using Rayleigh damping, 1D soil profile works with or without topography, but 3D velocity model only works without topography. Otherwise, the responses would be blown up to astronomical numbers.

Interestingly, the mass damping, no damping, and BKT2 work for the same analysis.

This issue is still under investigation. Please avoid using both 3D Velocity Model and topography at the same time.

@Clifford-Yen Clifford-Yen added the bug Something isn't working label May 17, 2024
@Clifford-Yen Clifford-Yen changed the title Responses are blown up for simulations considering both 3D Velocity Model and topography Responses are blown up for simulations considering both 3D Velocity Model and topography when the damping type is Rayleigh damping May 17, 2024
@Clifford-Yen
Copy link
Owner Author

Clifford-Yen commented May 24, 2024

After some testings, mass damping, no damping, and BKT2 not always work. In fact, although Rayleigh damping never work when considering both 3D Velocity Model and topography, it still generates different results (maybe only two types of result).

@Clifford-Yen
Copy link
Owner Author

The major problem of this issue should be fixed in 3b7754c. After fixing the incorrect memory access, the results generated with the same input file should be consistent now. However, the damping issue still persists. Lowering maximum_octant_level helps, but the worst performer is still the Rayleigh damping.

@Clifford-Yen
Copy link
Owner Author

#5 was created to separate the issue and to make it more clear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant