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

kde to support boundaries and adaptive bandwidth #45

Open
cranmer opened this issue Mar 29, 2016 · 1 comment
Open

kde to support boundaries and adaptive bandwidth #45

cranmer opened this issue Mar 29, 2016 · 1 comment

Comments

@cranmer
Copy link
Member

cranmer commented Mar 29, 2016

Does scikit-learn's normal kde alg support adaptive bandwidth?

Does it support boundary's? The s(x) function has a min and a max. It would be nice if we could enforce the kde has all probability within s_min and s_max

this worked well in the past
http://arxiv.org/abs/hep-ex/0011057

@glouppe
Copy link
Contributor

glouppe commented Mar 29, 2016

The current implementation of carl.distributions.KernelDensity is based on scipy.stats.gaussian_kde, which unfortunately does not support either of these features. It would be nice indeed, but this still would require some work to be done upstream in scipy. (or alternatively, to extend sklearn.neighbors.KernelDensity)

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