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

Implement hypergeometric Euler factors for tame primes #37881

Conversation

kedlaya
Copy link
Contributor

@kedlaya kedlaya commented Apr 27, 2024

We continue the port of hypergeometric motives from Magma by implementing the computation of hypergeometric traces and Euler factors for tame primes. This splits into two essentially separate cases: when t has nonzero p-adic valuation ("tame"), and when t-1 has nonzero p-adic valuation ("multiplicative"). In the second case, the computation uses the local functional equation as in the good reduction case.

Copy link

github-actions bot commented Apr 27, 2024

Documentation preview for this PR (built with commit 1de49be; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

Copy link
Collaborator

@tscrim tscrim left a comment

Choose a reason for hiding this comment

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

I am assuming the math is good (I don't really know anything about it). I just have a few minor comments about the code. Once addressed, I am willing to set a positive review.

src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
@kedlaya kedlaya changed the title Implement hypergeometric euler factors for some tame primes Implement hypergeometric Euler factors for tame primes May 5, 2024
@kedlaya
Copy link
Contributor Author

kedlaya commented May 6, 2024

I think all the content is (finally) here now, but I'm not sure what's going on with the build failures in CI.

@tscrim
Copy link
Collaborator

tscrim commented May 6, 2024

Those build failures are happening across all PRs. I don't know what's going on there either.

src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Show resolved Hide resolved
src/sage/modular/hypergeometric_motive.py Outdated Show resolved Hide resolved
# now p is good, or p is tame and t is a p-adic unit
elif (t-1) % p == 0:
typ = "mult"
d = (self.degree() - 1) // 2 * 2
Copy link
Collaborator

Choose a reason for hiding this comment

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

I still find it takes me a moment to realize that these operations should not simply cancel (and could follow my suggestion on the first round).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe it's better to be clear than concise, reworked the logic.

@fchapoton
Copy link
Contributor

For a tentative to repair the currently very broken CI , see #37926

@github-actions github-actions bot added v: large and removed v: small labels May 9, 2024
Copy link
Collaborator

@tscrim tscrim left a comment

Choose a reason for hiding this comment

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

Thank you. LGTM.

vbraun pushed a commit to vbraun/sage that referenced this pull request May 11, 2024
    
We continue the port of hypergeometric motives from Magma by
implementing the computation of hypergeometric traces and Euler factors
for tame primes. This splits into two essentially separate cases: when t
has nonzero p-adic valuation ("tame"), and when t-1 has nonzero p-adic
valuation ("multiplicative"). In the second case, the computation uses
the local functional equation as in the good reduction case.
    
URL: sagemath#37881
Reported by: kedlaya
Reviewer(s): kedlaya, Travis Scrimshaw
vbraun pushed a commit to vbraun/sage that referenced this pull request May 12, 2024
    
We continue the port of hypergeometric motives from Magma by
implementing the computation of hypergeometric traces and Euler factors
for tame primes. This splits into two essentially separate cases: when t
has nonzero p-adic valuation ("tame"), and when t-1 has nonzero p-adic
valuation ("multiplicative"). In the second case, the computation uses
the local functional equation as in the good reduction case.
    
URL: sagemath#37881
Reported by: kedlaya
Reviewer(s): kedlaya, Travis Scrimshaw
@vbraun vbraun merged commit 649c4d5 into sagemath:develop May 12, 2024
19 checks passed
@mkoeppe mkoeppe added this to the sage-10.4 milestone May 12, 2024
vbraun pushed a commit to vbraun/sage that referenced this pull request Jul 20, 2024
    
Since sagemath#37881 was merged, we have the ability to compute hypergeometric
traces at the degenerate value t=1. This PR makes a few changes to
expose the corresponding functionality for Euler factors (again
following the lead of Magma).
    
URL: sagemath#38322
Reported by: kedlaya
Reviewer(s): Travis Scrimshaw
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants