chore(CategoryTheory/HomologicalComplex): make ChainComplex.of implicit reducible - #43609
chore(CategoryTheory/HomologicalComplex): make ChainComplex.of implicit reducible#43609Thmoas-Guan wants to merge 17 commits into
ChainComplex.of implicit reducible#43609Conversation
PR summary e96e3c6fafImport changes for modified filesNo significant changes to the import graph Import changes for all files
|
| Current number | Change | Type (strong) |
|---|---|---|
| backward.defeqAttrib.useBackward | 4224 | -4 |
| backward.isDefEq.respectTransparency | 4678 | -1 |
Current commit e96e3c6faf
Reference commit 98b840bb29
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.py 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).
may need to rewrite simp lemma
ChainComplex.of implicit reducible
|
!bench |
|
Benchmark results for fa72d0d against 98b840b are in. No significant results found. @Thmoas-Guan
Small changes (1✅, 6🟥)
|
|
!bench |
|
Benchmark results for e96e3c6 against 98b840b are in. No significant results found. @Thmoas-Guan
Small changes (8🟥)
|
| /-- Construct an `α`-indexed chain complex from a dependently-typed differential. | ||
| -/ | ||
| abbrev of (X : α → V) (d : ∀ n, X (n + 1) ⟶ X n) (sq : ∀ n, d (n + 1) ≫ d n = 0) : | ||
| @[implicit_reducible] |
There was a problem hiding this comment.
Then, you may remove the of_X lemma below (which is not doing exactly the same thing, but I do not think it matters):
| @[implicit_reducible] | |
| @[implicit_reducible, simps X] |
| theorem of_X : (of X d sq).X = X := | ||
| rfl | ||
|
|
||
| theorem of_d (j : α) : (of X d sq).d (j + 1) j = d j := by |
There was a problem hiding this comment.
I think of_d should be a simp lemma, but not of_d'. Following the simps X suggestion above, I think it would make sense to replace this with
@[simp]
theorem of_d (j : α) : dsimp% (of X d sq).d (j + 1) j = d j := by|
It would probably be a good idea to do the same for |
This PR makes
ChainComplex.ofimplicit reducible def instaed of previous abbrev.We also removed the auxiliary definition
ChainComplex.of.d, for unification ofsimplemmas for applictions ofChainComplex.of, sincedis no longer simping toChainComplex.of.d.Co-authored-by: @Whysoserioushah