Skip to content

feat(Regularized): prove to_SupRegularized and of_Subadditive#1022

Merged
jstoobysmith merged 2 commits intoleanprover-community:masterfrom
pitmonticone:aristotle-regularized
May 5, 2026
Merged

feat(Regularized): prove to_SupRegularized and of_Subadditive#1022
jstoobysmith merged 2 commits intoleanprover-community:masterfrom
pitmonticone:aristotle-regularized

Conversation

@pitmonticone
Copy link
Copy Markdown
Member

This PR adds proofs autoformalised by @Aristotle-Harmonic.

Co-authored-by: Aristotle (Harmonic) aristotle-harmonic@harmonic.fun

Co-authored-by: Aristotle (Harmonic) <aristotle-harmonic@harmonic.fun>
Comment on lines +124 to +126
have liminf_neg : Filter.liminf fn Filter.atTop = -(Filter.limsup (-fn) Filter.atTop) := by
simp [Filter.limsup_eq, Filter.liminf_eq, Real.sInf_def]
exact Real.ext_cauchy (congrArg Real.cauchy liminf_neg)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
have liminf_neg : Filter.liminf fn Filter.atTop = -(Filter.limsup (-fn) Filter.atTop) := by
simp [Filter.limsup_eq, Filter.liminf_eq, Real.sInf_def]
exact Real.ext_cauchy (congrArg Real.cauchy liminf_neg)
refine Real.ext_cauchy (congrArg Real.cauchy ?_)
simp [Filter.limsup_eq, Filter.liminf_eq, Real.sInf_def]

Haven't checked, and will leave it to the QuantumInfo folk for a proper review. But this might be shorter.

@jstoobysmith jstoobysmith added the awaiting-author A reviewer has asked the author a question or requested changes label Apr 7, 2026
@jstoobysmith jstoobysmith requested a review from Timeroot April 10, 2026 09:44
Copy link
Copy Markdown
Member

@jstoobysmith jstoobysmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've approved this, as this is no doubt an improvement.

Copy link
Copy Markdown
Member

@jstoobysmith jstoobysmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving - and will merge now.

@jstoobysmith jstoobysmith merged commit c11b282 into leanprover-community:master May 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-author A reviewer has asked the author a question or requested changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants