Skip to content

Optuna and sweeps examples #338

Merged
merged 11 commits into from
Oct 10, 2022
Merged

Optuna and sweeps examples #338

merged 11 commits into from
Oct 10, 2022

Conversation

martins0n
Copy link
Contributor

@martins0n martins0n commented Dec 1, 2021

IMPORTANT: Please do not create a Pull Request without creating an issue first.

Before submitting (must do checklist)

  • Did you read the contribution guide?
  • Did you update the docs? We use Numpy format for all the methods and classes.
  • Did you write any new necessary tests?
  • Did you update the CHANGELOG?

Type of Change

  • Examples / docs / tutorials / contributors update
  • Bug fix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves an existing feature)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

Closing issues

closes #288

@codecov-commenter
Copy link

codecov-commenter commented Dec 1, 2021

Codecov Report

Merging #338 (251eb63) into master (261d930) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #338   +/-   ##
=======================================
  Coverage   85.43%   85.43%           
=======================================
  Files         148      148           
  Lines        7962     7962           
=======================================
  Hits         6802     6802           
  Misses       1160     1160           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

github-actions bot commented May 12, 2022

🚀 Deployed on https://deploy-preview-338--etna-docs.netlify.app

@github-actions github-actions bot temporarily deployed to pull request May 12, 2022 13:23 Inactive
iKintosh
iKintosh previously approved these changes May 12, 2022
Copy link
Contributor

@iKintosh iKintosh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve once again :)

@github-actions github-actions bot temporarily deployed to pull request October 3, 2022 15:31 Inactive
@github-actions github-actions bot temporarily deployed to pull request October 4, 2022 13:40 Inactive
@github-actions github-actions bot temporarily deployed to pull request October 4, 2022 13:44 Inactive
@martins0n martins0n changed the title Optuna and sweeps examples with feature selection Optuna and sweeps examples Oct 4, 2022
@martins0n martins0n marked this pull request as ready for review October 4, 2022 14:46
@github-actions github-actions bot temporarily deployed to pull request October 4, 2022 14:47 Inactive
@alex-hse-repository alex-hse-repository requested review from iKintosh and alex-hse-repository and removed request for iKintosh October 5, 2022 07:45
Copy link
Collaborator

@alex-hse-repository alex-hse-repository left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Make examples folder be handled by linters
  2. Update Changelog
  3. Add some comments inside the scripts for each logical block

examples/optuna/README.md Outdated Show resolved Hide resolved
examples/optuna/optuna_example.py Outdated Show resolved Hide resolved
examples/optuna/optuna_example.py Outdated Show resolved Hide resolved
examples/optuna/optuna_example.py Outdated Show resolved Hide resolved
examples/optuna/README.md Show resolved Hide resolved
examples/wandb/sweeps/config.yaml Outdated Show resolved Hide resolved
examples/wandb/sweeps/sweeps_example.py Outdated Show resolved Hide resolved
examples/wandb/sweeps/sweeps_example.py Outdated Show resolved Hide resolved
examples/wandb/sweeps/sweeps_example.py Show resolved Hide resolved
pipeline: Pipeline = hydra_slayer.get_from_params(**pipeline)
backtest_configs = hydra_slayer.get_from_params(**backtest)

init_logger(pipeline.to_dict())
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might work incorrectly sometimes, may be use config["pipeline"]?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think, it's better to use canonical way and to fix corner cases of to_dict.

Co-authored-by: alex-hse-repository <55380696+alex-hse-repository@users.noreply.github.com>
@github-actions github-actions bot temporarily deployed to pull request October 6, 2022 09:52 Inactive
@github-actions github-actions bot temporarily deployed to pull request October 8, 2022 11:09 Inactive
@alex-hse-repository alex-hse-repository merged commit 0e8fee9 into master Oct 10, 2022
@martins0n martins0n deleted the optuna-script branch October 10, 2022 11:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add Optuna script example
4 participants