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(ModularForms/JacobiTheta): derivative of theta function #11824

Closed
wants to merge 9 commits into from

Conversation

loefflerd
Copy link
Collaborator

@loefflerd loefflerd commented Apr 1, 2024

This is a rewrite of JacobiTheta/TwoVariable.lean, adding a number of new results needed for Hurwitz and Dirichlet L-series.

The main addition is developing the theory of the z-derivative of the theta function, as an object of study in its own right (functional equations, periodicity, holomorphy etc) – this is needed to prove analytic continuation + functional equations for odd Dirichlet characters.

We also add a number of new results about the existing jacobiTheta2 function:

  • converses of all the convergence statements (showing the series are never convergent if tau is outside the upper half-plane), which allows hypotheses to be removed from several results further downstream
  • differentiability in both variables jointly, not just each variable separately as before.

@loefflerd loefflerd added awaiting-review The author would like community review of the PR t-number-theory Number theory (also use t-algebra or t-analysis to specialize) t-analysis Analysis (normed *, calculus) labels Apr 1, 2024
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Apr 1, 2024
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Apr 2, 2024
@MichaelStollBayreuth
Copy link
Collaborator

Nice work!
I'll try to go through the proofs in more detail later, but so far it looks good to me.

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Apr 7, 2024
loefflerd and others added 2 commits April 7, 2024 10:08
Co-authored-by: Michael Stoll <99838730+MichaelStollBayreuth@users.noreply.github.com>
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Apr 7, 2024
@loefflerd
Copy link
Collaborator Author

On my system, compilation of JacobiTheta/TwoVariable.lean (at the current version, commit bf62a22) takes about 30 seconds. Merging #11980 brings this down to 26 seconds. This is not quite as much of a speedup as adding Michael's list of explicit custom instances (23 sec on my machine), but it's not too far off.

I think merging in the long list of explicit instances would probably be judged as contravening mathlib's "house style" (especially given the conclusion of the recent zulip debate "To automate or not to automate?", where several prominent members of the community argued strongly that we should use automated systems whenever possible and tolerate some slow-down as a result). Other than that, I don't think there's much more that can be done to speed it up.

May I suggest that we wait on #11980, and when that is merged, we merge this as it is?

@loefflerd loefflerd added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Apr 7, 2024
@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the blocked-by-other-PR This PR depends on another PR which is still in the queue. label Apr 7, 2024
@MichaelStollBayreuth
Copy link
Collaborator

On my system, compilation of JacobiTheta/TwoVariable.lean (at the current version, commit bf62a22) takes about 30 seconds. Merging #11980 brings this down to 26 seconds. This is not quite as much of a speedup as adding Michael's list of explicit custom instances (23 sec on my machine), but it's not too far off.

I think merging in the long list of explicit instances would probably be judged as contravening mathlib's "house style" (especially given the conclusion of the recent zulip debate "To automate or not to automate?", where several prominent members of the community argued strongly that we should use automated systems whenever possible and tolerate some slow-down as a result). Other than that, I don't think there's much more that can be done to speed it up.

May I suggest that we wait on #11980, and when that is merged, we merge this as it is?

I tend to agree. It would be good, however, to solve the underlying problem. See also here on Zulip.

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot removed the blocked-by-other-PR This PR depends on another PR which is still in the queue. label Apr 8, 2024
@leanprover-community-mathlib4-bot
Copy link
Collaborator

This PR/issue depends on:

@MichaelStollBayreuth
Copy link
Collaborator

maintainer merge

Copy link

github-actions bot commented Apr 8, 2024

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

@riccardobrasca
Copy link
Member

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Apr 8, 2024
mathlib-bors bot pushed a commit that referenced this pull request Apr 8, 2024
This is a rewrite of `JacobiTheta/TwoVariable.lean`, adding a number of new results needed for Hurwitz and Dirichlet L-series.

The main addition is developing the theory of the `z`-derivative of the theta function, as an object of study in its own right (functional equations, periodicity, holomorphy etc) – this is needed to prove analytic continuation + functional equations for odd Dirichlet characters. 

We also add a number of new results about the existing `jacobiTheta2` function:
- converses of all the convergence statements (showing the series are *never* convergent if tau is outside the upper half-plane), which allows hypotheses to be removed from several results further downstream
- differentiability in both variables jointly, not just each variable separately as before.
@mathlib-bors
Copy link

mathlib-bors bot commented Apr 8, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(ModularForms/JacobiTheta): derivative of theta function [Merged by Bors] - feat(ModularForms/JacobiTheta): derivative of theta function Apr 8, 2024
@mathlib-bors mathlib-bors bot closed this Apr 8, 2024
@mathlib-bors mathlib-bors bot deleted the DL_jacobi_odd branch April 8, 2024 13:41
Louddy pushed a commit that referenced this pull request Apr 15, 2024
This is a rewrite of `JacobiTheta/TwoVariable.lean`, adding a number of new results needed for Hurwitz and Dirichlet L-series.

The main addition is developing the theory of the `z`-derivative of the theta function, as an object of study in its own right (functional equations, periodicity, holomorphy etc) – this is needed to prove analytic continuation + functional equations for odd Dirichlet characters. 

We also add a number of new results about the existing `jacobiTheta2` function:
- converses of all the convergence statements (showing the series are *never* convergent if tau is outside the upper half-plane), which allows hypotheses to be removed from several results further downstream
- differentiability in both variables jointly, not just each variable separately as before.
atarnoam pushed a commit that referenced this pull request Apr 16, 2024
This is a rewrite of `JacobiTheta/TwoVariable.lean`, adding a number of new results needed for Hurwitz and Dirichlet L-series.

The main addition is developing the theory of the `z`-derivative of the theta function, as an object of study in its own right (functional equations, periodicity, holomorphy etc) – this is needed to prove analytic continuation + functional equations for odd Dirichlet characters. 

We also add a number of new results about the existing `jacobiTheta2` function:
- converses of all the convergence statements (showing the series are *never* convergent if tau is outside the upper half-plane), which allows hypotheses to be removed from several results further downstream
- differentiability in both variables jointly, not just each variable separately as before.
uniwuni pushed a commit that referenced this pull request Apr 19, 2024
This is a rewrite of `JacobiTheta/TwoVariable.lean`, adding a number of new results needed for Hurwitz and Dirichlet L-series.

The main addition is developing the theory of the `z`-derivative of the theta function, as an object of study in its own right (functional equations, periodicity, holomorphy etc) – this is needed to prove analytic continuation + functional equations for odd Dirichlet characters. 

We also add a number of new results about the existing `jacobiTheta2` function:
- converses of all the convergence statements (showing the series are *never* convergent if tau is outside the upper half-plane), which allows hypotheses to be removed from several results further downstream
- differentiability in both variables jointly, not just each variable separately as before.
callesonne pushed a commit that referenced this pull request Apr 22, 2024
This is a rewrite of `JacobiTheta/TwoVariable.lean`, adding a number of new results needed for Hurwitz and Dirichlet L-series.

The main addition is developing the theory of the `z`-derivative of the theta function, as an object of study in its own right (functional equations, periodicity, holomorphy etc) – this is needed to prove analytic continuation + functional equations for odd Dirichlet characters. 

We also add a number of new results about the existing `jacobiTheta2` function:
- converses of all the convergence statements (showing the series are *never* convergent if tau is outside the upper half-plane), which allows hypotheses to be removed from several results further downstream
- differentiability in both variables jointly, not just each variable separately as before.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintainer-merge ready-to-merge This PR has been sent to bors. t-analysis Analysis (normed *, calculus) t-number-theory Number theory (also use t-algebra or t-analysis to specialize)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants