We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
likelihood()
The broader question is whether it would not be clearer to define S3 methods for numeric, epichains and epichains_summary where
numeric
epichains
epichains_summary
likelihood.epichains <- function(chains, ...) { likelihood(summary(chains, ...)) } likelihood.epichains_summary <- function(chains, stat_max = Inf, ...) { stat_max_from_sim <- attr(chains, "stat_max") if (missing(stat_max)) { stat_max <- stat_max_from_sim } else if (stat_max != stat_max_from_sim) { warning(`stat_max` is different from the one used in the simulations given in `chains`") } likelihood(as.numeric(chains, stat_max, ...)) }
and the current likelihood() becomes likelihood.numeric()
likelihood.numeric()
Originally posted by @sbfnk in #213 (comment)
The text was updated successfully, but these errors were encountered:
This blog post by Hugo is relevant to this issue: https://epiverse-trace.github.io/posts/s3-generic/.
Sorry, something went wrong.
No branches or pull requests
The broader question is whether it would not be clearer to define S3 methods for
numeric
,epichains
andepichains_summary
whereand the current
likelihood()
becomeslikelihood.numeric()
Originally posted by @sbfnk in #213 (comment)
The text was updated successfully, but these errors were encountered: