-
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
Replace broadcasted lambda with explicit loop #738
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #738 +/- ##
==========================================
- Coverage 96.33% 96.27% -0.06%
==========================================
Files 34 34
Lines 3353 3356 +3
==========================================
+ Hits 3230 3231 +1
- Misses 123 125 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Benchmark Report for /home/runner/work/MixedModels.jl/MixedModels.jlJob Properties
ResultsA ratio greater than
Benchmark Group ListHere's a list of all the benchmark groups executed by this job:
Julia versioninfoTarget
Baseline
|
@palday This looks reasonable. Should we bring it up-to-date with the current main branch and merge it? In other words, is there a reason that you have left this PR as a Draft? |
@dmbates It was one of a few performance experiments I was doing while waiting for code in other projects to precompile. 😂 I think we can go ahead and merge it if you're happy? |
Thanks for contributing!
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: