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

Frobenius endomorphism over p-adics #14824

Closed
xcaruso opened this issue Jun 26, 2013 · 9 comments
Closed

Frobenius endomorphism over p-adics #14824

xcaruso opened this issue Jun 26, 2013 · 9 comments

Comments

@xcaruso
Copy link
Contributor

xcaruso commented Jun 26, 2013

This patch provides a method frobenius_endomorphism which returns the Frobenius endomorphism of a p-adic field.

Here is a small demo:

sage: K.<a> = Qq(5^3)
sage: Frob = K.frobenius_endomorphism()

sage: Frob
Frobenius endomorphism on Unramified Extension of 5-adic ... lifting a |--> a^5 on the residue field
sage: Frob^12
Identity endomorphism of Unramified Extension of 5-adic Field ...

sage: Frob(a) = a.frobenius()
True

Component: padics

Keywords: Frobenius

Author: Xavier Caruso

Branch/Commit: u/kedlaya/ticket/14824 @ dc5b393

Reviewer: Kiran Kedlaya

Issue created by migration from https://trac.sagemath.org/ticket/14824

@roed314
Copy link
Contributor

roed314 commented Jul 8, 2013

comment:2
  • Why do you need to store _degree_fixed?
  • There are a couple functions without doctests, and the implementation of hash can be easily improved (use power rather than the string representation).
  • One doctest failure:
File "/mnt/storage2TB/patchbot/Sage/sage-5.11.beta3/devel/sage/sage/rings/padics/padic_generic.py", line 566, in sage.rings.padics.padic_generic.pAdicGeneric.frobenius_endomorphism
Failed example:
    K.frobenius_endomorphism(6) == Frob
Expected:
    True
Got:
    False

@xcaruso
Copy link
Contributor Author

xcaruso commented Jul 9, 2013

comment:3

Replying to @roed314:

  • Why do you need to store _degree_fixed?

Oops. It was because I was first planning to implement a method fixed_points but then I gave up. It seems that I forgot to remode _degree_fixed.

  • There are a couple functions without doctests, and the implementation of hash can be easily improved (use power rather than the string representation).

Done.

  • One doctest failure:

Fixed.

@xcaruso
Copy link
Contributor Author

xcaruso commented Jul 9, 2013

@roed314
Copy link
Contributor

roed314 commented Jan 6, 2014

comment:4

Note that this ticket does not resolve the horrible performance of the frobenius method: see #12657. But the changes here are certainly worth incorporating, and don't need to wait on #12657.

@kedlaya
Copy link
Sponsor Contributor

kedlaya commented Jan 7, 2014

Branch: u/kedlaya/ticket/14824

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 7, 2014

Branch pushed to git repo; I updated commit sha1. New commits:

dc5b393Merge branch 'develop' into ticket/14824

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 7, 2014

Commit: dc5b393

@kedlaya
Copy link
Sponsor Contributor

kedlaya commented Jan 7, 2014

comment:7

With help from roed, I converted this to the git workflow; it merges into 6.1beta3 with only trivial modifications. Looks good to me.

@kedlaya
Copy link
Sponsor Contributor

kedlaya commented Jan 7, 2014

Reviewer: Kiran Kedlaya

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

No branches or pull requests

4 participants