Skip to content

Conversation

@shrutipatel31
Copy link
Contributor

Differential Revision: D82658357

@meta-cla meta-cla bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Sep 24, 2025
@facebook-github-bot
Copy link
Contributor

@shrutipatel31 has exported this pull request. If you are a Meta employee, you can view the originating diff in D82658357.

@codecov-commenter
Copy link

codecov-commenter commented Sep 24, 2025

Codecov Report

❌ Patch coverage is 95.83333% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.56%. Comparing base (12990bb) to head (8761cf7).

Files with missing lines Patch % Lines
ax/core/data.py 50.00% 3 Missing ⚠️
ax/core/experiment.py 90.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4342      +/-   ##
==========================================
- Coverage   96.56%   96.56%   -0.01%     
==========================================
  Files         551      551              
  Lines       56905    56988      +83     
==========================================
+ Hits        54953    55033      +80     
- Misses       1952     1955       +3     

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

shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Sep 25, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@facebook-github-bot
Copy link
Contributor

@shrutipatel31 has exported this pull request. If you are a Meta employee, you can view the originating diff in D82658357.

shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Sep 25, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@facebook-github-bot
Copy link
Contributor

@shrutipatel31 has exported this pull request. If you are a Meta employee, you can view the originating diff in D82658357.

shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Sep 25, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@facebook-github-bot
Copy link
Contributor

@shrutipatel31 has exported this pull request. If you are a Meta employee, you can view the originating diff in D82658357.

shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Sep 25, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@facebook-github-bot
Copy link
Contributor

@shrutipatel31 has exported this pull request. If you are a Meta employee, you can view the originating diff in D82658357.

shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 1, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@facebook-github-bot
Copy link
Contributor

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

shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 6, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@meta-codesync
Copy link

meta-codesync bot commented Oct 6, 2025

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

shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 7, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 7, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 8, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@shrutipatel31 shrutipatel31 force-pushed the export-D82658357 branch 2 times, most recently from 466097b to 286f3f9 Compare October 8, 2025 17:48
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 17, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 17, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@shrutipatel31 shrutipatel31 force-pushed the export-D82658357 branch 2 times, most recently from 6424c83 to af0a072 Compare October 21, 2025 00:07
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 21, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 21, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 28, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
@shrutipatel31 shrutipatel31 force-pushed the export-D82658357 branch 2 times, most recently from 291a6ab to 18379a2 Compare October 28, 2025 15:44
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 28, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 29, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Reviewed By: mpolson64

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 29, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Reviewed By: mpolson64

Differential Revision: D82658357
@shrutipatel31 shrutipatel31 force-pushed the export-D82658357 branch 2 times, most recently from 6fa4abf to 76fb428 Compare October 30, 2025 14:55
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 30, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Reviewed By: mpolson64

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 30, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Reviewed By: mpolson64

Differential Revision: D82658357
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 31, 2025
… a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Reviewed By: mpolson64

Differential Revision: D82658357
@shrutipatel31 shrutipatel31 force-pushed the export-D82658357 branch 2 times, most recently from 242a30f to 51f1e24 Compare October 31, 2025 13:52
shrutipatel31 added a commit to shrutipatel31/Ax that referenced this pull request Oct 31, 2025
…is a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Reviewed By: mpolson64

Differential Revision: D82658357
…is a status quo to relativize against (facebook#4342)

Summary: Pull Request resolved: facebook#4342

Reviewed By: mpolson64

Differential Revision: D82658357
@meta-codesync
Copy link

meta-codesync bot commented Oct 31, 2025

This pull request has been merged in 3ee9711.

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