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

allow other RHS in analysis callback #1205

Merged
merged 3 commits into from
Aug 11, 2022
Merged

Conversation

ranocha
Copy link
Member

@ranocha ranocha commented Aug 11, 2022

This fixes the problem brought up in #1202 (comment). It allows us to use an ODE RHS different from our rhs! function. This can be very nice, e.g., for hyperbolic-parabolic systems set up as SplitODEProblem with rhs! and rhs_parabolic! in #1149 or when the user wants to modify the RHS call as in https://discourse.julialang.org/t/ann-trixi-jl-v0-3-sciml-integration-and-a-new-modular-approach-for-easy-extension/50419/43.

I made this a PR to main instead of dev or so since it is generally useful and should be shipped right now.

This is a cleaned-up version of #1204.

This was referenced Aug 11, 2022
@ranocha ranocha requested a review from sloede August 11, 2022 08:35
Co-authored-by: Michael Schlottke-Lakemper <[email protected]>
@ranocha ranocha requested a review from sloede August 11, 2022 08:41
@ranocha ranocha enabled auto-merge (squash) August 11, 2022 08:41
Copy link
Member

@sloede sloede left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@codecov
Copy link

codecov bot commented Aug 11, 2022

Codecov Report

Merging #1205 (02a1c22) into main (22348a7) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1205   +/-   ##
=======================================
  Coverage   96.78%   96.78%           
=======================================
  Files         310      310           
  Lines       24284    24284           
=======================================
  Hits        23501    23501           
  Misses        783      783           
Flag Coverage Δ
unittests 96.78% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/time_integration/methods_3Sstar.jl 95.88% <ø> (ø)
src/callbacks_step/analysis.jl 96.27% <100.00%> (ø)
src/time_integration/methods_2N.jl 93.51% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@ranocha ranocha merged commit 787815e into main Aug 11, 2022
@ranocha ranocha deleted the hr/fix_analysis_rhs_clean branch August 11, 2022 11:27
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

Successfully merging this pull request may close these issues.

2 participants