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

[Merged by Bors] - chore: review of automation in category theory #4793

Closed
wants to merge 4 commits into from

Conversation

semorrison
Copy link
Contributor

Clean up of automation in the category theory library. Leaving out unnecessary proof steps, or fields done by aesop_cat, and making more use of available autoparameters.


Open in Gitpod

@semorrison semorrison added awaiting-review The author would like community review of the PR awaiting-CI labels Jun 7, 2023
@jcommelin
Copy link
Member

This currently doesn't build. Once it builds, I would be interested in running a benchmark to see how much of a difference this use of automation makes. (Hopefully very little!)

@semorrison
Copy link
Contributor Author

!bench

@semorrison semorrison added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Jun 8, 2023
restore a proof
@semorrison semorrison removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Jun 8, 2023
@semorrison
Copy link
Contributor Author

!bench

@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit ae8d859.
There were significant changes against commit 28f3fa3:

  Benchmark                                                     Metric         Change
  ===================================================================================
- build                                                         aesop            6.3%
- ~Mathlib.Algebra.Category.GroupCat.EquivalenceGroupAddGroup   instructions     8.8%
- ~Mathlib.Algebra.Homology.Opposite                            instructions     6.9%
- ~Mathlib.CategoryTheory.Adjunction.Limits                     instructions    17.4%
- ~Mathlib.CategoryTheory.Adjunction.Opposites                  instructions     7.2%
- ~Mathlib.CategoryTheory.Bicategory.End                        instructions    32.0%
- ~Mathlib.CategoryTheory.Category.Bipointed                    instructions     9.3%
- ~Mathlib.CategoryTheory.Category.Pairwise                     instructions    42.6%
- ~Mathlib.CategoryTheory.Category.TwoP                         instructions     6.7%
- ~Mathlib.CategoryTheory.Endofunctor.Algebra                   instructions    43.2%
- ~Mathlib.CategoryTheory.FinCategory                           instructions    12.1%
- ~Mathlib.CategoryTheory.Functor.Hom                           instructions    16.6%
- ~Mathlib.CategoryTheory.IsConnected                           instructions    12.8%
- ~Mathlib.CategoryTheory.Limits.Cones                          instructions     6.0%
- ~Mathlib.CategoryTheory.Limits.HasLimits                      instructions    10.5%
- ~Mathlib.CategoryTheory.Limits.Shapes.Multiequalizer          instructions     9.2%
- ~Mathlib.CategoryTheory.Monad.Algebra                         instructions     6.2%
- ~Mathlib.CategoryTheory.Monoidal.FunctorCategory              instructions    31.6%
- ~Mathlib.CategoryTheory.Over                                  instructions    21.8%
- ~Mathlib.CategoryTheory.PEmpty                                instructions    17.5%
- ~Mathlib.CategoryTheory.PUnit                                 instructions     9.3%
- ~Mathlib.CategoryTheory.Sites.Plus                            instructions     7.5%
- ~Mathlib.CategoryTheory.Sites.Pushforward                     instructions    16.7%
- ~Mathlib.CategoryTheory.Sites.Sheaf                           instructions    12.3%
+ ~Mathlib.CategoryTheory.Subterminal                           instructions   -16.4%
- ~Mathlib.CategoryTheory.Triangulated.Triangulated             instructions    14.5%
- ~Mathlib.Order.Category.DistLatCat                            instructions    15.6%
- ~Mathlib.Order.Category.FinPartOrd                            instructions    12.9%
- ~Mathlib.Order.Category.LatCat                                instructions    11.1%
- ~Mathlib.Order.Category.LinOrdCat                             instructions    15.1%
- ~Mathlib.Order.Category.NonemptyFinLinOrdCat                  instructions     7.2%
- ~Mathlib.Order.Category.PartOrdCat                            instructions    14.1%
- ~Mathlib.Order.Category.PreordCat                             instructions    13.7%
- ~Mathlib.Topology.Sheaves.LocalPredicate                      instructions     9.6%

@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit d743f11.
The entire run failed.
Found no runs to compare against.

@semorrison
Copy link
Contributor Author

bench suggests an increase of 0.75% in wall-clock.

@jcommelin
Copy link
Member

Thanks 🎉

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Jun 9, 2023
bors bot pushed a commit that referenced this pull request Jun 9, 2023
Clean up of automation in the category theory library. Leaving out unnecessary proof steps, or fields done by `aesop_cat`, and making more use of available autoparameters.



Co-authored-by: Scott Morrison <scott.morrison@anu.edu.au>
@bors
Copy link

bors bot commented Jun 9, 2023

Build failed:

@jcommelin
Copy link
Member

bors d+

@bors
Copy link

bors bot commented Jun 9, 2023

✌️ semorrison can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@semorrison
Copy link
Contributor Author

bors merge

bors bot pushed a commit that referenced this pull request Jun 9, 2023
Clean up of automation in the category theory library. Leaving out unnecessary proof steps, or fields done by `aesop_cat`, and making more use of available autoparameters.



Co-authored-by: Scott Morrison <scott.morrison@anu.edu.au>
@bors
Copy link

bors bot commented Jun 9, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title chore: review of automation in category theory [Merged by Bors] - chore: review of automation in category theory Jun 9, 2023
@bors bors bot closed this Jun 9, 2023
@bors bors bot deleted the review_automation branch June 9, 2023 07:57
qawbecrdtey pushed a commit to qawbecrdtey/greedoid-mathlib4 that referenced this pull request Jun 12, 2023
…4793)

Clean up of automation in the category theory library. Leaving out unnecessary proof steps, or fields done by `aesop_cat`, and making more use of available autoparameters.



Co-authored-by: Scott Morrison <scott.morrison@anu.edu.au>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants