#4539: plural wrapper
---------------------------+------------------------------------------------
   Reporter:  burcin       |       Owner:  OleksandrMotsak, AlexanderDreyer
       Type:  enhancement  |      Status:  needs_work                      
   Priority:  major        |   Milestone:  sage-4.7                        
  Component:  algebra      |    Keywords:  libsingular plural wrapper      
     Author:               |    Upstream:  N/A                             
   Reviewer:               |      Merged:                                  
Work_issues:               |  
---------------------------+------------------------------------------------

Comment(by jhpalmieri):

 It would be great to have this in Sage!

 I'm seeing some problems with the patch.  First, it doesn't apply cleanly
 to Sage 4.7.alpha1.  I haven't tried applying to 4.6.2.  I rebased it by
 hand.

 Second, the change
 {{{
 #!diff
 -                    block_name, block_length, _ =
 re.split(length_pattern,block)
 +                    block_name, block_length, _ = re.split(pattern,
 block.strip())
 }}}
 in term_order.py is problematic, because "pattern" is not defined
 anywhere.  Replacing it by "length_pattern" seems to work.

 Third, in multipolynomial_ideal.py, {{{_groebner_basis_libsingular}}} is
 being called with keywords "deg_bound" and "mult_bound", but it doesn't
 accept those keywords as valid arguments.  Should we add {{{*args,
 **kwds}}} to the argument list?    Or should those keywords be dealt with
 explicitly?  I tried adding generic {{{*args}}} and {{{**kwds}}}, but
 doctesting on that file segfaults.

 When I doctest the whole Sage library after making these changes, I get
 the following failures:
 {{{
 The following tests failed:

         sage -t  -long -force_lib
 devel/sage/sage/rings/polynomial/multi_polynomial_ideal.py #
 Killed/crashed
         sage -t  -long -force_lib
 devel/sage/sage/rings/polynomial/multi_polynomial_libsingular.pyx # 1
 doctests failed
         sage -t  -long -force_lib
 devel/sage/sage/libs/singular/polynomial.pyx # 1 doctests failed
         sage -t  -long -force_lib
 devel/sage/sage/rings/polynomial/plural.pyx # 6 doctests failed
 }}}
 On one machine, I also had this failure:
 {{{
         sage -t -long -force_lib
 devel/sage/sage/rings/polynomial/multi_polynomial.pyx # Killed/crashed
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4539#comment:41>
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