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.

Sign up for GitHub

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

Jump to bottom

live adjustment of bound values in AxSweep experiments #1627

Closed
wants to merge 1 commit into from
Closed

live adjustment of bound values in AxSweep experiments #1627

wants to merge 1 commit into from

Conversation

bernardbeckerman
Copy link
Contributor

Summary: Allows users to adjust OutcomeConstraint or ObjectiveThreshold bound values during execution of an AxSweep, e.g., to shift the focus the sweep to a new part of the Pareto curve.

Reviewed By: zcohn, lena-kashtelyan

Differential Revision: D46090213

@facebook-github-bot facebook-github-bot added CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported labels May 23, 2023
@facebook-github-bot
Copy link
Contributor

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

@codecov-commenter
Copy link

codecov-commenter commented May 23, 2023 •
edited
Loading

Codecov Report

Patch coverage: 25.00% and project coverage change: -0.03 ⚠️

Comparison is base (8ec6f68) 94.81% compared to head (cb9726b) 94.79%.

❗ Current head cb9726b differs from pull request most recent head c0a9201. Consider uploading reports for the commit c0a9201 to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #1627      +/-   ##
==========================================
- Coverage   94.81%   94.79%   -0.03%     
==========================================
  Files         442      442              
  Lines       40895    40911      +16     
==========================================
+ Hits        38776    38780       +4     
- Misses       2119     2131      +12
Impacted Files Coverage Δ
ax/storage/sqa_store/save.py 90.21% <25.00%> (-6.22%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

bernardbeckerman pushed a commit to bernardbeckerman/Ax that referenced this pull request May 24, 2023
@facebook-github-bot
live adjustment of bound values in AxSweep experiments (facebook#1627)
e26fd93 
Summary:
Pull Request resolved: facebook#1627

Allows users to adjust OutcomeConstraint or ObjectiveThreshold bound values during execution of an AxSweep, e.g., to shift the focus the sweep to a new part of the Pareto curve.

Reviewed By: zcohn, lena-kashtelyan

Differential Revision: D46090213

fbshipit-source-id: f3d2260156dbd3e7c1e5e17f654540004851805c
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
live adjustment of bound values in AxSweep experiments (facebook#1627)
c0a9201 
Summary:
Pull Request resolved: facebook#1627

Allows users to adjust OutcomeConstraint or ObjectiveThreshold bound values during execution of an AxSweep, e.g., to shift the focus the sweep to a new part of the Pareto curve.

Reviewed By: zcohn, lena-kashtelyan

Differential Revision: D46090213

fbshipit-source-id: 353dd21c12af4ec6021fd5e398b0abcc9cb7a3b4
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e41736b.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported Merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bernardbeckerman @facebook-github-bot @codecov-commenter