-
Notifications
You must be signed in to change notification settings - Fork 13
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
IMEX Multistage #453
Merged
Merged
IMEX Multistage #453
Changes from 24 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
1a56e1b
add unit test
alexbrown1995 fae09a9
Needs tidying, but IMEX_Euler3 is a seemingly working version of IMEX…
alexbrown1995 97ff98f
Tidy and adding new schemes
alexbrown1995 1a1fdc0
Merge branch 'main' into IMEX_multistage
alexbrown1995 aa2d599
Add back in courant logging, tidy time disc
alexbrown1995 b1b4cf5
change to split advective and div term in SWE
alexbrown1995 00f9966
SSP3 correction
alexbrown1995 a6b28a1
Tidy of code, lint corrections
alexbrown1995 9fef4b2
Merge branch 'main' into IMEX_multistage
alexbrown1995 c8207bc
revert explicitmultistage changes
alexbrown1995 78f5557
revert test changes
alexbrown1995 177c308
adressed most review comments
alexbrown1995 3a34def
Warning for explicit implicit splitting added
alexbrown1995 564cafb
Comments added and equations refactor moved into time_discretisation.…
alexbrown1995 31ddf80
Splitting on conservative term added. Need to make it work in case of…
alexbrown1995 0c6fba4
Merge to main
alexbrown1995 85e57a7
Lint fix
alexbrown1995 10ed951
Labelling of implicit and explicit terms moved outside of time_disc, …
alexbrown1995 e866193
change error message
alexbrown1995 cccc433
Merge to main
alexbrown1995 5878f23
lint fix + firedrake fml move
alexbrown1995 958e002
Error switched to Runtime
alexbrown1995 6daae50
linearisation terms added
alexbrown1995 9191901
bug fix to test func / q
alexbrown1995 d29d137
Update gusto/common_forms.py
atb1995 7a14129
Change in description
alexbrown1995 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -90,6 +90,8 @@ def __call__(self, target, value=None): | |
# ---------------------------------------------------------------------------- # | ||
|
||
time_derivative = Label("time_derivative") | ||
implicit = Label("implicit") | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nice to see these labels added! |
||
explicit = Label("explicit") | ||
transport = Label("transport", validator=lambda value: type(value) == TransportEquationType) | ||
diffusion = Label("diffusion") | ||
transporting_velocity = Label("transporting_velocity", validator=lambda value: type(value) in [Function, ufl.tensors.ListTensor]) | ||
|
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you mention that this will split up all conservative terms in the equation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done! Thanks Tom