Skip to content

[Merged by Bors] - feat: Add Real.pi_nonneg = Real.pi_pos.le#10596

Closed
girving wants to merge 2 commits intomasterfrom
GI_pi
Closed

[Merged by Bors] - feat: Add Real.pi_nonneg = Real.pi_pos.le#10596
girving wants to merge 2 commits intomasterfrom
GI_pi

Conversation

@girving
Copy link
Copy Markdown
Collaborator

@girving girving commented Feb 15, 2024

I think if Real.exp_nonneg exists as a separate lemma, Real.pi_nonneg is reasonable. And it's useful as a separate lemma for Aesop purposes too.


Open in Gitpod

I think if Real.exp_nonneg exists as a separate lemma, Real.pi_nonneg is
reasonable.  And it's useful as a separate lemma for Aesop purposes too.
@girving girving added awaiting-review t-analysis Analysis (normed *, calculus) labels Feb 15, 2024
@girving girving changed the title feat: Add Real.pi_nonneg = Real.pi_pos.le feat: Add Real.pi_nonneg = Real.pi_pos.le Feb 15, 2024
@adomani
Copy link
Copy Markdown
Contributor

adomani commented Feb 16, 2024

I am not sure where to draw the line on lemmas about "important" constants. In particular, on the one hand, I understand that these lemmas can be helpful for certain tactics, on the other hand these lemmas also seem the ones that automation could make obsolete!

I'm pinging @urkud and @sgouezel who can make a more informed decision about this PR.

@sgouezel
Copy link
Copy Markdown
Contributor

This one will show up so often that it's definitely worth having explicitely. Thanks!
bors r+

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Feb 16, 2024
mathlib-bors bot pushed a commit that referenced this pull request Feb 16, 2024
I think if `Real.exp_nonneg` exists as a separate lemma, `Real.pi_nonneg` is reasonable.  And it's useful as a separate lemma for Aesop purposes too.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 16, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: Add Real.pi_nonneg = Real.pi_pos.le [Merged by Bors] - feat: Add Real.pi_nonneg = Real.pi_pos.le Feb 16, 2024
@mathlib-bors mathlib-bors bot closed this Feb 16, 2024
@mathlib-bors mathlib-bors bot deleted the GI_pi branch February 16, 2024 14:44
riccardobrasca pushed a commit that referenced this pull request Feb 18, 2024
I think if `Real.exp_nonneg` exists as a separate lemma, `Real.pi_nonneg` is reasonable.  And it's useful as a separate lemma for Aesop purposes too.
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
I think if `Real.exp_nonneg` exists as a separate lemma, `Real.pi_nonneg` is reasonable.  And it's useful as a separate lemma for Aesop purposes too.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-analysis Analysis (normed *, calculus)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants