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.

Sign up for GitHub

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

Jump to bottom

State-of-the-art density estimators #430

Open
johannbrehmer opened this issue Jul 1, 2020 · 0 comments
Open

State-of-the-art density estimators #430

johannbrehmer opened this issue Jul 1, 2020 · 0 comments
Assignees
@johannbrehmer
Labels
potential student project Might be suitable for a short student project

Comments

@johannbrehmer
Copy link
Collaborator

Hi,

the NDE and SCANDAL methods currently rely on our own MAF implementation, which is not really state-of-the-art in neural density estimation anymore. I propose we switch to neural spline flows and use the implementation in the excellent nflows library.

This would break backwards compatibility with trained NDE / SCANDAL models, but I am not aware of anyone using these methods currently.

Cheers,
Johann
@johannbrehmer johannbrehmer self-assigned this Jul 1, 2020
@johannbrehmer johannbrehmer added the potential student project Might be suitable for a short student project label Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johannbrehmer johannbrehmer

Labels
potential student project Might be suitable for a short student project
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@johannbrehmer