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

Make applyo use constraints #19

Open
brandonwillard opened this issue Jan 1, 2020 · 3 comments
Open

Make applyo use constraints #19

brandonwillard opened this issue Jan 1, 2020 · 3 comments
Labels
CLP Constraint Logic Programming enhancement New feature or request important new goal This issue implies the creation of a new goal (constructor)

Comments

@brandonwillard
Copy link
Member

brandonwillard commented Jan 1, 2020

buildo (soon to be replaced by applyo) will actually apply a rator to its rands and unify the result with its third argument: i.e. buildo(add, (1, 1), q_lv) should unify q_lv with 3.

We need to use the new constraint system to make sure that, for example, add(1, 1) is computed and unified with q_lv once rator_lv and rands_lv are fully ground in a goal like buildo(rator_lv, rands_lv, q_lv).

@brandonwillard brandonwillard added enhancement New feature or request important labels Jan 1, 2020
@brandonwillard brandonwillard added CLP Constraint Logic Programming new goal This issue implies the creation of a new goal (constructor) labels Mar 12, 2020
@brandonwillard
Copy link
Member Author

There are some ambiguities regarding when/how a term should be constructed by the aforementioned constraint, and that leads me to believe that an entirely different goal should be designated to this function, and that applyo should simply cons the operator + arguments (like it currently does on most occasions).

@majidaldo
Copy link
Contributor

is this still relevant since there is applyo?

@brandonwillard
Copy link
Member Author

is this still relevant since there is applyo?

Yes, applyo still doesn't use constraints and—as a result—isn't correct/complete.

@brandonwillard brandonwillard changed the title Better buildo/applyo Make applyo use constraints Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLP Constraint Logic Programming enhancement New feature or request important new goal This issue implies the creation of a new goal (constructor)
Projects
None yet
Development

No branches or pull requests

2 participants