-
Notifications
You must be signed in to change notification settings - Fork 48
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
allow the user to disable amalgamation #673
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #673 +/- ##
==========================================
+ Coverage 95.72% 95.81% +0.08%
==========================================
Files 35 35
Lines 3226 3247 +21
==========================================
+ Hits 3088 3111 +23
+ Misses 138 136 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
@palday thanks for putting this PR together. It appears to solve the issues we have been having with model fitting. Do you have a sense of when this will be merged and released? I would be happy to contribute to any testing or documentation that is still needed prior to merge. Thanks again! |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@liamlundy this should now be available in the latest release |
closes #638
Did behavior change? Did you add need features? If so, please update NEWS.md
docs/NEWS-update.jl
to update the cross-references.Should we release your changes right away? If so, bump the version: