Skip to content

feat: a minimalistic port of compute_degree_le#6007

Closed
adomani wants to merge 4 commits intomasterfrom
adomani_cdle_short
Closed

feat: a minimalistic port of compute_degree_le#6007
adomani wants to merge 4 commits intomasterfrom
adomani_cdle_short

Conversation

@adomani
Copy link
Copy Markdown
Contributor

@adomani adomani commented Jul 19, 2023

This PR is an alternative to #5882. It gives a "minimalistic" port of the compute_degree_le tactic.

This very short tactic works on all tests. The other PR is more structured and is probably further along the direction of a more complete implementation of a compute_degree tactic.

In the long run, I think that the definitions in the other tactic will likely make up the compute_degree tactic, but they are not needed for the easier compute_degree_le tactic.

PR #6221 implements the full compute_degree tactic with a method similar to the one in this PR. I am tempted to close this PR, except for the fact that it might be a possible stepping stone to get to the full implementation.


Open in Gitpod

@adomani adomani changed the title feat: a minimalistic port of compute_degree_leAdd minimal tactic and several tests feat: a minimalistic port of compute_degree_le Jul 19, 2023
@adomani adomani mentioned this pull request Jul 19, 2023
1 task
@adomani adomani added awaiting-review modifies-tactic-syntax This PR adds a new interactive tactic or modifies the syntax of an existing tactic. t-meta Tactics, attributes or user commands labels Jul 19, 2023
@adomani
Copy link
Copy Markdown
Contributor Author

adomani commented Jul 29, 2023

I am closing this PR since #6221 ports more functionality.

@adomani adomani closed this Jul 29, 2023
@adomani adomani deleted the adomani_cdle_short branch January 12, 2024 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifies-tactic-syntax This PR adds a new interactive tactic or modifies the syntax of an existing tactic. t-meta Tactics, attributes or user commands

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant