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
KINSOL: allow for custom setup #15677
Conversation
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.
Yes, that's nice!
Out of curiosity, what is it that you're trying to customize?
include/deal.II/sundials/kinsol.h
Outdated
* | ||
* @code | ||
* ode.custom_setup = [&](void *arkode_mem) { | ||
* KINSetErrFile(arkode_mem, errfile); |
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.
Can you also show the code to set up errfile
?
Separately, is arkode_mem
really a void *
or does ARKode use an actual type for that?
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.
Internally they likely use a custom type for that but to users this type is always erased. When calling KINCreate
we are only handed a void*
and are supposed to hand it back to all "member functions". It's a C Library 🤷
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.
Ha, yes :-)
/rebuild |
d2a2d68
to
32a98ad
Compare
This functionality is already available in ARKODE and is now introduced equivalently to KINSOL. I modified test
kinsol_05
to test this functionality.