Skip to content

[Merged by Bors] - feat: Eisenstein q exp identity#27606

Closed
CBirkbeck wants to merge 126 commits intoleanprover-community:masterfrom
CBirkbeck:Eisenstein_q_exp_identity
Closed

[Merged by Bors] - feat: Eisenstein q exp identity#27606
CBirkbeck wants to merge 126 commits intoleanprover-community:masterfrom
CBirkbeck:Eisenstein_q_exp_identity

Conversation

Comment thread Mathlib/NumberTheory/ModularForms/EisensteinSeries/QExpansion.lean Outdated
Comment thread Mathlib/NumberTheory/ModularForms/EisensteinSeries/QExpansion.lean Outdated
@MichaelStollBayreuth MichaelStollBayreuth added the awaiting-author A reviewer has asked the author a question or requested changes. label Nov 12, 2025
@CBirkbeck CBirkbeck removed the awaiting-author A reviewer has asked the author a question or requested changes. label Nov 12, 2025
@MichaelStollBayreuth MichaelStollBayreuth added the awaiting-author A reviewer has asked the author a question or requested changes. label Nov 13, 2025
@CBirkbeck CBirkbeck removed the awaiting-author A reviewer has asked the author a question or requested changes. label Nov 14, 2025
@MichaelStollBayreuth
Copy link
Copy Markdown
Contributor

LGTM now.
maintainer merge

@github-actions
Copy link
Copy Markdown

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

@ghost ghost added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Nov 14, 2025
@riccardobrasca
Copy link
Copy Markdown
Member

Can you add at least the main declaration of this PR to the doc at the beginning of the file? Thanks!

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Nov 14, 2025

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

@ghost ghost added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels Nov 14, 2025
@CBirkbeck
Copy link
Copy Markdown
Collaborator Author

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Nov 14, 2025
We prove that Eisenstein series have a q-expansion of the form `1 - (2k / bernoulli k) ∑' n, σ_{k-1}(n) q^n`
@bryangingechen bryangingechen changed the title Eisenstein q exp identity feat: Eisenstein q exp identity Nov 14, 2025
@bryangingechen
Copy link
Copy Markdown
Contributor

bryangingechen commented Nov 14, 2025

Just requeueing with a title that fits our conventions better.
bors r-
bors r+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Nov 14, 2025

Canceled.

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed ready-to-merge This PR has been sent to bors. delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). labels Nov 14, 2025
mathlib-bors bot pushed a commit that referenced this pull request Nov 14, 2025
We prove that Eisenstein series have a q-expansion of the form `1 - (2k / bernoulli k) ∑' n, σ_{k-1}(n) q^n`
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Nov 14, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: Eisenstein q exp identity [Merged by Bors] - feat: Eisenstein q exp identity Nov 14, 2025
@mathlib-bors mathlib-bors bot closed this Nov 14, 2025
@leanprover-radar
Copy link
Copy Markdown

Benchmarking c38fcdb (status) against 472dcfc (status).

React with 👀 to be notified when the results are in. The command author is always notified.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

large-import Automatically added label for PRs with a significant increase in transitive imports

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants