-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
IndexKeyMap
#192
base: main
Are you sure you want to change the base?
IndexKeyMap
#192
Conversation
Codecov Report
@@ Coverage Diff @@
## main #192 +/- ##
==========================================
- Coverage 81.77% 79.11% -2.66%
==========================================
Files 7 8 +1
Lines 439 613 +174
==========================================
+ Hits 359 485 +126
- Misses 80 128 +48
... and 3 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
6771d15
to
0a70754
Compare
We should probably also do something like this for `datafit` and `global_datafit`.
a4a9808
to
da12369
Compare
Bayesian Ensemble fitting does not return `Ensemble`s
We're allocating over 10_000 times as much memory as needed for the returned objects: julia> sol2 = @time solve(fit_enprob; saveat = t_forecast);
717.090530 seconds (5.89 G allocations: 865.902 GiB, 27.12% gc time) so performance is dreadful. |
This PR includes #191.
With the PR:
Master is around 400us, or maybe around 200us with the latest Unityper.