Avatar for the conda user
conda
conda-build
BlogDocs

reduce cost of large variant matrix

#5392Merged
Comparing
minrk:reduce_variants
(
311e48b
) with
main
(
433f048
)
CodSpeed Performance Gauge
×2.6
Improvements
1
Regressions
0
Untouched
3
New
0
Dropped
0
Ignored
0

Benchmarks

Improved

test_render_recipe
tests/test_render.py::test_render_recipe
CodSpeed Performance Gauge
×2.6
64.7 s
25.1 s

Passed

test_pin_subpackage_benchmark
tests/test_api_render.py::test_pin_subpackage_benchmark
CodSpeed Performance Gauge
0%
7.4 s
7.3 s
test_which_package_battery
tests/test_inspect_pkg.py::test_which_package_battery
CodSpeed Performance Gauge
0%
834.2 ms
832.9 ms
test_select_lines_battery
tests/test_metadata.py::test_select_lines_battery
CodSpeed Performance Gauge
0%
22.8 ms
22.8 ms

Commits

Click on a commit to change the comparison range
Base
main
433f048
+20%
discard unused variants before copying metadata when variant matrix is large and mostly unused (as in conda-forge), the length of input_variants may be several thousand when only a few are actually used. This causes `get_loop_vars` and `metadata.copy()` to become very expensive.
9f5ef06
6 months ago
by minrk
+1%
try reducing with all used vars instead of loop vars should reduce less
568aed3
6 months ago
by minrk
-20%
remove redundant deepcopy of config.variant config.copy already copies this, no need to do it twice in metadata.copy
d1ba529
5 months ago
by minrk
0%
Merge branch 'main' into reduce_variants
5007189
5 months ago
by beckermr
0%
add config.copy_variants method to avoid calling pickle in too many places
5f48708
3 months ago
by minrk
0%
Merge branch 'main' into reduce_variants
0df82cc
3 months ago
by beckermr
0%
Update news/5392-variant-copy
644baaf
3 months ago
by beeankha
+158%
Add benchmark test for `render_recipe` (#5490)
311e48b
3 months ago
by kenodegard
Home Terms PrivacyDocs