Skip to content

Make lgbm large actually large, rename load_rf_lgbm#35

Merged
Pavel Zwerschke (pavelzw) merged 2 commits into
mainfrom
lgbm-large
Mar 20, 2023
Merged

Make lgbm large actually large, rename load_rf_lgbm#35
Pavel Zwerschke (pavelzw) merged 2 commits into
mainfrom
lgbm-large

Conversation

@pavelzw

@pavelzw Pavel Zwerschke (pavelzw) commented Mar 20, 2023

Copy link
Copy Markdown
Member

This contains fixes from #15.

@github-actions

github-actions Bot commented Mar 20, 2023

Copy link
Copy Markdown

(benchmark 4467498829 / attempt 1)
Base results / Our results / Change

Model Size Dump Time Load Time
sklearn rf 77.2 MiB / 11.1 MiB / 6.94 x 0.07 s / 0.11 s / 1.56 x 0.08 s / 0.10 s / 1.25 x
sklearn rf LZMA 17.3 MiB / 5.3 MiB / 3.24 x 35.12 s / 5.81 s / 0.17 x 1.63 s / 0.57 s / 0.35 x
sklearn gb 2.2 MiB / 1.1 MiB / 2.08 x 0.03 s / 0.17 s / 5.89 x 0.03 s / 0.13 s / 3.89 x
sklearn gb LZMA 0.6 MiB / 0.2 MiB / 3.82 x 0.95 s / 0.37 s / 0.39 x 0.09 s / 0.15 s / 1.63 x
LGBM gbdt 5.3 MiB / 1.9 MiB / 2.81 x 0.17 s / 0.40 s / 2.40 x 0.03 s / 0.32 s / 12.94 x
LGBM gbdt LZMA 1.7 MiB / 0.8 MiB / 1.96 x 3.27 s / 0.99 s / 0.30 x 0.16 s / 0.39 s / 2.42 x
LGBM gbdt large 57.6 MiB / 18.9 MiB / 3.05 x 1.70 s / 4.10 s / 2.42 x 0.28 s / 3.33 s / 12.09 x
LGBM gbdt large LZMA 15.3 MiB / 6.7 MiB / 2.29 x 42.60 s / 12.58 s / 0.30 x 1.57 s / 4.14 s / 2.63 x
LGBM rf 1.6 MiB / 0.5 MiB / 3.42 x 0.05 s / 0.08 s / 1.82 x 0.01 s / 0.08 s / 13.81 x
LGBM rf LZMA 0.1 MiB / 0.1 MiB / 1.70 x 0.23 s / 0.12 s / 0.53 x 0.02 s / 0.09 s / 4.37 x

@pavelzw Pavel Zwerschke (pavelzw) changed the title Make lgbm large actually large Make lgbm large actually large, rename load_rf_lgbm Mar 20, 2023
@pavelzw Pavel Zwerschke (pavelzw) added the ci Changes in the CI configuration label Mar 20, 2023
@pavelzw Pavel Zwerschke (pavelzw) deleted the lgbm-large branch March 20, 2023 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes in the CI configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant