-
Notifications
You must be signed in to change notification settings - Fork 60
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
Some love to model comparison #315
Some love to model comparison #315
Conversation
Codecov ReportAttention: Patch coverage is ❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
|
fb49715
to
3a4c422
Compare
it seems the linter again failed for something not in this PR... |
That's on me. It's fixed on dev now. |
- individual models can rely on a shared simulations (makes mixed batches much more useable) - mixed batches are sampled more efficiently (batched samples per model, rather than batching individual simulations)
3a4c422
to
379f8c0
Compare
Thanks @stefanradev93! Tests are passing now |
Model comparison still had some rough edges, so I am taking the liberty to make it smoother.
.predict
method for the model comparison approximatormc_confusion_matrix_plot