Opened 9 years ago

Closed 9 years ago

#12185 closed defect (duplicate)

Bug in norm for orders of relative number fields

Reported by: johanbosman Owned by: davidloeffler
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: number fields Keywords: sd35
Cc: Merged in:
Authors: Reviewers: David Loeffler
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

sage: K.<a,b> = NumberField([x^2 - 2, x^3 - 2])
sage: OK = K.maximal_order()
sage: OK(b).norm()
...
AttributeError: 'RelativeOrder' object has no attribute 'pari_polynomial'

Duplicate, fixed by #11904.

Attachments (2)

12185.patch (1.9 KB) - added by johanbosman 9 years ago.
This patch fixes a similar problem with the trace
12185.2.patch (1.1 KB) - added by johanbosman 9 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 follow-up: Changed 9 years ago by mkosters

The same problem occurs for the norm function of relative orders.

comment:2 in reply to: ↑ 1 Changed 9 years ago by mkosters

Replying to mkosters:

The same problem occurs for the TRACE function of relative orders.

comment:3 Changed 9 years ago by johanbosman

  • Authors set to Johan Bosman
  • Status changed from new to needs_review

Changed 9 years ago by johanbosman

This patch fixes a similar problem with the trace

comment:4 Changed 9 years ago by johanbosman

  • Status changed from needs_review to needs_work

Wait 1 second, maybe I've got a better fix.

Changed 9 years ago by johanbosman

comment:5 Changed 9 years ago by johanbosman

  • Description modified (diff)
  • Status changed from needs_work to needs_review

comment:6 Changed 9 years ago by johanbosman

  • Description modified (diff)

comment:7 Changed 9 years ago by davidloeffler

  • Milestone changed from sage-4.8 to sage-duplicate/invalid/wontfix
  • Status changed from needs_review to needs_info

Johan, the examples here work in 4.8.alpha5 without applying any patches! Apparently the issue has been fixed by some other patch merged in alpha5 (most likely #11904).

comment:8 Changed 9 years ago by mstreng

  • Keywords sd35 added
  • Reviewers set to David Loeffler

I confirm that the examples work in 4.8.alpha4 + #11904.

comment:9 Changed 9 years ago by davidloeffler

  • Authors Johan Bosman deleted
  • Description modified (diff)
  • Status changed from needs_info to needs_review

comment:10 Changed 9 years ago by davidloeffler

  • Status changed from needs_review to positive_review

comment:11 Changed 9 years ago by jdemeyer

  • Description modified (diff)
  • Resolution set to duplicate
  • Status changed from positive_review to closed

Cool, my patch #11904 fixed a bug by accident.

Note: See TracTickets for help on using tickets.