-
Notifications
You must be signed in to change notification settings - Fork 112
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
Filtering #978
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #978 +/- ##
==========================================
- Coverage 14.33% 14.22% -0.12%
==========================================
Files 253 255 +2
Lines 14303 14423 +120
==========================================
Hits 2051 2051
- Misses 12252 12372 +120 ☔ View full report in Codecov by Sentry. |
I've made some changes and now the PR only filters the residuals. This is more correct and also results in 0% overhead compared to no filtering. Hence, disregard the images above this comment. As this is my last day at TUM, @vikaskurapati is going to take this over. |
This PR adds filtering to SeisSol (compare with the filtering section in Hesthaven's Nodal DG book).
The idea is to decrease the coefficients of the high-order terms of the modal basis.
For matrix computation, refer to: SeisSol/Matrices#10
Filtering works (and is tested to some extent):
Works on the following architectures:
You can use it like this in the config file: