Skip to content

Support NumericalAttribute intervals in dpsynth.constraints.Constraint: - #226

Closed
copybara-service[bot] wants to merge 1 commit into
mainfrom
cl/992417631
Closed

copybara-service[bot] wants to merge 1 commit into
mainfrom
cl/992417631

Conversation

@copybara-service

@copybara-service copybara-service Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Support NumericalAttribute intervals in dpsynth.constraints.Constraint:

  • Extend Constraint to accept (low, high) interval tuples and None (for clip_to_range=False) on NumericalAttribute columns with fixed bin_edges in possible_combinations and impossible_combinations.
  • Validate at configuration time that NumericalAttribute domains specify bin_edges and that interval endpoints match attr.bin_edges or [min_value, max_value].
  • Discretize numerical intervals in Constraint.to_mbi() using attr.bin_edges before delegating to the categorical to_mbi() path.

@copybara-service
copybara-service Bot force-pushed the cl/992417631 branch 2 times, most recently from f0f1ce6 to 7a42946 Compare October 2, 2026 21:35
@copybara-service copybara-service Bot changed the title Support NumericalAttribute intervals and out-of-domain values in dpsynth.constraints.Constraint: Support NumericalAttribute intervals in dpsynth.constraints.Constraint: Oct 2, 2026
@copybara-service
copybara-service Bot force-pushed the cl/992417631 branch 4 times, most recently from b3d98e9 to 60ca6af Compare October 4, 2026 01:16
- Extend Constraint to accept (low, high) interval tuples and None (for clip_to_range=False) on NumericalAttribute columns with fixed bin_edges in possible_combinations and impossible_combinations.
- Validate at configuration time that NumericalAttribute domains specify bin_edges and that interval endpoints match attr.bin_edges or [min_value, max_value].
- Discretize numerical intervals in Constraint.to_mbi() using attr.bin_edges before delegating to the categorical to_mbi() path.

PiperOrigin-RevId: 992417631
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant