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.
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
Datatype
DAEMesh
for DAEs #384Datatype
DAEMesh
for DAEs #384Changes from 6 commits
901e3e4
18806f3
2ca0d5c
64dbb94
c7f4cfd
eaf4448
20b8f40
b001b7c
0fd9525
49de7ef
bbb605a
9594e1d
68cc407
6315781
88e8175
af1848e
87d6d56
ad0ca26
bda5014
c68305b
1555ee9
561061e
4b3c09c
da5e663
a45900d
82b692d
be4f248
04fdedc
58101ea
522d80c
3af100f
bdc548c
9540354
40bf6c6
70ef02e
75d1d8b
27ea264
d44e2a2
e24ff2b
486cfbb
c359d22
5921aff
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
But here @pancetta and @brownbaerchen the error is computed over the differential variables. When I would use the generic error hook classes I got the maximum error over both the differential and the algebraic variables. I can rename it such that it is clear what this hook class does. So, I still would like to keep this class at least for the semi-explicit DAEs. For the fully-implicit DAEs the generic error hook classes does make sense to use!
Check warning on line 109 in pySDC/projects/DAE/misc/HookClass_DAE.py
Codecov / codecov/patch
pySDC/projects/DAE/misc/HookClass_DAE.py#L109
Check warning on line 112 in pySDC/projects/DAE/misc/HookClass_DAE.py
Codecov / codecov/patch
pySDC/projects/DAE/misc/HookClass_DAE.py#L111-L112
Check warning on line 114 in pySDC/projects/DAE/misc/HookClass_DAE.py
Codecov / codecov/patch
pySDC/projects/DAE/misc/HookClass_DAE.py#L114
Check warning on line 117 in pySDC/projects/DAE/misc/HookClass_DAE.py
Codecov / codecov/patch
pySDC/projects/DAE/misc/HookClass_DAE.py#L116-L117
Check warning on line 119 in pySDC/projects/DAE/misc/HookClass_DAE.py
Codecov / codecov/patch
pySDC/projects/DAE/misc/HookClass_DAE.py#L119
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.
I am curious. When is this relevant? Why don't you use
mesh
if there is no algebraic part? Are there places in the code where you solve DAEs but temporarily remove the algebraic part?Check warning on line 99 in pySDC/projects/DAE/misc/dae_mesh.py
Codecov / codecov/patch
pySDC/projects/DAE/misc/dae_mesh.py#L99