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

[dt] #272 rebased #274

Merged
merged 6 commits into from
Oct 24, 2024
Merged

[dt] #272 rebased #274

merged 6 commits into from
Oct 24, 2024

Conversation

FissoreD
Copy link
Collaborator

@FissoreD FissoreD commented Oct 23, 2024

This solves bug from issue #272 (parentheses forgotten in the code)
Moreover, it reduces the size of paths: for each predicate we store the size of the longest list in its rules and use this length to prevent lists in the goal to be longer then this bound

Copy link
Contributor

@gares gares left a comment

Choose a reason for hiding this comment

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

A comment describing the difference between listTailVar and listTailVarUnif would help, otherwise it looks fine to me.

@gares
Copy link
Contributor

gares commented Oct 24, 2024

Is there a known issue or it can be merged?

@FissoreD FissoreD changed the title Dt 272 rebased [dt] #272 rebased Oct 24, 2024
@FissoreD
Copy link
Collaborator Author

FissoreD commented Oct 24, 2024

I think everything is OK, I just want to add the comment to listTailVarUnif

@gares gares marked this pull request as ready for review October 24, 2024 07:12
@gares gares merged commit 9419bc8 into LPCIC:master Oct 24, 2024
8 checks passed
@FissoreD FissoreD deleted the dt_272_rebased branch October 25, 2024 09:08
Sign up for free to 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