-
Notifications
You must be signed in to change notification settings - Fork 1.2k
perf(Analysis/Normed/Group): lower instance priorities #37962
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
base: master
Are you sure you want to change the base?
Changes from all commits
aec0f19
0e0c658
7121b64
588a6c1
877ba14
9dcb80c
53446cc
268a6a5
3a213a7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,6 +8,7 @@ module | |
| public import Mathlib.Analysis.Complex.Circle | ||
| public import Mathlib.LinearAlgebra.Determinant | ||
| public import Mathlib.LinearAlgebra.Matrix.GeneralLinearGroup.Basic | ||
| import Mathlib.Tactic.SuppressCompilation | ||
|
|
||
| /-! | ||
| # Isometries of the Complex Plane | ||
|
|
@@ -32,6 +33,7 @@ The proof of `linear_isometry_complex_aux` is separated in the following parts: | |
|
|
||
|
|
||
| noncomputable section | ||
| suppress_compilation -- needed to avoid a panic! | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you minimize this? This seems worth reporting! |
||
|
|
||
| open Complex | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -218,12 +218,13 @@ lemma prodMk [Finite ι] (hX : HasGaussianLaw (fun ω ↦ (X · ω)) P) (i j : | |
| letI := Fintype.ofFinite ι | ||
| hX.map (.prod (.proj i) (.proj j)) | ||
|
|
||
| variable [Fintype ι] | ||
|
|
||
| lemma toLp_pi (p : ℝ≥0∞) [Fact (1 ≤ p)] (hX : HasGaussianLaw (fun ω ↦ (X · ω)) P) : | ||
| lemma toLp_pi [Finite ι] (p : ℝ≥0∞) [Fact (1 ≤ p)] (hX : HasGaussianLaw (fun ω ↦ (X · ω)) P) : | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for the fix. I gather this is linter with the lower priority, but not now --- do you know why/ is there a deeper problem we should fix?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similarly to the other change, the linter now recognized that some type class assumption was unused. This is because the synthesis order changed, so a different instance was found. This does not necessarily mean that something is wrong, though it is a bit counter intuitive.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Makes sense, thanks! |
||
| HasGaussianLaw (fun ω ↦ toLp p (X · ω)) P := | ||
| have := Fintype.ofFinite ι | ||
| hX.map_equiv (PiLp.continuousLinearEquiv p ℝ E).symm | ||
|
|
||
| variable [Fintype ι] | ||
|
|
||
| lemma sum {E : Type*} [NormedAddCommGroup E] [NormedSpace ℝ E] [MeasurableSpace E] | ||
| [BorelSpace E] [SecondCountableTopology E] | ||
| {X : ι → Ω → E} (hX : HasGaussianLaw (fun ω ↦ (X · ω)) P) : | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason for this change? (I like it regardless; if it compiles on its own, I'm happy to splice-bot it.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, a linter suddenly told me that these hypotheses were unused.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
splits-bot
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Um, once more:
splice-bot
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Split PR created
Split off the changes to Mathlib/Analysis/CStarAlgebra/CStarMatrix.lean in #37996.