#4386: [with patch, needs work] Sage 3.1.4: optional doctest failure in
sage/rings/number_field/totallyreal_phc.py
----------------------+-----------------------------------------------------
 Reporter:  mabshoff  |        Owner:  mhampton
     Type:  defect    |       Status:  new     
 Priority:  major     |    Milestone:  sage-3.2
Component:  doctest   |   Resolution:          
 Keywords:            |  
----------------------+-----------------------------------------------------
Changes (by mabshoff):

  * summary:  Sage 3.1.4: optional doctest failure in
              sage/rings/number_field/totallyreal_phc.py =>
              [with patch, needs work] Sage 3.1.4: optional
              doctest failure in
              sage/rings/number_field/totallyreal_phc.py

Comment:

 Craig's patch exposes the next problem:
 {{{
 **********************************************************************
 File "/scratch/mabshoff/release-
 cycle/sage-3.1.3.final/tmp/totallyreal_phc.py", line 87:
     sage: __lagrange_bounds_phc(3,5,[8,1,2,0,1],tmpfile='phc') # optional
 Exception raised:
     Traceback (most recent call last):
       File "/scratch/mabshoff/release-
 cycle/sage-3.1.3.final/local/lib/python2.5/doctest.py", line 1228, in
 __run
         compileflags, 1) in test.globs
       File "<doctest __main__.example_2[3]>", line 1, in <module>
 
__lagrange_bounds_phc(Integer(3),Integer(5),[Integer(8),Integer(1),Integer(2),Integer(0),Integer(1)],tmpfile='phc')
 # optional###line 87:
     sage: __lagrange_bounds_phc(3,5,[8,1,2,0,1],tmpfile='phc') # optional
       File "/scratch/mabshoff/release-
 cycle/sage-3.1.3.final/local/lib/python2.5/site-
 packages/sage/rings/number_field/totallyreal_phc.py", line 118, in
 __lagrange_bounds_phc
         for P in sage.combinat.combinat.partitions_list(n-1,m-1):
     AttributeError: 'module' object has no attribute 'partitions_list'
 **********************************************************************
 }}}
 mhansen pointed out in IRC:
 {{{
 [9:13pm] mhansen: partitions_list should be deprecated.
 Partitions(n).list()
 should be used if you want a list otherwise Partitions(n) provides an
 __iter__
 }}}

 Cheers,

 Michael

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4386#comment:1>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to