[Merged by Bors] - feat(Combinatorics/SimpleGraphs): generalize SimpleGraph.map to any function#36130
[Merged by Bors] - feat(Combinatorics/SimpleGraphs): generalize SimpleGraph.map to any function#36130vbeffara wants to merge 13 commits intoleanprover-community:masterfrom
Conversation
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. 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 80f4a5fc04Import changes for modified filesNo significant changes to the import graph Import changes for all files
Declarations diff
You can run this locally as follows## summary with just the declaration names:
./scripts/pr_summary/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>The doc-module for No changes to technical debt.You can run this locally as
|
YaelDillies
left a comment
There was a problem hiding this comment.
Looks good!
maintainer merge
|
🚀 Pull request has been placed on the maintainer queue by YaelDillies. |
SnirBroshi
left a comment
There was a problem hiding this comment.
Thanks for doing this!
Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
Co-authored-by: Snir Broshi <26556598+SnirBroshi@users.noreply.github.com>
| { f with map_rel_iff' := by { | ||
| rw [Equiv.coe_toEmbedding] | ||
| grind [Equiv.apply_eq_iff_eq, map_adj', Adj.ne] | ||
| } } |
There was a problem hiding this comment.
| { f with map_rel_iff' := by { | |
| rw [Equiv.coe_toEmbedding] | |
| grind [Equiv.apply_eq_iff_eq, map_adj', Adj.ne] | |
| } } | |
| { f with map_rel_iff' := by aesop (add simp map_adj') } |
|
✌️ vbeffara can now approve this pull request. To approve and merge a pull request, simply reply with |
|
bors r+ |
…unction (#36130) The point is to be able to use map to define graph contractions. Added the injectivity assumption where needed.
|
Pull request successfully merged into master. Build succeeded: |
The point is to be able to use map to define graph contractions. Added the injectivity assumption where needed.