-
Notifications
You must be signed in to change notification settings - Fork 36
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
Wrong result with absento #2
Comments
Is this intentional? The readme mentions something about |
The original implementation of Michael, would it be difficult to support the full general behavior of Thanks! |
I just submitted my first ever pull request! :) This patch checks that the first argument to Handling generalized |
@webyrd Is there an example of a tricky case you're thinking about? I'm not sure it's too bad, except perhaps for reification concerns that would be handled by the old reifier this implementation inherits.
Applying Generalized absento still doesn't allow a constraint to restrict the range of values for a variable to a finite range, so it shouldn't break any of the assumptions that make the attributed variable implementation safe. |
P.S.
If I rewrite it to use my own "not-membero" relation instead of absento, it produces the correct '() result.
P.P.S
webyrd/miniKenren-with-symbolic-constraints seems to have the correct behavior on this example.
The text was updated successfully, but these errors were encountered: