Skip to content

Conversation

@saitcakmak
Copy link
Contributor

Summary:
Looking at any tutorial with early stopping (ex: https://ax.dev/docs/next/tutorials/sklearn/) shows a ton of these warnings: [WARNING 08-06 05:15:48] ax.early_stopping.strategies.base: No metric names specified. Defaulting to the objective metric(s).

The docstring specifies this to be the behavior when metrics are not explicitly specifies. This warning does not need to be this intrusive, we can reduce it to debug level.

Differential Revision: D79740162

Summary:
Looking at any tutorial with early stopping (ex: https://ax.dev/docs/next/tutorials/sklearn/) shows a ton of these warnings: `[WARNING 08-06 05:15:48] ax.early_stopping.strategies.base: No metric names specified. Defaulting to the objective metric(s).`

The docstring specifies this to be the behavior when metrics are not explicitly specifies. This warning does not need to be this intrusive, we can reduce it to debug level.

Differential Revision: D79740162
@meta-cla meta-cla bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Aug 6, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D79740162

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.07%. Comparing base (53c4a2e) to head (bca573a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4108   +/-   ##
=======================================
  Coverage   96.07%   96.07%           
=======================================
  Files         563      563           
  Lines       56980    56980           
=======================================
  Hits        54743    54743           
  Misses       2237     2237           

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in b2f85d4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants