-
Notifications
You must be signed in to change notification settings - Fork 251
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
[Merged by Bors] - feat: port Geometry.Manifold.ContMdiffMfderiv #5496
Conversation
urkud
commented
Jun 26, 2023
•
edited
Loading
edited
- depends on: [Merged by Bors] - feat: port Geometry.Manifold.Mfderiv #5454
This PR/issue depends on:
|
3d53acf
to
9322f70
Compare
f6ff7fa
to
a132471
Compare
Mathbin -> Mathlib fix certain import statements move "by" to end of line add import to Mathlib.lean
a132471
to
441ebfb
Compare
(extChartAt I (g x₀) (g x))) | ||
x₀ := by |
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.
Very minor, but I was confused by the syntax here. Perhaps it would be more readable with x₀
at the end of the previous line?
(range I) (extChartAt I (g x₀) (g x))) | ||
x₀ := by |
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.
ditto here
(fun p : H × E => | ||
(f p.fst, | ||
(fderivWithin 𝕜 (writtenInExtChartAt I I' p.fst f) (I.symm ⁻¹' s ∩ range I) | ||
((extChartAt I p.fst) p.fst) : | ||
E →L[𝕜] E') | ||
p.snd)) | ||
(Prod.fst ⁻¹' s) |
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.
Excessive newlines?
bors d+ |
✌️ urkud can now approve this pull request. To approve and merge a pull request, simply reply with |
Co-authored-by: Scott Morrison <scott@tqft.net>
Co-authored-by: Scott Morrison <scott@tqft.net>
bors merge |
Pull request successfully merged into master. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |