-
Notifications
You must be signed in to change notification settings - Fork 427
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
Lazy index #5488
Lazy index #5488
Conversation
CodSpeed Performance ReportMerging #5488 will improve performances by ×26Comparing Summary
Benchmarks breakdown
|
@zklaus we currently don't have any memray tests hence the memray CI job fails, do we have a test to add here or should we remove the memray CI job? |
@zklaus @kenodegard @jaimergp Do we not have the proper benchmark to see performance improvements from this change? Should we write one? |
@beckermr I agree adding a benchmark to see the improvement would be great. I'll take a look and see if I can come up with something simple. |
Regarding memray, it would be nice to have a similar reading about possible memory usage improvements. I'll see if I can add something simple to #5508 quickly. |
Could we cut a new release please with this change? We have memory problems on Azure 😔 |
I guess this will be included in 24.11. |
Description
Update conda-build to use the new
conda.core.index.Index
.Xref conda/conda#13880
Xref #5154
Checklist - did you ...
news
directory (using the template) for the next release's release notes?