Closed sritchie closed 3 months ago
Attention: Patch coverage is 62.74510%
with 19 lines
in your changes missing coverage. Please review.
Project coverage is 87.64%. Comparing base (
8fb417f
) to head (36d115c
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
183:
adds
emmy.{autodiff, tape}
toemmy.sci
's exported namespace setadds
emmy.dual/extract-id
implementations for all supported output types (every type that already implementsemmy.dual/IPerturbed
)moves
emmy.tape/Completed
toemmy.dual/Completed
; it doesn't really make sense there, but this is the only current way to remove the circular dependency betweenemmy.dual
andemmy.tape
. (tape
needs adual
import to gainemmy.dual/IPerturbed
.)simplifies the
emmy.tape/gradient
implementation down to only handle single real-or-structural arguments, just likeemmy.dual/derivative
. We'll share the "handle-multiple-input" implementation between the two in a follow-up PRmakes the tests in
emmy.calculus.derivative
generic on the derivative implementation, so we can run all tests in forward and reverse mode.