[BREAKING] Force use of Tangents
in pushforward
, pullback
and hvp
#455
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Warning
Breaking change:
pushforward
,pullback
andhvp
no longer accept plain objects asdx
ordy
: this argument must now be wrapped intoTangents
.Tip
Since
Tangents
becomes part of the public API, this also allows users to leverage batch mode directly. At the moment it is only useful with ForwardDiff, but we're working on adding Enzyme support.DI source
pushforward
,pullback
andhvp
to useTangents
and remove the fallbacks.Tangents
constructor so that it acceptsVararg
, also modify its second type parameterTangents
like indexing andmap
DI extensions
Tangents
constructorDI docs
Tangents
part of the public API, enabling users to access batch modeDIT source
Scenario
constructors so that they only acceptTangents
forpushforward
,pullback
andhvp