Skip to content
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

adjoint sensitivities #4

Open
oyvfos opened this issue Jun 28, 2019 · 6 comments
Open

adjoint sensitivities #4

oyvfos opened this issue Jun 28, 2019 · 6 comments

Comments

@oyvfos
Copy link

oyvfos commented Jun 28, 2019

Hi, great work!
Any plans to add adjoint sensitivities?
Kr, oyvind?

@sn248
Copy link
Owner

sn248 commented Jun 29, 2019

Hi Oyvind

Yes, it is on my to-do list, after

  • adding events to cvode to enable repeated initialization

  • adding ability to specify specific y and p elements for forward sensitivities

You want to get a crack on it? I can help!

S. Nayak

@oyvfos
Copy link
Author

oyvfos commented Jun 30, 2019

Hi Satya,
I am interested. But I now realize that the current version does not support spatial discretization (see cvAdvDiff_bnd.c). This I need for my application (Hull White PDE). Maybe I can start with that ? What is your developing environment ? I am using windows. Is there a minimal developing environement? Can I use inline Rcpp for instance? Kr, Oyvind

@sn248
Copy link
Owner

sn248 commented Jun 30, 2019

Hi Oyvind

Yes, please start with cvAdvDiff_bnd.c, that looks like a plan. Windows is fine, we do have to check on all the environments CRAN supports to get accepted there. I am using Rcpp, although not the inline function. I will take a look at cvAdvDiff_bnd.c to see if it needs anything else from SUNDIALS which is not included in the package src directory yet.

Thanks!
Satya

@oyvfos
Copy link
Author

oyvfos commented Jul 1, 2019

Hi, I am set up. Didnt know that Rstudio came with an IDE. Is there a branch we can cooparate on? Did you have any structure in mind?
Kr, Oyvind

@oyvfos
Copy link
Author

oyvfos commented Jul 2, 2019

Hi Satya,
I suggest restricting the set-up to a convection-diffusion form of the PDE. I will use cvAdvDiff_bnd.c as a template. In addition to the current input of cvode also a space vector, coefficients a,b,c and d have to be submitted by the user. These parameters and the ic needs to be functions of t and x. See https://developers.opengamma.com/quantitative-research/numerical-solutions-to-pdes-with-financial-applications-opengamma.pdf for a reference. Can you help with this?

@sn248
Copy link
Owner

sn248 commented Jul 2, 2019

Hi Ovyfos

Thanks for the reference. I will take a look at cvAdvDiff_bnd.c. I might be able to implement it in the package.

Satya

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants