You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In #23, we realized that while it's straightforward to test sensitivities for one-argument UnionAll constructors, e.g. Symmetric(X) and Diagonal(X), things can get more complicated when attempting to test constructors for concrete subtypes. The example in the linked PR was for Symmetric{T,M}, which requires a second argument when used as constructor. We should find a way to make it easier to test such things, perhaps by refactoring rrule_test?
The text was updated successfully, but these errors were encountered:
I noticed last night that we'd need to be careful about which argument is the one we're checking. Consider e.g. BLAS functions, which often have the differentiable argument last.
In #23, we realized that while it's straightforward to test sensitivities for one-argument
UnionAll
constructors, e.g.Symmetric(X)
andDiagonal(X)
, things can get more complicated when attempting to test constructors for concrete subtypes. The example in the linked PR was forSymmetric{T,M}
, which requires a second argument when used as constructor. We should find a way to make it easier to test such things, perhaps by refactoringrrule_test
?The text was updated successfully, but these errors were encountered: