Vaibhav Thakkar
Vaibhav Thakkar
This is a known issue and related to #627, i.e. clad currently requires a default constructor for all class objects. This is required for the creation of temporary class instances...
@vgvassilev One improvement that we can make in this PR itself is to fix the printing order of generated functions; this can be done because we have dynamically created the...
> I think so, is there a catch? The only issue I see is that diagnostic messages won't align with the sequence of generated functions in the code dump. So,...
> Hm... I am confused - how is it different from now? Currently, if the order of differentiation is f1 followed by f2. Then, any diagnostic warning or error in...
> Let's have this improvement in a separate PR and then we can discuss further. What do you think? Sounds good 👍🏼
This is related to #740 for adding concurrency support.
After #873, the only remaining method is `ProcessDiffRequest` which is used for forward declaration of functions. This can be removed once we are able to capture the Differentiation requests graph...
This same issue happens even when passing parameters as arrays. I think the crux of the problem is similar to the one mentioned in https://github.com/vgvassilev/clad/issues/382. The function is computed twice,...
The crux of the issue seems to be that if some computation/expression is independent of the variables w.r.t which the differentiation is performed, these expressions can be treated as constant,...
related to https://github.com/vgvassilev/clad/issues/562