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

Move 'evidence_epsilon' in 'est_evidence_mcmc' #87

Closed
richelbilderbeek opened this issue Feb 5, 2019 · 2 comments
Closed

Move 'evidence_epsilon' in 'est_evidence_mcmc' #87

richelbilderbeek opened this issue Feb 5, 2019 · 2 comments
Assignees
Labels
high priority Do this first
Projects

Comments

@richelbilderbeek
Copy link
Owner

richelbilderbeek commented Feb 5, 2019

Replace:

create_pir_params <- function(
  # ...
  evidence_epsilon = 1e-12,
  evidence_filename = tempfile(fileext = ".csv")
)

by

create_pir_params <- function(
  # ...
  est_evidence_mcmc = create_nested_sampling_mcmc(),
  # ...
)

which has the same epsilon as a default.

As shown in

overview

@richelbilderbeek richelbilderbeek self-assigned this Feb 5, 2019
@richelbilderbeek richelbilderbeek added this to To do in v1.2 via automation Feb 5, 2019
@richelbilderbeek richelbilderbeek added the high priority Do this first label Feb 5, 2019
@richelbilderbeek richelbilderbeek moved this from To do to In progress in v1.2 Feb 5, 2019
@richelbilderbeek
Copy link
Owner Author

Nonsense, must be part of experiment 😊

@richelbilderbeek
Copy link
Owner Author

And is already there... closing this nonsense Issue ...

v1.2 automation moved this from In progress to Done Feb 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority Do this first
Projects
v1.2
  
Done
Development

No branches or pull requests

1 participant