Opened 14 years ago

Closed 14 years ago

#5027 closed defect (fixed)

[with patch, positive review] doctest failure for rings/polynomial/toy_d_basis.py

Reported by: mhampton Owned by: mabshoff
Priority: major Milestone: sage-3.3
Component: algebra Keywords: groebner, toy, toy_d_basis
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

I get this failure on an intel mac:

sage -t  "devel/sage/sage/rings/polynomial/toy_d_basis.py"
**********************************************************************
File ".../devel/sage/sage/rings/polynomial/toy_d_basis.py", line 91:
    sage: d_basis(I)
Expected:
    [x + 170269749119, y + 2149906854, z + 735710619426, 282687803443]
Got:
    [x + 170269749119, y + 2149906854, z + 170335012540, 282687803443]
********************************************************************** 

Attachments (1)

trac_5027.patch (869 bytes) - added by mabshoff 14 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 14 years ago by mabshoff

  • Owner changed from tbd to mabshoff
  • Status changed from new to assigned

After chatting with malb we decided to dot out the constant since it is the same GBasis.

Cheers,

Michael

Changed 14 years ago by mabshoff

comment:2 Changed 14 years ago by mabshoff

  • Summary changed from doctest failure for rings/polynomial/toy_d_basis.py to [with patch, needs review] doctest failure for rings/polynomial/toy_d_basis.py

comment:3 Changed 14 years ago by jsp

  • Summary changed from [with patch, needs review] doctest failure for rings/polynomial/toy_d_basis.py to [with patch, positive review] doctest failure for rings/polynomial/toy_d_basis.py

After applying the patch:

sage -t  "devel/sage/sage/rings/polynomial/toy_d_basis.py"  
	 [9.5 s]
 
----------------------------------------------------------------------
All tests passed!
Total time for all tests: 9.5 seconds
[jaap@paix sage-3.3.alpha4]$ 

On fedora 9, 32 bits.

Jaap

comment:4 Changed 14 years ago by mabshoff

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

Merged in Sage 3.3.alpha6.

Cheers,

Michael

Note: See TracTickets for help on using tickets.