Opened 12 years ago

Closed 12 years ago

#827 closed defect (fixed)

[with patch, with positive review] nfrootsof1 from Pari

Reported by: jvoight Owned by: was
Priority: major Milestone: sage-2.10.1
Component: number theory Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

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 12 years ago.
ncalexan-827-updated.patch (2.7 KB) - added by ncalexan 12 years ago.

Download all attachments as: .zip

Change History (5)

Changed 12 years ago by ncalexan

comment:1 Changed 12 years ago by ncalexan

  • Summary changed from nfrootsof1 from Pari to [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 12 years ago by ncalexan

comment:2 Changed 12 years ago by craigcitro

  • Summary changed from [with patch, needs review] nfrootsof1 from Pari to [with patch, with positive review] nfrootsof1 from Pari

comment:3 Changed 12 years ago by mabshoff

  • Resolution set to fixed
  • Status changed from new to closed

Merged in Sage 2.10.1.alpha0

Note: See TracTickets for help on using tickets.