-
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: bounds on alternating convergent series #10120
Conversation
|
||
/-- Partial sums of an alternating antitone series with an even number of terms provide | ||
lower bounds on the limit. -/ | ||
theorem Antitone.alternating_series_le_tendsto (hfa : Antitone f) (k : ℕ) : |
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.
You should be able to reduce that to Monotone.tendsto_le_alternating_series
applied to -f
(and similarly for the next result).
Possibly the intermediate result in the have
would make sense as a separate lemma (in all four versions, but with essentially only two proofs).
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.
I need an extra ContinuousNeg E
assumption to go through that route. Indeed the theorems as their preconditions are set up now work even when E = Rat
.
|
||
|
||
variable {E : Type*} [OrderedRing E] [TopologicalSpace E] [OrderClosedTopology E] | ||
{l : E} {f : ℕ → E} (hfl : Tendsto (fun n ↦ ∑ i in range n, (-1) ^ i * f i) atTop (𝓝 l)) |
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.
Prop assumptions like hfl
should be mentioned in the theorems, not in the variable
line: the reason is that otherwise the file is very hard to understand for the readers, as they don't know which assumptions are used.
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.
Done.
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.
Thanks 🎉
bors merge
Build failed (retrying...): |
Build failed (retrying...): |
Pull request successfully merged into master. Build succeeded: |
https://leanprover.zulipchat.com/#narrow/stream/217875-Is-there-code-for-X.3F/topic/Bounds.20on.20alternating.20convergent.20series/near/418770183