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

Complete PR comments for hyperopt refactoring #2082

Merged
merged 5 commits into from
Jun 2, 2022
Merged

Complete PR comments for hyperopt refactoring #2082

merged 5 commits into from
Jun 2, 2022

Commits on Jun 2, 2022

  1. test: add test for merge_with_defaults 

    incorporate reviewer comments
    @jimthompson5802
    jimthompson5802 committed Jun 2, 2022
    Configuration menu
    Copy the full SHA
    27d921d View commit details
    Browse the repository at this point in the history

  2. test: additional assertion checks

    @jimthompson5802
    jimthompson5802 committed Jun 2, 2022
    Configuration menu
    Copy the full SHA
    2a67a05 View commit details
    Browse the repository at this point in the history

  3. refactor: split search_alg code into separate module

    @jimthompson5802
    jimthompson5802 committed Jun 2, 2022
    Configuration menu
    Copy the full SHA
    4d06095 View commit details
    Browse the repository at this point in the history

  4. [pre-commit.ci] auto fixes from pre-commit.com hooks 

    for more information, see https://pre-commit.ci
    @pre-commit-ci
    pre-commit-ci[bot] committed Jun 2, 2022
    Configuration menu
    Copy the full SHA
    11ae139 View commit details
    Browse the repository at this point in the history

  5. test: reviewer comment simplify test

    @jimthompson5802
    jimthompson5802 committed Jun 2, 2022
    Configuration menu
    Copy the full SHA
    4621bb5 View commit details
    Browse the repository at this point in the history