Opened 12 years ago

Closed 12 years ago

#10363 closed enhancement (fixed)

Implement a.ord(p) for QQ and number fields

Reported by: rlm Owned by: davidloeffler
Priority: major Milestone: sage-4.6.2
Component: number fields Keywords:
Cc: Merged in: sage-4.6.2.alpha0
Authors: Robert Miller Reviewers: John Cremona
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description


Attachments (1)

trac_10363.patch​ (2.2 KB) - added by rlm 12 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 12 years ago by rlm

Status: new → needs_review

comment:2 Changed 12 years ago by cremona

Reviewers: → John Cremona
Status: needs_review → positive_review

Applies fine to 4.6.1.alpha2, looks good and all tests pass.

comment:3 Changed 12 years ago by robertwb

You beat me to it by 37 minutes! Positive review from me too.

comment:4 Changed 12 years ago by jdemeyer

Status: positive_review → needs_work

Please update the commit message of the patch.

Changed 12 years ago by rlm

Attachment: trac_10363.patch​ added

comment:5 Changed 12 years ago by rlm

Status: needs_work → positive_review

comment:6 Changed 12 years ago by jdemeyer

Merged in: → sage-4.6.2.alpha0
Resolution: → fixed
Status: positive_review → closed
Note: See TracTickets for help on using tickets.