Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove GroundValueConstraint and GroundLinearConstraint #232

Open
eriq-augustine opened this issue Oct 30, 2019 · 0 comments
Open

Remove GroundValueConstraint and GroundLinearConstraint #232

eriq-augustine opened this issue Oct 30, 2019 · 0 comments

Comments

@eriq-augustine
Copy link
Member

Issue by eriq-augustine
Thursday Nov 15, 2018 at 14:28 GMT
Originally opened as eriq-augustine#185


Both of these are just arithmetic rules.
GroundValueConstraint is just RVA = val.
GroundLinearConstraint is just coeff * RVA + coeff * RVA ... <=|=|>= val.

I think the constraint blocker is the only thing that uses them.
We will have to be a bit careful on the semantics:

  • This would be a ground rule without an associated rule (unless we create a dummy).
  • There are both special cases of arithmetic rules.

Maybe instead of removal, we can just treat them as special cases (subclasses).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant