Skip to content

golf: use compute_degree_le to simplify two proofs#5883

Closed
adomani wants to merge 3 commits intomasterfrom
adomani_cdle_cancelLeads
Closed

golf: use compute_degree_le to simplify two proofs#5883
adomani wants to merge 3 commits intomasterfrom
adomani_cdle_cancelLeads

Conversation

@adomani
Copy link
Copy Markdown
Contributor

@adomani adomani commented Jul 14, 2023

This PR simply uses the tactic compute_degree_le from #5882 to remove golf two proofs.

One of the two proofs was the only mathlib3 use of compute_degree_le and the new tactic removes a porting note.


Open in Gitpod

@kim-em kim-em added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jul 14, 2023
@adomani adomani mentioned this pull request Jul 14, 2023
1 task
@adomani adomani changed the title golf: use compute_degree_le to restore a mathlib3 proof golf: use compute_degree_le to simplify two proofs Jul 14, 2023
@ghost ghost added merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) and removed merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) labels Jul 19, 2023
@ghost ghost removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jul 28, 2023
@ghost ghost deleted a comment from kim-em Jul 28, 2023
@ghost
Copy link
Copy Markdown

ghost commented Jul 28, 2023

This PR/issue depends on:

adomani added 2 commits July 28, 2023 19:20
Add docs, remove tests from Tactic file.

Improve docs in tests.

Add degree disclaimer

Fix issue with 1.

Add pre-existing ml3 tests.

Use compute_degree_le, remove porting note

Add debug-mode and a few explicit terms

Parse optional `!`.

Use MVarIds

Remove unneeded deg-->natDeg lemma

Update Mathlib/Tactic/ComputeDegree.lean

Update Mathlib/Tactic/ComputeDegree.lean

Update Mathlib/Tactic/ComputeDegree.lean

Update Mathlib/Tactic/ComputeDegree.lean

Update Mathlib/Tactic/ComputeDegree.lean

Update Mathlib/Tactic/ComputeDegree.lean

Update Mathlib/Tactic/ComputeDegree.lean

Extract `getPolsName` from `cDegCore`

Adjust `open`s and `section`

Fix typo

Error golf

Cosmetic changes

Golf?

Symmetric support for natDegree and degree

Trivial lemmas for compute_degree_le

Restore newer "all" files

Rename lemmas

Section PRed lemmas

Partial introduction of DegInfo

More tests with Ring

Streamline and golf from cdle_dev

Add repeated `OfNat` test
@adomani adomani force-pushed the adomani_cdle_cancelLeads branch from a0de17f to bc068bc Compare July 28, 2023 17:24
@adomani
Copy link
Copy Markdown
Contributor Author

adomani commented Jul 29, 2023

This PR was a sample usage of a tactic compute_degree_le in a PR that is now closed.

@adomani adomani closed this Jul 29, 2023
@adomani adomani deleted the adomani_cdle_cancelLeads 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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants