Ax icon indicating copy to clipboard operation
Ax copied to clipboard

Allow parameter constraints that involve `FixedParameter`s

Open Balandat opened this issue 5 months ago • 2 comments
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

Balandat avatar Jun 04 '25 18:06 Balandat

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

facebook-github-bot avatar Jun 04 '25 18:06 facebook-github-bot

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.

codecov-commenter avatar Jun 04 '25 18:06 codecov-commenter