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.

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

[HPO] enable HPO with segmentation #919

Merged
merged 3 commits into from
Feb 22, 2022
Merged

Conversation

eunwoosh
Copy link
Contributor

This PR includes changes for HPO with segmentation task.

The list of changes are as follows:

  • Implement code of setting epoch for segmentation task.
  • Implement HPO test code in segmentation test.

@eunwoosh eunwoosh requested review from Ilya-Krylov and a team February 14, 2022 13:12
@github-actions github-actions bot added CLI Any changes in OTE CLI TEST Any changes in tests labels Feb 14, 2022
@eunwoosh
Copy link
Contributor Author

run ote_sdk test

@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

@nervana-ff
Copy link

Can one of the admins verify this patch?

@eunwoosh eunwoosh force-pushed the es/hpo_segmentation branch from d11df25 to f4ed13d Compare February 16, 2022 07:22
@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

@Ilya-Krylov
Copy link
Contributor

10:30:35 isort (ote_sdk)..........................................................Passed
10:30:35 isort (ote_cli|external).................................................Passed
10:30:38 black (ote_sdk|ote_cli)..................................................Failed
10:30:38 - hook id: black
10:30:38 - files were modified by this hook
10:30:38
10:30:38 reformatted ote_cli/ote_cli/utils/hpo.py
10:30:38 All done! ✨ 🍰 ✨
10:30:38 1 file reformatted, 228 files left unchanged.
10:30:38
10:30:38 black (rest).............................................................Passed
10:30:40 flake8...................................................................Passed
10:30:40 prettier.................................................................Failed
10:30:40 - hook id: prettier
10:30:40 - files were modified by this hook
10:30:40
10:30:40 .github/labeler.yml
10:30:40
10:30:46 mypy (ote_sdk)...........................................................Passed
10:30:46 mypy (ote_cli)...........................................................Passed
10:30:47 mypy (external)..........................................................Passed
10:31:09 pylint...................................................................Failed
10:31:09 - hook id: pylint
10:31:09 - exit code: 12
10:31:09
10:31:09 ************* Module ote_cli.utils.hpo
10:31:09 ote_cli/ote_cli/utils/hpo.py:98:0: R0914: Too many local variables (18/15) (too-many-locals)
10:31:09 ote_cli/ote_cli/utils/hpo.py:156:0: R1721: Unnecessary use of a comprehension (unnecessary-comprehension)
10:31:09 ote_cli/ote_cli/utils/hpo.py:159:57: W0108: Lambda may not be necessary (unnecessary-lambda)

@eunwoosh eunwoosh force-pushed the es/hpo_segmentation branch from f4ed13d to eb683a1 Compare February 18, 2022 07:07
@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

2 similar comments
@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

@eunwoosh eunwoosh force-pushed the es/hpo_segmentation branch from eb683a1 to 14c1d94 Compare February 21, 2022 01:05
@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

4 similar comments
@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

@eunwoosh
Copy link
Contributor Author

run ote_sdk tests

@Ilya-Krylov
Copy link
Contributor

run ote_sdk tests

@eunwoosh
Copy link
Contributor Author

@Ilya-Krylov Could you check this PR?

@Ilya-Krylov Ilya-Krylov merged commit c1c3753 into develop Feb 22, 2022
@Ilya-Krylov Ilya-Krylov deleted the es/hpo_segmentation branch February 22, 2022 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI Any changes in OTE CLI TEST Any changes in tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants