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

MNPE class similar to MNLE #1362

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

MNPE class similar to MNLE #1362

wants to merge 6 commits into from

Conversation

dgedon
Copy link
Collaborator

@dgedon dgedon commented Jan 10, 2025

Implementation of mixed NPE where we have some continuous parameters theta followed by one (or multiple with this PR #1269) discrete parameters. The observation space is fully continuous.

Deprecated mnle.py in net_builders und unified mnle/mnpe as mixed_nets.py.

@dgedon
Copy link
Collaborator Author

dgedon commented Jan 17, 2025

Update:

  • MNPE and tests are implemented
  • for test with Bernoulli prior, I had to change mcmc_transforms to handle discrete distirbution. As default we just compute mean/std for discrete distributions
  • currently MNPE with embedding nets does not work yet. Gives some backwards inplace operations error that I couldn't solve yet.

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

Successfully merging this pull request may close these issues.

1 participant