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

Update HPO documentation #3235

Merged
merged 8 commits into from
Mar 29, 2024
Merged

Conversation

eunwoosh
Copy link
Contributor

@eunwoosh eunwoosh commented Mar 28, 2024

Summary

This PR updates HPO documentation.
Additionally, I removed unnecessary part in HPO.

How to test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have added e2e tests for validation.
  • I have added the description of my changes into CHANGELOG in my target branch (e.g., CHANGELOG in develop).​
  • I have updated the documentation in my target branch accordingly (e.g., documentation in develop).
  • I have linked related issues.

License

  • I submit my code changes under the same Apache License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below).
# Copyright (C) 2024 Intel Corporation
# SPDX-License-Identifier: Apache-2.0

@github-actions github-actions bot added TEST Any changes in tests DOC Improvements or additions to documentation OTX 2.0 labels Mar 28, 2024
@harimkang harimkang added this to the 2.0.0 milestone Mar 29, 2024
Copy link

codecov bot commented Mar 29, 2024

Codecov Report

Attention: Patch coverage is 87.50000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 80.07%. Comparing base (b0c8583) to head (8d8830b).
Report is 11 commits behind head on releases/2.0.0.

Files Patch % Lines
src/otx/hpo/hpo_runner.py 0.00% 1 Missing ⚠️
src/otx/hpo/resource_manager.py 92.30% 1 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                 @@
##           releases/2.0.0    #3235      +/-   ##
==================================================
- Coverage           80.09%   80.07%   -0.03%     
==================================================
  Files                 186      186              
  Lines               15386    15397      +11     
==================================================
+ Hits                12324    12329       +5     
- Misses               3062     3068       +6     
Flag Coverage Δ
py310 80.07% <87.50%> (-0.03%) ⬇️
py311 80.07% <87.50%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@eunwoosh eunwoosh marked this pull request as ready for review March 29, 2024 06:45
@eunwoosh eunwoosh enabled auto-merge (squash) March 29, 2024 06:46
@eunwoosh eunwoosh merged commit 16683f2 into openvinotoolkit:releases/2.0.0 Mar 29, 2024
15 checks passed
@harimkang harimkang mentioned this pull request Apr 1, 2024
8 tasks
eunwoosh added a commit to eunwoosh/training_extensions that referenced this pull request Apr 3, 2024
* refactor hpo code

* write draft hpo docs

* update test

* bugfix

* align with pre-commit

* fix link format in docs

* update integration test

* hpo_config only gets HpoConfig
eunwoosh added a commit to eunwoosh/training_extensions that referenced this pull request Apr 9, 2024
* refactor hpo code

* write draft hpo docs

* update test

* bugfix

* align with pre-commit

* fix link format in docs

* update integration test

* hpo_config only gets HpoConfig
eunwoosh added a commit that referenced this pull request Apr 16, 2024
* Update HPO documentation (#3235)

* refactor hpo code

* write draft hpo docs

* update test

* bugfix

* align with pre-commit

* fix link format in docs

* update integration test

* hpo_config only gets HpoConfig

* skip padim hpo

* add api to set metric name for HPO

* enable test_otx_hpo test

* align with pre-commit

* disable scheduler HPO

* change warnging to raising an error

* fix way to find a dataset size

* update hpo integration test

* update e2e test

* if metric is loss, change hpo model to min

* pass checkpoint args when need to resume
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DOC Improvements or additions to documentation TEST Any changes in tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants