Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix a wrong HPO log #3972

Merged
merged 2 commits into from
Sep 23, 2024
Merged

Fix a wrong HPO log #3972

merged 2 commits into from
Sep 23, 2024

Conversation

eunwoosh
Copy link
Contributor

Summary

This PR fixes wrong HPO log.

How to test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have ran e2e tests and there is no issues.
  • 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

Copy link

codecov bot commented Sep 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.68%. Comparing base (40f5ddd) to head (3e4ae69).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3972      +/-   ##
===========================================
- Coverage    81.69%   81.68%   -0.01%     
===========================================
  Files          289      289              
  Lines        27916    27916              
===========================================
- Hits         22805    22804       -1     
- Misses        5111     5112       +1     
Flag Coverage Δ
py310 81.50% <100.00%> (ø)
py311 81.67% <100.00%> (+0.19%) ⬆️

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 enabled auto-merge September 23, 2024 07:04
@eunwoosh eunwoosh added this pull request to the merge queue Sep 23, 2024
Merged via the queue into develop with commit 0e2ab6c Sep 23, 2024
22 checks passed
@eunwoosh eunwoosh deleted the fix_wrong_hpo_log branch September 23, 2024 09:20
eugene123tw added a commit to eugene123tw/training_extensions that referenced this pull request Sep 26, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants