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

Document how we deal with errors in call-backs. #15146

Merged
merged 1 commit into from Apr 29, 2023
Merged

Conversation

bangerth
Copy link
Member

This is a proposal to deal with #15112. I'm not actually implementing anything specific, but I think it's useful to first put a design out there and discuss it that we can then refine and ultimately implement.

@bangerth
Copy link
Member Author

Since this seemed to find favor over in #15112, I would appreciate if it could be merged so that I can reference the glossary entry in the documentation of the classes in which I will implement this resolution.

Copy link
Member

@masterleinad masterleinad left a comment

Choose a reason for hiding this comment

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

Sounds like a reasonable approach.

* the problem such as the relative scaling of solution variables within the
* vector $\mathbf U$ that can be used to make the problem better conditioned.
*
* These functions are often called "call-backs" because the ODE or nonlinear
Copy link
Member

Choose a reason for hiding this comment

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

I think this spelling is more common.

Suggested change
* These functions are often called "call-backs" because the ODE or nonlinear
* These functions are often called "callbacks" because the ODE or nonlinear

*
* These functions are often called "call-backs" because the ODE or nonlinear
* solver libraries "call back" into user code. In code written in the C programming
* language, these call backs would often be described by pointers to user
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* language, these call backs would often be described by pointers to user
* language, these callbacks would often be described by pointers to user

@bangerth
Copy link
Member Author

I've standardized on "callback" as suggested.

@tjhei tjhei merged commit bdd2f2c into dealii:master Apr 29, 2023
11 of 14 checks passed
@tjhei
Copy link
Member

tjhei commented Apr 29, 2023

Hi @bangerth. Feel free to introduce the new exception but it would be great if we could wait with changing the return values of the interface until after @singima can post his PR.

@bangerth bangerth deleted the exc branch April 30, 2023 22:50
@bangerth
Copy link
Member Author

@tjhei For which interface is that pending patch? I played with the KINSOL interfaces so far, and will work my way through one library after the other. I can certainly push one to the back of the queue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants