Skip to content
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: API for continuous extension of meromorphic functions #22867

Closed
wants to merge 48 commits into from

Conversation

kebekus
Copy link
Collaborator

@kebekus kebekus commented Mar 12, 2025

Defines the normal form of meromorphic functions and provides API for continuous extension, as discussed on Zulip. More material will be provided in upcoming PRs.

This material is used in Project VD, which aims to formalize Value Distribution Theory for meromorphic functions on the complex plane.


Open in Gitpod

Copy link

github-actions bot commented Mar 12, 2025

PR summary 3a93a46216

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.Analysis.Meromorphic.NormalFormAt (new file) 1644

Declarations diff

+ AnalyticAt.meromorphicNFAt
+ MeromorphicAt.eqOn_compl_singleton_toMermomorphicNFAt
+ MeromorphicAt.eq_nhdNE_toMeromorphicNFAt
+ MeromorphicNFAt
+ MeromorphicNFAt.meromorphicAt
+ MeromorphicNFAt.order_nonneg_iff_analyticAt
+ eventuallyEq_nhds_of_eventuallyEq_nhdsNE
+ meromorphicNFAt_congr
+ meromorphicNFAt_iff_analyticAt_or
+ meromorphicNFAt_toMeromorphicNFAt
+ toMeromorphicNFAt
+ toMeromorphicNFAt_eq_self
+ toMeromorphicNFAt_of_not_meromorphicAt

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

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

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@kebekus kebekus added the t-analysis Analysis (normed *, calculus) label Mar 12, 2025
@YaelDillies YaelDillies added the awaiting-author A reviewer has asked the author a question or requested changes label Mar 12, 2025
Copy link
Collaborator

@loefflerd loefflerd left a comment

Choose a reason for hiding this comment

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

I took a careful look through this and found some golfs, and a few minor renaming suggestions, but the overall design looks great. Thanks for this nice contribution!

@kebekus kebekus removed the awaiting-author A reviewer has asked the author a question or requested changes label Mar 16, 2025
@YaelDillies YaelDillies added the awaiting-author A reviewer has asked the author a question or requested changes label Mar 16, 2025
@kebekus kebekus removed the awaiting-author A reviewer has asked the author a question or requested changes label Mar 16, 2025
@loefflerd
Copy link
Collaborator

I believe this PR is now ready for merging. One can debate forever about code-style issues, aesthetics etc; but I think all the necessary functionality is now present and in a good shape, and we shouldn't hold up the larger project of which this is part with further nit-picking.

maintainer merge

Copy link

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

Copy link
Member

@kbuzzard kbuzzard left a comment

Choose a reason for hiding this comment

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

This looks great! I particularly like the (currently apparently rather unfashionable) idea of putting docstrings on the key theorems. I had a docstring nit.

bors d+

@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Mar 16, 2025

✌️ kebekus can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@kebekus
Copy link
Collaborator Author

kebekus commented Mar 16, 2025

@jcommelin @YaelDillies @loefflerd @kbuzzard Thank you for the incredible work and all the time and effort you are putting into this.

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Mar 16, 2025
Defines the normal form of meromorphic functions and provides API for continuous extension, as discussed [on Zulip](https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/API.20for.20continuous.20extension.20of.20meromorphic.20functions). More material will be provided in upcoming PRs.

This material is used in [Project VD](https://github.com/kebekus/ProjectVD), which aims to formalize Value Distribution Theory for meromorphic functions on the complex plane.
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Mar 16, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: API for continuous extension of meromorphic functions [Merged by Bors] - feat: API for continuous extension of meromorphic functions Mar 16, 2025
@mathlib-bors mathlib-bors bot closed this Mar 16, 2025
@mathlib-bors mathlib-bors bot deleted the kebekus/meromorphic_normalFormAt branch March 16, 2025 13:01
qawbecrdtey added a commit that referenced this pull request Mar 17, 2025
…vProdLpPiLp` (#22993)

* feat: scalar tower instances for quotients (#22951)

We already have the `SMulCommClass` and `IsScalarTower` versions for `RingQuot`; this develops them for `Con` and `RingCon`, with the eventual aim of replacing `RingQuot` with `RingCon.Quotient`.

* chore(Order/Group/Abs): use `@[to_additive]` (#22468)

* feat(FieldTheory/Finite/Basic): lemmas about the prime subfield in positive characteristic (#22843)

This PR adds some elementary results about the prime subfield of a characteristic p field, e.g., size is p, elements are integer multiples of one, and elements are characterized by being fixed by the p-th power map.

* docs(Data/Real/EReal): fix capitalization error (#22943)

Changes `Ereal` to `EReal` in the module docstring for `Data/Real/EReal`.

* chore(Ideal/Quotient): change `Fintype` to `Finite` (#22947)

As discussed [here](#22902 (comment))

* feat: add `norm_num` extensions for factorials (#8832)

Add `norm_num` extensions to evaluate `Nat.factorial`, `Nat.ascFactorial` and `Nat.descFactorial`.



Co-authored-by: Eric Wieser <[email protected]>

* perf(CategoryTheory/Limits/Shapes): reorder instance arguments (#22968)

This PR is in the same spirit as #22953.

The problem is that some instances about category theoretical limits have silly side conditions that end up searching through the whole algebraic type class hierarchy. This PR attempts to keep the type class search limited to category theoretical type classes.

* feat(LinearAlgebra/FreeModule/CardQuotient): compute indices of subgroups via determinant (#22940)

* feat: API for continuous extension of meromorphic functions (#22867)

Defines the normal form of meromorphic functions and provides API for continuous extension, as discussed [on Zulip](https://leanprover.zulipchat.com/#narrow/channel/287929-mathlib4/topic/API.20for.20continuous.20extension.20of.20meromorphic.20functions). More material will be provided in upcoming PRs.

This material is used in [Project VD](https://github.com/kebekus/ProjectVD), which aims to formalize Value Distribution Theory for meromorphic functions on the complex plane.

* feat(Data/Complex/Trigonometric): closer upper bound for cos 1 (#22945)

cos 1 is approximately 0.5403..., so this bound is fairly tight.

Co-authored-by: Vlad Tsyrklevich <[email protected]>

* chore: move `List.Lex` lemmas out of the `List.Lex` namespace (#22935)

This better matches the naming convention.

* working on it.

* Added sup_disjSum and inf_disjSum.

* Finished one branch.

* Finished proof.

---------

Co-authored-by: Eric Wieser <[email protected]>
Co-authored-by: Yury G. Kudryashov <[email protected]>
Co-authored-by: Scott Carnahan <[email protected]>
Co-authored-by: plp127 <[email protected]>
Co-authored-by: Xavier Roblot <[email protected]>
Co-authored-by: Sebastian Zimmer <[email protected]>
Co-authored-by: JovanGerb <[email protected]>
Co-authored-by: Stefan Kebekus <[email protected]>
Co-authored-by: Vlad Tsyrklevich <[email protected]>
Co-authored-by: Vlad Tsyrklevich <[email protected]>
Co-authored-by: Yaël Dillies <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated t-analysis Analysis (normed *, calculus)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants