-
Notifications
You must be signed in to change notification settings - Fork 12
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
the (AD) derivative of the function **atan2** #83
Comments
What is the type of |
which I guess should qualify
e.g. in
|
we can use std::atan(_temperature[_qp]) in "MetaPhysicL" but not for std::atan2(_grad_u[_qp] (1), _grad_u[_qp] (0)), _grad_u[_qp] (1), _grad_u[_qp] (0) both are not vectors... |
Ah, I see; the markdown formatting turned those gradient indices into hyperlinks before. I'd still want to know the type of those arguments; the exact compiler error message you get would probably have the type in it. |
As i mentioned before: If we use std::atan2(_grad_u[_qp] (1), _grad_u[_qp] (0)) the residual with an ADKernel in MOOSE , then we will have the error:
Cite: I'd still want to know the type of those arguments; the exact compiler error message you get would probably have the type in it.
....... |
Oh, I misunderstood, then - I thought you'd had some difficulty compiling that line so you tried to get away without it (taking the raw_type, say). If that line compiled then this is the answer to your question:
Because in any case where a derivative exists, we never "half-define" a function on Forgive my suspicion here, but: what are your initial conditions? If you have a zero gradient at any quadrature point in the field, and if you try to use |
Hi MetaPhysicL team,
is the (AD) derivative of the function atan2 not defined in "MetaPhysicL"?
If we use std::atan2(_grad_u_qp, _grad_u_qp) for the residual with an ADKernel in MOOSE , then we will have the error:
The text was updated successfully, but these errors were encountered: