feat(Combinatorics/SimpleGraph/Connectivity): add edge reachability and connectivity numbers - #42494
feat(Combinatorics/SimpleGraph/Connectivity): add edge reachability and connectivity numbers#42494JadAbouHawili wants to merge 76 commits into
Conversation
…nd connectivity numbers
Welcome new contributor!Thank you for contributing to Mathlib! If you haven't done so already, please review our contribution guidelines, as well as the style guide and naming conventions. In particular, we kindly remind contributors that we have guidelines regarding the use of AI when making pull requests. We use a review queue to manage reviews. If your PR does not appear there, it is probably because it is not successfully building (i.e., it doesn't have a green checkmark), has the If you haven't already done so, please come to https://leanprover.zulipchat.com/, introduce yourself, and mention your new PR. Thank you again for joining our community. |
PR summary aa9e05e6db
|
| File | Base Count | Head Count | Change |
|---|---|---|---|
| Mathlib.Combinatorics.SimpleGraph.Connectivity.EdgeConnectivity | 799 | 811 | +12 (+1.50%) |
Import changes for all files
| Files | Import difference |
|---|---|
3 filesMathlib.Combinatorics.SimpleGraph.Acyclic Mathlib.Combinatorics.SimpleGraph.Girth Mathlib.Combinatorics.SimpleGraph.Star |
1 |
Mathlib.Combinatorics.SimpleGraph.Connectivity.EdgeConnectivity Mathlib.Combinatorics.SimpleGraph.Hamiltonian |
12 |
Declarations diff (regex)
+ IsEdgeConnected.le_edgeConnectivity
+ IsEdgeConnected.le_minDegree
+ IsEdgeReachable.le_edgeReachability
+ Reachable.edgeReachability_ne_zero
+ edgeConnectivity
+ edgeConnectivity_eq_top_of_subsingleton
+ edgeConnectivity_le_edgeReachability
+ edgeConnectivity_le_minDegree
+ edgeReachability
+ edgeReachability_comm
+ edgeReachability_eq_top_of_subsingleton
+ edgeReachability_le_degree_left
+ edgeReachability_le_degree_right
+ edgeReachability_self
You can run this locally as follows
## from your `mathlib4` directory:
git clone https://github.com/leanprover-community/mathlib-ci.git ../mathlib-ci
## summary with just the declaration names:
../mathlib-ci/scripts/pr_summary/declarations_diff.sh <optional_commit>
## more verbose report:
../mathlib-ci/scripts/pr_summary/declarations_diff.sh long <optional_commit>The doc-module for scripts/pr_summary/declarations_diff.sh in the mathlib-ci repository contains some details about this script.
Declarations diff (Lean)
✅ Lean-aware diff — post-build, computed from the Lean environment (commit
aa9e05e).
- +14 new declarations
- −0 removed declarations
+SimpleGraph.IsEdgeConnected.le_edgeConnectivity
+SimpleGraph.IsEdgeConnected.le_minDegree
+SimpleGraph.IsEdgeReachable.le_edgeReachability
+SimpleGraph.Reachable.edgeReachability_ne_zero
+SimpleGraph.edgeConnectivity
+SimpleGraph.edgeConnectivity_eq_top_of_subsingleton
+SimpleGraph.edgeConnectivity_le_edgeReachability
+SimpleGraph.edgeConnectivity_le_minDegree
+SimpleGraph.edgeReachability
+SimpleGraph.edgeReachability_comm
+SimpleGraph.edgeReachability_eq_top_of_subsingleton
+SimpleGraph.edgeReachability_le_degree_left
+SimpleGraph.edgeReachability_le_degree_right
+SimpleGraph.edgeReachability_selfNo changes to strong technical debt.
No changes to weak technical debt.
Current commit aa9e05e6db
Reference commit a7cadaedd7
This script lives in the mathlib-ci repository. To run it locally, from your mathlib4 directory:
git clone https://github.com/leanprover-community/mathlib-ci.git ../mathlib-ci
../mathlib-ci/scripts/reporting/technical-debt-metrics.sh pr_summary
- The
relativevalue is the weighted sum of the differences with weight given by the inverse of the current value of the statistic. - The
absolutevalue is therelativevalue divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).
|
Welcome! Could you disclose if and how you're using LLMs, per the AI guidelines? Also this PR contains a bunch of unrelated changes to docstrings; could you remove them or move them into separate PRs? |
|
No LLMS used , I'll move the docstring changes to separate PRs |
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
At first I thought it might cause merge conflicts with this PR but that's not the case. So i would change
sounds like another dependent PR I'll work on. Regarding the work in this PR towards resolving #34961 , I've proved all the things I could handle and don't have anything to add. I wasn't able to figure out Regardless , I'm going to move onto working on the dependent PRs. Thanks for the reviews , I learned alot and hopefully future contributions will go smoother and with lesser mistakes. future work: |
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
…y.lean Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
By the way: I think such an explanation is great, and I would like to strongly encourage you to write this up somewhere. (Even "anywhere reasonable" and then linking to it from the docs overview would be an improvement.) |
|
Thanks! I've posted on #mathlib4 > Mathlib: the Missing Manuals @ 💬 and hopefully things will take off from there |
Define edge reachability and connectivity numbers with some basic API
Partially solves #34961