Skip to content

[Merged by Bors] - feat(Analysis/SpecialFunctions/Complex/Arg): add definition for slit plane and API, and use it#9116

Closed
MichaelStollBayreuth wants to merge 11 commits intomasterfrom
MS_SlitPlane
Closed

[Merged by Bors] - feat(Analysis/SpecialFunctions/Complex/Arg): add definition for slit plane and API, and use it#9116
MichaelStollBayreuth wants to merge 11 commits intomasterfrom
MS_SlitPlane

Conversation

@MichaelStollBayreuth
Copy link
Copy Markdown
Contributor

In preparation of future PRs dealing with estimates of the complex logarithm and its Taylor series, this introduces Complex.slitPlane for the set of complex numbers not on the closed negative real axis (in Analysis.SpecialFunctions.Complex.Arg), adds a bunch of API lemmas, and replaces hypotheses of the form 0 < x.re ∨ x.im ≠ 0 by x ∈ slitPlane in several other files.

(We do not introduce a new file for that to avoid circular imports with Analysis.SpecialFunctions.Complex.Arg.)


Open in Gitpod

@MichaelStollBayreuth MichaelStollBayreuth added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. t-analysis Analysis (normed *, calculus) labels Dec 16, 2023
@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Dec 16, 2023
@urkud
Copy link
Copy Markdown
Member

urkud commented Dec 20, 2023

I generalized the "star shaped" lemma and moved it to a new file. I also dropped in a lemma I wrote a few days ago but forgot to PR. Also, I added some lemmas and golfed proofs using ComplexOrder.

@MichaelStollBayreuth
Copy link
Copy Markdown
Contributor Author

OK, nice. There is still a broken proof, though.

@urkud
Copy link
Copy Markdown
Member

urkud commented Dec 21, 2023

LGTM but I contributed too much to merge it myself. Thanks!
maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by urkud.

@sgouezel sgouezel added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Dec 22, 2023
@MichaelStollBayreuth MichaelStollBayreuth added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Dec 22, 2023
@urkud urkud added the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Dec 22, 2023
@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Dec 22, 2023
@jcommelin
Copy link
Copy Markdown
Member

Thanks 🎉

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Dec 23, 2023
mathlib-bors bot pushed a commit that referenced this pull request Dec 23, 2023
…plane and API, and use it (#9116)

In preparation of future PRs dealing with estimates of the complex logarithm and its Taylor series, this introduces `Complex.slitPlane` for the set of complex numbers not on the closed negative real axis (in `Analysis.SpecialFunctions.Complex.Arg`), adds a bunch of API lemmas, and replaces hypotheses of the form `0 < x.re  ∨ x.im ≠ 0` by `x ∈ slitPlane` in several other files.

(We do not introduce a new file for that to avoid circular imports with `Analysis.SpecialFunctions.Complex.Arg`.)



Co-authored-by: Yury G. Kudryashov <urkud@urkud.name>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Dec 23, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Analysis/SpecialFunctions/Complex/Arg): add definition for slit plane and API, and use it [Merged by Bors] - feat(Analysis/SpecialFunctions/Complex/Arg): add definition for slit plane and API, and use it Dec 23, 2023
@mathlib-bors mathlib-bors bot closed this Dec 23, 2023
@mathlib-bors mathlib-bors bot deleted the MS_SlitPlane branch December 23, 2023 07:19
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.

4 participants