Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Reduce peak memory usage during performance evaluation #450

Merged
merged 5 commits into from
Sep 13, 2023
Merged

Reduce peak memory usage during performance evaluation #450

merged 5 commits into from
Sep 13, 2023

Commits on Sep 12, 2023

  1. Merge get_submodls() and get_sub_summaries() to reduce peak memor… 

    …y usage

(before, the submodel fits (`outdmin` elements stored in `submodl` objects)
corresponding to the increasing submodel sizes along the predictor ranking were
stored at the same time, which led to a high peak memory usage; this is avoided
now).
    @fweber144
    fweber144 committed Sep 12, 2023
    Configuration menu
    Copy the full SHA
    f8b930d View commit details
    Browse the repository at this point in the history

Commits on Sep 13, 2023

  1. Fix a bug for K-fold CV introduced by merging get_sub_summaries() i… 

    …nto `get_submodls()`

in the last commit.
    @fweber144
    fweber144 committed Sep 13, 2023
    Configuration menu
    Copy the full SHA
    1e3b72f View commit details
    Browse the repository at this point in the history

  2. Fix a bug in varsel() introduced by merging get_sub_summaries() i… 

    …nto `get_submodls()`

in the second-to-last commit.
    @fweber144
    fweber144 committed Sep 13, 2023
    Configuration menu
    Copy the full SHA
    0e94602 View commit details
    Browse the repository at this point in the history

  3. Fix a bug for nprjdraws introduced by merging get_sub_summaries()… 

    … into `get_submodls()`

in the third-to-last commit.
    @fweber144
    fweber144 committed Sep 13, 2023
    Configuration menu
    Copy the full SHA
    2cc9c3f View commit details
    Browse the repository at this point in the history

  4. Add a NEWS.md entry for the reduced peak memory usage.

    @fweber144
    fweber144 committed Sep 13, 2023
    Configuration menu
    Copy the full SHA
    2d0e8a3 View commit details
    Browse the repository at this point in the history