Skip to content

add ObjectiveAsConstraint Transform#4906

Closed
sdaulton wants to merge 1 commit intofacebook:mainfrom
sdaulton:export-D93118131
Closed

add ObjectiveAsConstraint Transform#4906
sdaulton wants to merge 1 commit intofacebook:mainfrom
sdaulton:export-D93118131

Conversation

@sdaulton
Copy link
Contributor

Summary:
In the case of no feasible points and when there is a status_quo, this transform will add a constraint using the objective metric where the bound is the SQ value (equivalent to a >=0% relative constraint).

This is important for using p(feasible), since it will make sure we do not regress the objective metric when looking for feasible points w.r.t. the other constraints.

Differential Revision: D93118131

@meta-codesync
Copy link

meta-codesync bot commented Feb 12, 2026

@sdaulton has exported this pull request. If you are a Meta employee, you can view the originating Diff in D93118131.

@meta-cla meta-cla bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Feb 12, 2026
@codecov-commenter
Copy link

codecov-commenter commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 97.65625% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.78%. Comparing base (44c7c38) to head (a63b12c).

Files with missing lines Patch % Lines
ax/adapter/transforms/objective_as_constraint.py 92.43% 9 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##             main    #4906    +/-   ##
========================================
  Coverage   96.78%   96.78%            
========================================
  Files         593      595     +2     
  Lines       62303    62687   +384     
========================================
+ Hits        60297    60673   +376     
- Misses       2006     2014     +8     

☔ 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.

sdaulton added a commit to sdaulton/Ax-1 that referenced this pull request Feb 13, 2026
Summary:

In the case of no feasible points and when there is a status_quo, this transform will add a constraint using the objective metric where the bound is the SQ value (equivalent to a >=0% relative constraint).

This is important for using p(feasible), since it will make sure we do not regress the objective metric when looking for feasible points w.r.t. the other constraints.

Reviewed By: saitcakmak

Differential Revision: D93118131
Summary:

In the case of no feasible points and when there is a status_quo, this transform will add a constraint using the objective metric where the bound is the SQ value (equivalent to a >=0% relative constraint).

This is important for using p(feasible), since it will make sure we do not regress the objective metric when looking for feasible points w.r.t. the other constraints.

Reviewed By: saitcakmak

Differential Revision: D93118131
@meta-codesync
Copy link

meta-codesync bot commented Feb 13, 2026

This pull request has been merged in ca68b28.

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 meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants