-
-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add NNKolmogorov and NNParamKolmogorov #83
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #83 +/- ##
==========================================
+ Coverage 61.97% 68.20% +6.22%
==========================================
Files 8 10 +2
Lines 647 758 +111
==========================================
+ Hits 401 517 +116
+ Misses 246 241 -5 ☔ View full report in Codecov by Sentry. |
a158d47
to
1f89dee
Compare
Needs #81 to go in first. |
2799c06
to
9f19381
Compare
1e61848
to
8fb87fe
Compare
```math | ||
\partial_t u(t,x) = \mu(t, x, γ_mu) \nabla_x u(t,x) + \frac{1}{2} \sigma^2(t, x, γ_sigma) \Delta_x u(t,x) | ||
``` | ||
with initial condition given by `g(x, γ_phi)` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
terminal condition? I'm not sure this really matches??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From : https://arxiv.org/pdf/1806.00421.pdf
Incase of forward eqns it is the initial condition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes in forward equations it's an initial condition. But for backwards we have a terminal condition. How are the two separate cases being handled?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't in this case the conversion as simple as v(t,x) = u(T - t, x)
for changing from fwd to backward. I found this:
I got this from here : https://jberner.info/data/Thesis_Berner.pdf
```math | ||
\partial_t u(t,x) = \mu(t, x) \nabla_x u(t,x) + \frac{1}{2} \sigma^2(t, x) \Delta_x u(t,x) | ||
``` | ||
with initial condition given by `g(x)` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Terminal condition?
Checklist
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
Additional context
Add any other context about the problem here.