#9798: Accelerate Polyhedron constructor and fix cddlib output ordering
----------------------------+-----------------------------------------------
   Reporter:  vbraun        |       Owner:  mhampton  
       Type:  defect        |      Status:  needs_work
   Priority:  major         |   Milestone:  sage-4.6  
  Component:  geometry      |    Keywords:            
     Author:  Volker Braun  |    Upstream:  N/A       
   Reviewer:                |      Merged:            
Work_issues:                |  
----------------------------+-----------------------------------------------
Changes (by jhpalmieri):

  * status:  needs_review => needs_work


Comment:

 mark2 is still testing, but it shows one failure so far:
 {{{
 sage -t  -long devel/sage/sage/rings/polynomial/groebner_fan.py
 **********************************************************************
 File "/home/palmieri/mark2/sage-4.6.alpha2/devel/sage-
 main/sage/rings/polynomial/groebner_fan.py", line 1043:
     sage: cone_data = Polyhedron(ieqs = g_cone_ieqs, eqns =
 [[1,-1,-1,-1,-1]])
 Exception raised:
     Traceback (most recent call last):
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/ncadoctest.py",
 line 1231, in run_one_test
         self.run_one_example(test, example, filename, compileflags)
       File
 "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/sagedoctest.py", line 38,
 in run_one_example
         OrigDocTestRunner.run_one_example(self, test, example, filename,
 compileflags)
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/ncadoctest.py",
 line 1172, in run_one_example
         compileflags, 1) in test.globs
       File "<doctest __main__.example_40[7]>", line 1, in <module>
         cone_data = Polyhedron(ieqs = g_cone_ieqs, eqns =
 [[Integer(1),-Integer(1),-Integer(1),-Integer(1),-Integer(1)]])###line
 1043:
     sage: cone_data = Polyhedron(ieqs = g_cone_ieqs, eqns =
 [[1,-1,-1,-1,-1]])
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/lib/python/site-
 packages/sage/geometry/polyhedra.py", line 1536, in __init__
         if self.ambient_dim() < len(Polyhedron._render_method):
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/lib/python/site-
 packages/sage/geometry/polyhedra.py", line 2629, in ambient_dim
         return self._ambient_dim
     AttributeError: 'Polyhedron' object has no attribute '_ambient_dim'
 **********************************************************************
 File "/home/palmieri/mark2/sage-4.6.alpha2/devel/sage-
 main/sage/rings/polynomial/groebner_fan.py", line 1044:
     sage: cone_lines = gf._4d_to_3d(cone_data)
 Exception raised:
     Traceback (most recent call last):
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/ncadoctest.py",
 line 1231, in run_one_test
         self.run_one_example(test, example, filename, compileflags)
       File
 "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/sagedoctest.py", line 38,
 in run_one_example
         OrigDocTestRunner.run_one_example(self, test, example, filename,
 compileflags)
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/ncadoctest.py",
 line 1172, in run_one_example
         compileflags, 1) in test.globs
       File "<doctest __main__.example_40[8]>", line 1, in <module>
         cone_lines = gf._4d_to_3d(cone_data)###line 1044:
     sage: cone_lines = gf._4d_to_3d(cone_data)
     NameError: name 'cone_data' is not defined
 **********************************************************************
 File "/home/palmieri/mark2/sage-4.6.alpha2/devel/sage-
 main/sage/rings/polynomial/groebner_fan.py", line 1045:
     sage: cone_lines
 Exception raised:
     Traceback (most recent call last):
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/ncadoctest.py",
 line 1231, in run_one_test
         self.run_one_example(test, example, filename, compileflags)
       File
 "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/sagedoctest.py", line 38,
 in run_one_example
         OrigDocTestRunner.run_one_example(self, test, example, filename,
 compileflags)
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/ncadoctest.py",
 line 1172, in run_one_example
         compileflags, 1) in test.globs
       File "<doctest __main__.example_40[9]>", line 1, in <module>
         cone_lines###line 1045:
     sage: cone_lines
     NameError: name 'cone_lines' is not defined
 **********************************************************************
 File "/home/palmieri/mark2/sage-4.6.alpha2/devel/sage-
 main/sage/rings/polynomial/groebner_fan.py", line 1083:
     sage: three_d = gf.render3d()
 Exception raised:
     Traceback (most recent call last):
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/ncadoctest.py",
 line 1231, in run_one_test
         self.run_one_example(test, example, filename, compileflags)
       File
 "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/sagedoctest.py", line 38,
 in run_one_example
         OrigDocTestRunner.run_one_example(self, test, example, filename,
 compileflags)
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/bin/ncadoctest.py",
 line 1172, in run_one_example
         compileflags, 1) in test.globs
       File "<doctest __main__.example_41[4]>", line 1, in <module>
         three_d = gf.render3d()###line 1083:
     sage: three_d = gf.render3d()
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/lib/python/site-
 packages/sage/rings/polynomial/groebner_fan.py", line 1106, in render3d
         cone_info = [Polyhedron(ieqs = q, eqns = [[1,-1,-1,-1,-1]]) for q
 in g_cones_ieqs]
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/lib/python/site-
 packages/sage/geometry/polyhedra.py", line 1536, in __init__
         if self.ambient_dim() < len(Polyhedron._render_method):
       File "/home/palmieri/mark2/sage-4.6.alpha2/local/lib/python/site-
 packages/sage/geometry/polyhedra.py", line 2629, in ambient_dim
         return self._ambient_dim
     AttributeError: 'Polyhedron' object has no attribute '_ambient_dim'
 **********************************************************************
 2 items had failures:
    3 of  10 in __main__.example_40
    1 of   7 in __main__.example_41
 ***Test Failed*** 4 failures.
 }}}
 I get the same error on fulvia (Solaris on x86), so it's not something
 wrong with the build on mark2.

 (I installed the new cddlib spkg and also applied the patch here.  Was
 there anything else I was supposed to do?)

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9798#comment:37>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
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