chore: rename continuous{,On,At,Within}_const to ContinuousFoo.const#31607
chore: rename continuous{,On,At,Within}_const to ContinuousFoo.const#31607grunweg wants to merge 5 commits intoleanprover-community:masterfrom
continuous{,On,At,Within}_const to ContinuousFoo.const#31607Conversation
PR summary 56e100aabdImport changes for modified filesNo significant changes to the import graph Import changes for all files
|
j-loreaux
left a comment
There was a problem hiding this comment.
I don't think any of these are required to be protected, because we don't have const in the root namespace. If you think it's better to do so, that's fine.
bors d+
| ((hff' x hx).const_mul (g b - g a)).sub ((hgg' x hx).const_mul (f b - f a)) | ||
| have hhc : ContinuousOn h (Icc a b) := | ||
| (continuousOn_const.mul hfc).sub (continuousOn_const.mul hgc) | ||
| (ContinuousOn.const.mul hfc).sub (ContinuousOn.const.mul hgc) |
There was a problem hiding this comment.
I'm not necessarily saying you should do this, but I expect this would work:
| (ContinuousOn.const.mul hfc).sub (ContinuousOn.const.mul hgc) | |
| .sub (.mul .const hfc) (.mul .const hgc) |
|
✌️ grunweg can now approve this pull request. To approve and merge a pull request, simply reply with |
continuous{,On,At,Within}_const to ContinuousXXX.const`continuous{,On,At,Within}_const to ContinuousFoo.const`
|
This pull request has conflicts, please merge |
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in #31607.
|
This pull request has conflicts, please merge |
86fc9bd to
8501fa8
Compare
Use fun_prop more, and make indentation match mathlib style better. Extracted from leanprover-community#31607.
Use fun_prop more, and make indentation match mathlib style better. Extracted from #31607.
|
This PR/issue depends on: |
|
This pull request has conflicts, please merge |
continuous{,On,At,Within}_const to ContinuousFoo.const`continuous{,On,At,Within}_const to ContinuousFoo.const
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607.
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607.
Use fun_prop more, and make indentation match mathlib style better. Extracted from leanprover-community#31607.
|
This pull request has conflicts, please merge |
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607.
Use fun_prop more, and make indentation match mathlib style better. Extracted from leanprover-community#31607.
The goal of this PR is to remove many explicit references to continuous(On)_const; to reduce the diff in leanprover-community#31607.
Use fun_prop more, and make indentation match mathlib style better. Extracted from leanprover-community#31607.
|
This pull request has conflicts, please merge |
|
This pull request has conflicts, please merge |
This issue was pre-existing, but has been exposed through leanprover-community#31607 where it caused test failures.
1553504 to
e7a1acf
Compare
|
This pull request has conflicts, please merge |
|
This pull request has conflicts, please merge |
Zulip discussion: #mathlib4 > Naming convention @ 💬