Skip to content

Predicate processing - #1328

Closed
mmatera wants to merge 9 commits into
masterfrom
predicate-processing
Closed

Predicate processing#1328
mmatera wants to merge 9 commits into
masterfrom
predicate-processing

Conversation

@mmatera

Copy link
Copy Markdown
Contributor

This is the part of ConditionalExpressions that involves advanced logical processing.

new inference module. Many Improvings on inference
Comment threadmathics/builtin/inference.py Outdated
evaluation.message("Inconsistent assumptions")

if assumptions_list is None:
return remove_nots_when_unnecesary(pred, evaluation).evaluate(evaluation)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pred is not defined here.

Comment threadmathics/builtin/inference.py Outdated
assumption_rules.append(Rule(pat, symbol_value))
# TODO: expand the pred and assumptions into an standard,
# atomized form, and then apply the rules...
print("assumptions:")

@rockyrockyMay 1, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this print and the one below supposed to be here?

new inference module. Many Improvings on inference
@rocky
rocky marked this pull request as draft May 1, 2021 11:36
@rocky

rocky commented May 1, 2021

Copy link
Copy Markdown
Member

@mmatera the code here doesn't look like it is ready for a PR. What's up?

@mmatera

Copy link
Copy Markdown
ContributorAuthor

Superseded by #1337

1 similar comment
@mmatera

Copy link
Copy Markdown
ContributorAuthor

Superseded by #1337

@mmaterammatera closed this May 2, 2021
@mmaterammatera mentioned this pull request May 3, 2021
Sign up for freeto 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.

2 participants

@mmatera@rocky