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

Residue incorrect for extensions of CA and FM extensions #25879

Closed
saraedum opened this issue Jul 19, 2018 · 15 comments
Closed

Residue incorrect for extensions of CA and FM extensions #25879

saraedum opened this issue Jul 19, 2018 · 15 comments

Comments

@saraedum
Copy link
Member

This has been reported on sage-devel recently:

sage: K = ZpCA(3,5)
sage: R.<a> = K[]
sage: L.<a> = K.extension(a^2 - 3)
sage: a.residue()
1

CC: @roed314 @xcaruso

Component: padics

Author: Julian Rüth

Branch: 5917d5d

Reviewer: Frédéric Chapoton

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

@saraedum saraedum added this to the sage-8.4 milestone Jul 19, 2018
@saraedum
Copy link
Member Author

comment:2

The issue seems to be in expansion(n):

sage: a.expansion()
[0, 1]
sage: a.expansion(0)
1

The problem is a line that says:

return ulist[n - self.ordp]

Here the index is negative in our case (n=0, self.ordp=1)

@saraedum
Copy link
Member Author

Branch: u/saraedum/25879

@saraedum
Copy link
Member Author

New commits:

ff8e3f4Port expansion fix from CR to CA and FM

@saraedum
Copy link
Member Author

Author: Julian Rüth

@saraedum

This comment has been minimized.

@saraedum
Copy link
Member Author

Commit: ff8e3f4

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 19, 2018

Changed commit from ff8e3f4 to 5917d5d

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 19, 2018

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

5917d5dPort expansion fix from CR to CA and FM

@saraedum
Copy link
Member Author

New commits:

5917d5dPort expansion fix from CR to CA and FM

@saraedum
Copy link
Member Author

Work Issues: is the patchbot happy?

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:7

let it be

@saraedum
Copy link
Member Author

Changed work issues from is the patchbot happy? to none

@vbraun
Copy link
Member

vbraun commented Aug 5, 2018

Changed branch from u/saraedum/25879 to 5917d5d

@fchapoton
Copy link
Contributor

Changed commit from 5917d5d to none

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

3 participants