Skip to content

Let @validate check ispolyhedral for constraints(_list)#4012

Draft
schillic wants to merge 2 commits intomasterfrom
schillic/validate_constraints_list
Draft

Let @validate check ispolyhedral for constraints(_list)#4012
schillic wants to merge 2 commits intomasterfrom
schillic/validate_constraints_list

Conversation

@schillic
Copy link
Copy Markdown
Member

@schillic schillic commented Aug 22, 2025

We have constraints_list for Complement, which does not actually require ispolyhedral. So we should either

  • rename that function to something else (my preference because the meaning is different) or
  • not annotate the constraints_list method for Complement (what this PR currently does) or
  • not require ispolyhedral for constraints_list (i.e., not use this PR).

@schillic schillic force-pushed the schillic/validate_constraints_list branch 3 times, most recently from 8167e60 to 562e53b Compare August 22, 2025 11:00
@schillic schillic added the discussion 🗣️ Requires human input label Aug 22, 2025
@schillic schillic force-pushed the schillic/validate_constraints_list branch 2 times, most recently from 46161bd to 50ba53d Compare August 22, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

discussion 🗣️ Requires human input

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant