Closed DhairyaLGandhi closed 2 months ago
Attention: Patch coverage is 0%
with 4 lines
in your changes are missing coverage. Please review.
Project coverage is 31.74%. Comparing base (
9d87ca0
) to head (3e64e25
). Report is 24 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
ext/SciMLBaseChainRulesCoreExt.jl | 0.00% | 2 Missing :warning: |
ext/SciMLBaseZygoteExt.jl | 0.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checklist
Additional context
SDESolution
doesn't seem to exist anymore. The more appropriate type seems to beRODESolution
. This PR removes mentions of SDESolution and replaces them withRODESolution
. Further, one of the adjoints was incorrect. It was definining an adjoint over SDEProblem, but instead tries returning SDESolution in the forwards pass. Not sure how this passed tests thoughAdd any other context about the problem here.