Ax
Ax copied to clipboard
Allow parameter constraints that involve `FixedParameter`s
trafficstars
Summary: Currently, we error out in validate_constraint_parameters if the list of parameters invovles at least one FixedParameter. That doensn't make a lot of sense, we should allow constraints as long as not ALL parameters are FixedParameters.
Differential Revision: D75958304
This pull request was exported from Phabricator. Differential Revision: D75958304
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.88%. Comparing base (
8b01aef) to head (83b407e).
Additional details and impacted files
@@ Coverage Diff @@
## main #3868 +/- ##
=======================================
Coverage 95.88% 95.88%
=======================================
Files 556 556
Lines 54987 54995 +8
=======================================
+ Hits 52725 52733 +8
Misses 2262 2262
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.