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

API structure #1

Open
theogf opened this issue Feb 8, 2021 · 3 comments
Open

API structure #1

theogf opened this issue Feb 8, 2021 · 3 comments

Comments

@theogf
Copy link
Owner

theogf commented Feb 8, 2021

So Bayesian quadrature is quite unlike other quadrature packages, therefore we need to think how to approach it.

On top of the likelihood/prior input, there are two important playing parameters:

  • The method to select new samples
  • The method to evaluate the different integrals (vanilla or with transformation)

I think for simplicity it is better to pass these two separately as there is no real benefit to embed the first one in the second.

@theogf
Copy link
Owner Author

theogf commented Feb 8, 2021

An additional point I forgot to consider was on how to compute the kernel means, i.e :
m(x) = \int k(x, x') p(x')dx'

@johannesgiersdorf
Copy link
Collaborator

On top of the likelihood/prior input, there are two important playing parameters:

* The method to select new samples

We should keep in mind that some methods depend on prev. produced samples (e.g. active sampling) and it should be easy to create samples in batches (for hyper parameter estimation).

@theogf
Copy link
Owner Author

theogf commented Feb 8, 2021

Yep that's the idea! Have a look at AbstractMCMC.jl I think we should use their interface.

@theogf theogf mentioned this issue Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants