Skip to content

Conversation

@fusawa-yugo
Copy link
Contributor

Motivation

Description of the changes

I integrated recipes in optunahub-registry into optunahub as a tutorial.

@codecov
Copy link

codecov bot commented Mar 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.75%. Comparing base (8b9778f) to head (43264ae).
Report is 11 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #91   +/-   ##
=======================================
  Coverage   88.75%   88.75%           
=======================================
  Files           9        9           
  Lines         169      169           
=======================================
  Hits          150      150           
  Misses         19       19           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

@nabenabe0928 nabenabe0928 left a comment

Choose a reason for hiding this comment

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

I left some minor comments:)

Copy link
Contributor

Choose a reason for hiding this comment

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

@nabenabe0928
Copy link
Contributor

nabenabe0928 commented Apr 4, 2025

TOOD

  • Inspect URL expirations

@y0z y0z added the document Documentation related. label Apr 7, 2025
@y0z y0z closed this Apr 16, 2025
@y0z y0z reopened this Apr 16, 2025
@fusawa-yugo
Copy link
Contributor Author

Thank you for review!
I changed the destination URL in optunahub.
After this PR is merged, I will fix the URL in other repositories.

@nabenabe0928
Copy link
Contributor

@fusawa-yugo
Hey, really sorry for the late response!
Could you please resolve the conflicts?

@nabenabe0928
Copy link
Contributor

Almost LGTM:)

@fusawa-yugo
Copy link
Contributor Author

Thank you!!
I fixed it.
Could you review it again?

Copy link
Contributor

@nabenabe0928 nabenabe0928 left a comment

Choose a reason for hiding this comment

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

Thank you for the updates, LGTM!

@nabenabe0928 nabenabe0928 merged commit a8fdb7c into optuna:main Apr 24, 2025
13 checks passed
@nabenabe0928 nabenabe0928 added this to the v0.3.0 milestone Apr 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

document Documentation related.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants