Opened 12 years ago

Closed 12 years ago

#9764 closed enhancement (fixed)

Change hashing and printing for NumberFieldIdeals

Reported by: Jeroen Demeyer Owned by: David Loeffler
Priority: major Milestone: sage-4.6
Component: number fields Keywords: number field ideal hash
Cc: William Stein Merged in: sage-4.6.alpha2
Authors: William Stein, Jeroen Demeyer Reviewers: David Loeffler
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Jeroen Demeyer)

This ticket is a fork from #9400.

  • implement hashing for number field ideals that isn't a stupid string repr, hence vastly faster
  • make number field ideals *not* print in reduced form; this will look uglier, but is massively faster and more sensible for any real applications, as people learned constantly at sage days 22.

Dependencies: #9898, #9753

Apply only 9764_ideal_repr_new.patch

Attachments (2)

9764_ideal_repr.patch (17.5 KB) - added by Jeroen Demeyer 12 years ago.
Patch against sage-4.5.3.alpha0
9764_ideal_repr_new.patch (17.4 KB) - added by Jeroen Demeyer 12 years ago.
Patch against sage-4.6.alpha0 + #9400 + #9898 + #9753

Download all attachments as: .zip

Change History (10)

comment:1 Changed 12 years ago by Jeroen Demeyer

Status: newneeds_work

Needs to be rebased to #9343.

Changed 12 years ago by Jeroen Demeyer

Attachment: 9764_ideal_repr.patch added

Patch against sage-4.5.3.alpha0

comment:2 Changed 12 years ago by Jeroen Demeyer

Authors: William SteinWilliam Stein, Jeroen Demeyer

comment:3 Changed 12 years ago by Jeroen Demeyer

Keywords: number field ideal hash added
Status: needs_workneeds_review

Changed 12 years ago by Jeroen Demeyer

Attachment: 9764_ideal_repr_new.patch added

Patch against sage-4.6.alpha0 + #9400 + #9898 + #9753

comment:4 Changed 12 years ago by David Loeffler

Reviewers: David Loeffler
Status: needs_reviewpositive_review

I applied this to sage-4.6.alpha1 with #9898 and #9753 installed. Changes look sensible, patch applies fine, and all doctests pass. Positive review.

comment:5 Changed 12 years ago by Jeroen Demeyer

Description: modified (diff)

comment:6 Changed 12 years ago by Jeroen Demeyer

Description: modified (diff)

comment:7 Changed 12 years ago by Mitesh Patel

Merged in: sage-4.6.alpha2

comment:8 Changed 12 years ago by Mitesh Patel

Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.