Ticket #805 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

[with patch] is_trivial() does not work for fractional ideals of number field

Reported by: dmharvey Owned by: robertwb
Priority: major Milestone: sage-2.8.13
Component: number theory Keywords:
Cc: Author(s):
Report Upstream: Reviewer(s):
Merged in: Work issues:

Description

sage: F.<a> = QuadraticField(-5)
sage: I = F.ideal(3)
sage: I.is_trivial()
---------------------------------------------------------------------------
<type 'exceptions.AttributeError'>        Traceback (most recent call last)

/Users/david/sage-2.8.5/<ipython console> in <module>()

/Users/david/sage-2.8.5/local/lib/python2.5/site-packages/sage/rings/ideal.py in is_trivial(self)
    229             return True
    230         elif self.is_principal():
--> 231             return self.gen().is_unit()
    232         raise NotImplementedError
    233 

<type 'exceptions.AttributeError'>: 'NumberFieldIdeal' object has no attribute 'gen'

Attachments

805-trivial_nf_ideal.diff Download (1.5 KB) - added by robertwb 3 years ago.
805-ncalexan-general-2.diff Download (3.4 KB) - added by ncalexan 3 years ago.

Change History

Changed 3 years ago by mabshoff

  • milestone set to sage-2.9

Changed 3 years ago by was

  • milestone changed from sage-2.9.1 to sage-2.8.12

Changed 3 years ago by robertwb

  • owner changed from was to robertwb
  • status changed from new to assigned

Changed 3 years ago by robertwb

Changed 3 years ago by robertwb

  • summary changed from is_trivial() does not work for fractional ideals of number field to [with patch] is_trivial() does not work for fractional ideals of number field

Changed 3 years ago by ncalexan

Changed 3 years ago by ncalexan

Both patches above should be applied: Robert's handles the special case of number fields; mine fixes a bug in multipolynomial ideals and makes generic ideals more robust.

Changed 3 years ago by mabshoff

  • milestone changed from sage-2.9 to sage-2.8.13

Changed 3 years ago by was

GOOD -- I especially like Nick's improvements.

Changed 3 years ago by mabshoff

  • status changed from assigned to closed
  • resolution set to fixed

Merged in 2.8.13.alpha1.

Note: See TracTickets for help on using tickets.