Skip to content

[Merged by Bors] - feat(SpecialFunctions/Log): add extended nonnegative real logarithm#14018

Closed
pitmonticone wants to merge 23 commits intomasterfrom
pitmonticone/ENNRealLog
Closed

[Merged by Bors] - feat(SpecialFunctions/Log): add extended nonnegative real logarithm#14018
pitmonticone wants to merge 23 commits intomasterfrom
pitmonticone/ENNRealLog

Conversation

@pitmonticone
Copy link
Copy Markdown
Member

@pitmonticone pitmonticone commented Jun 21, 2024

This PR adds the SpecialFunctions/Log/ENNRealLog.lean file where we define log as an extension of the logarithm of a positive real to the extended nonnegative reals ℝ≥0∞. The function takes values in the extended reals EReal, with log 0 = ⊥ and log ⊤ = ⊤.

Co-authored with @D-Thomine.

Co-Authored-By: D-Thomine <100795491+D-Thomine@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 21, 2024

PR summary 269142b3fb

Import changes

No significant changes to the import graph


Declarations diff

+ log
+ log_bijective
+ log_bot_lt_iff
+ log_continuous
+ log_eq_bot_iff
+ log_eq_iff
+ log_eq_one_iff
+ log_eq_top_iff
+ log_homeomorph
+ log_homeomorph_apply
+ log_injective
+ log_le_iff_le
+ log_le_one_iff
+ log_lt_iff_lt
+ log_lt_one_iff
+ log_lt_top_iff
+ log_monotone
+ log_mul_add
+ log_of_nnreal
+ log_one
+ log_one_le_iff
+ log_one_lt_iff
+ log_orderIso
+ log_orderIso_apply
+ log_pos_real
+ log_pos_real'
+ log_pow
+ log_rpow
+ log_strictMono
+ log_surjective
+ log_top
+ log_zero

You can run this locally as follows
## summary with just the declaration names:
./scripts/no_lost_declarations.sh short <optional_commit>

## more verbose report:
./scripts/no_lost_declarations.sh <optional_commit>

@pitmonticone pitmonticone added awaiting-review t-analysis Analysis (normed *, calculus) labels Jun 21, 2024
pitmonticone and others added 2 commits June 21, 2024 17:34
Co-Authored-By: D-Thomine <100795491+D-Thomine@users.noreply.github.com>
Co-Authored-By: D-Thomine <100795491+D-Thomine@users.noreply.github.com>
@pitmonticone pitmonticone marked this pull request as ready for review June 21, 2024 15:41
@pitmonticone pitmonticone requested a review from sgouezel June 21, 2024 15:46
Copy link
Copy Markdown
Contributor

@sgouezel sgouezel left a comment

Choose a reason for hiding this comment

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

Are you planning to also add exp in the other direction? (I'm asking because the order iso version and the homeomorphism version would love to have a simp lemma saying that, when you apply their symm, it coincides with exp). If yes in a near future, then everything is perfect. If not, can you add a todo about this at the end of the file-level comment?

@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Jun 21, 2024
Copy link
Copy Markdown
Contributor

@dupuisf dupuisf left a comment

Choose a reason for hiding this comment

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

I just have a few comments, but globally it looks good! I think it would be good to also add lemmas about (ENNReal.log x).toReal, my guess is that it would come up in practice quite a bit.

@dupuisf
Copy link
Copy Markdown
Contributor

dupuisf commented Jun 21, 2024

(Sorry about some of the double comments, I was reviewing at the same time as Sébastien...)

@pitmonticone
Copy link
Copy Markdown
Member Author

pitmonticone commented Jun 22, 2024

Are you planning to also add exp in the other direction? (I'm asking because the order iso version and the homeomorphism version would love to have a simp lemma saying that, when you apply their symm, it coincides with exp). If yes in a near future, then everything is perfect. If not, can you add a todo about this at the end of the file-level comment?

No, unfortunately I won't have enough time in the near future. I'm going to add the TODO.

@pitmonticone
Copy link
Copy Markdown
Member Author

pitmonticone commented Jun 22, 2024

Ok, I should have addressed all the review requests.

Thank you very much @sgouezel and @dupuisf for your quick and insightful reviews!

@pitmonticone pitmonticone removed the awaiting-author A reviewer has asked the author a question or requested changes. label Jun 22, 2024
Co-authored-by: Frédéric Dupuis <31101893+dupuisf@users.noreply.github.com>
@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Jun 22, 2024
@pitmonticone pitmonticone added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Jun 22, 2024
@pitmonticone pitmonticone requested review from dupuisf and sgouezel June 22, 2024 13:33
@pitmonticone
Copy link
Copy Markdown
Member Author

I should have addressed all the review requests.

Thank you very much @sgouezel and @dupuisf for your reviews!

@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Jun 23, 2024
@pitmonticone
Copy link
Copy Markdown
Member Author

pitmonticone commented Jun 23, 2024

Now I should have really addressed all the review requests 😅.

Thank you again @sgouezel and @dupuisf for your reviews!

@pitmonticone pitmonticone added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Jun 23, 2024
@sgouezel
Copy link
Copy Markdown
Contributor

bors r+
Thanks!

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Jun 23, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jun 23, 2024
…14018)

This PR adds the `SpecialFunctions/Log/ENNRealLog.lean` file where we define `log` as an extension of the logarithm of a positive real to the extended nonnegative reals `ℝ≥0∞`. The function takes values in the extended reals `EReal`, with `log 0 = ⊥` and `log ⊤ = ⊤`.

Co-authored with @D-Thomine.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jun 23, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(SpecialFunctions/Log): add extended nonnegative real logarithm [Merged by Bors] - feat(SpecialFunctions/Log): add extended nonnegative real logarithm Jun 23, 2024
@mathlib-bors mathlib-bors bot closed this Jun 23, 2024
@mathlib-bors mathlib-bors bot deleted the pitmonticone/ENNRealLog branch June 23, 2024 16:28
kbuzzard pushed a commit that referenced this pull request Jun 26, 2024
…14018)

This PR adds the `SpecialFunctions/Log/ENNRealLog.lean` file where we define `log` as an extension of the logarithm of a positive real to the extended nonnegative reals `ℝ≥0∞`. The function takes values in the extended reals `EReal`, with `log 0 = ⊥` and `log ⊤ = ⊤`.

Co-authored with @D-Thomine.
dagurtomas pushed a commit that referenced this pull request Jul 2, 2024
…14018)

This PR adds the `SpecialFunctions/Log/ENNRealLog.lean` file where we define `log` as an extension of the logarithm of a positive real to the extended nonnegative reals `ℝ≥0∞`. The function takes values in the extended reals `EReal`, with `log 0 = ⊥` and `log ⊤ = ⊤`.

Co-authored with @D-Thomine.
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