feat(SpaceTime): the spacetime algebra and its Taylor series - #1702
Conversation
|
Thank you for this pull-request (PR). If this is your first PR, welcome to the community! Below is what will happen next. Please read carefully if you are not familiar with the process. You may open other PRs while this one is being reviewed, and can stack PRs on top of each other, so don't let these steps slow you down.
Tip: The easiest way to get have a fast review is to submit a PR that is small and self-contained, and has clear documentation explaining why things are the way they are in your chages. If you have any problems or questions, please reach out to the community on the Zulip. |
| ring | ||
|
|
||
| /-- The factorial `s!` is nonzero. -/ | ||
| lemma prod_factorial_ne_zero (s : Multiset (Fin 1 ⊕ Fin 3)) : |
There was a problem hiding this comment.
This seems like a rewrite of a Mathlib proof, and we should likely just include it directly in lemmas rather then making it seperate here
| -/ | ||
|
|
||
| /-- The series whose base-point derivative values are `F`. -/ | ||
| noncomputable def taylorSeries (F : Multiset (Fin 1 ⊕ Fin 3) → ℂ) : SpaceTimeAlgebra := |
There was a problem hiding this comment.
If possible, I think we should define taylorSeries as a map from a smooth function SpaceTime → ℂ and give this another name.
|
awaiting-author |
|
I've renamed the derivative-values construction to -awaiting-author |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
| open scoped ContDiff | ||
|
|
||
| /-- The `ℂ`-subalgebra of smooth complex-valued functions on `SpaceTime d`. -/ | ||
| def smoothFunctions (d : ℕ := 3) : Subalgebra ℂ (SpaceTime d → ℂ) where |
There was a problem hiding this comment.
I wouldn't define smooth functions here.
| -/ | ||
|
|
||
| /-- The Taylor series at `x₀` of a smooth complex-valued function on spacetime. -/ | ||
| noncomputable def taylorSeries (x₀ : SpaceTime) : smoothFunctions →ₗ[ℂ] SpaceTimeAlgebra where |
There was a problem hiding this comment.
I think to prevent the need for smooth functions (unless they are already bundled as a type in Mathlib), we should define this function for all functions. And then show that on smooth functions it has the desired properties.
|
I'm wondering what is the best way to move forward with this @jstoobysmith There is a Mathlib type C^∞⟮…⟯, but for ℂ-valued functions on spacetime it's only an ℝ-module and not a ring
I'm leaning towards options 1 but would like your opinion on this -awaiting-author |
|
I also think 1 makes more sense here. It ends up being easier in the long run I think. awaiting-author |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Have made this change -awaiting-author |
There was a problem hiding this comment.
If we can get away without making these definitions or making them general properties of derivatives on SpaceTime I think this would be better.
There was a problem hiding this comment.
have addressed, this was honestly quite dim from my side. I got far too excited to add something to the new mathlib folder
|
awaiting-author |
|
-awaiting-author |
jstoobysmith
left a comment
There was a problem hiding this comment.
Approved. Will merge shortly.
70e4037
Adds the spacetime algebra, formal power series in the four spacetime directions. Compared with
#1415, the repeated
foldlis now the definitioniteratedPDeriv, and Taylor's formula andtaylorEquivare new. The remaining lemmas are carried over from #1415, renamed to useiteratedPDeriv. I would appreciate a careful review of the documentation as well. I used AI tohelp with this PR.