Opened 15 years ago

Closed 15 years ago

#827 closed defect (fixed)

[with patch, with positive review] nfrootsof1 from Pari

Reported by: John Voight Owned by: William Stein
Priority: major Milestone: sage-2.10.1
Component: number theory Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

Import nfrootsof1 from Pari so as to count the number of roots of unity in a number field.

Attachments (2)

ncalexan-827.patch (2.8 KB) - added by ncalexan 15 years ago.
ncalexan-827-updated.patch (2.7 KB) - added by ncalexan 15 years ago.

Download all attachments as: .zip

Change History (5)

Changed 15 years ago by ncalexan

Attachment: ncalexan-827.patch added

comment:1 Changed 15 years ago by ncalexan

Summary: nfrootsof1 from Pari[with patch, needs review] nfrootsof1 from Pari

It appears that nfrootsof1 is available, i.e. via K.pari_nf().nfrootsof1. This just uses it, and fixes (what I think is) a small bug in CyclotomicField?(7).zeta(14).

Changed 15 years ago by ncalexan

Attachment: ncalexan-827-updated.patch added

comment:2 Changed 15 years ago by Craig Citro

Summary: [with patch, needs review] nfrootsof1 from Pari[with patch, with positive review] nfrootsof1 from Pari

comment:3 Changed 15 years ago by Michael Abshoff

Resolution: fixed
Status: newclosed

Merged in Sage 2.10.1.alpha0

Note: See TracTickets for help on using tickets.