Skip to content

feat({Tactic + test}/ComputeDegree): add tactic compute_degree_le#5276

Closed
adomani wants to merge 11 commits intomasterfrom
adomani_compute_degree_le
Closed

feat({Tactic + test}/ComputeDegree): add tactic compute_degree_le#5276
adomani wants to merge 11 commits intomasterfrom
adomani_compute_degree_le

Conversation

@adomani
Copy link
Copy Markdown
Contributor

@adomani adomani commented Jun 20, 2023

This PR is a port of the mathlib3 tactic with the same name. It lacks some of the functionality of the original tactic, since it no longer even attempts to deal with exponents that are not closed terms of type Nat. Other than that, it passes all the remaining tests for mathlib3's compute_degree_le.


Open in Gitpod

@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 Jun 20, 2023
@adomani adomani added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Jul 12, 2023
@adomani adomani mentioned this pull request Jul 14, 2023
1 task
@adomani
Copy link
Copy Markdown
Contributor Author

adomani commented Jul 14, 2023

#5882 re-ports this tactic, so I am closing this PR.

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

Labels

awaiting-author A reviewer has asked the author a question or requested changes. 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