#9907: Move generally usable decorators from plot.misc to misc.decorators
---------------------------+------------------------------------------------
   Reporter:  jsrn         |       Owner:  tdb                   
       Type:  enhancement  |      Status:  needs_work            
   Priority:  minor        |   Milestone:                        
  Component:  relocation   |    Keywords:  generality, decorators
     Author:  jsrn         |    Upstream:  N/A                   
   Reviewer:               |      Merged:                        
Work_issues:               |  
---------------------------+------------------------------------------------
Changes (by jsrn):

  * status:  needs_review => needs_work


Comment:

 Rob Beezer posted the following problem with the current patch for Trac
 #6094, but that is actually a problem with the current patch for this
 trac. When testing the unpickle-function in sage.structure.sage_object,
 the relocation of the decorators yield problems:

 {{{
 r...@wave:/sage/dev$ ./sage -t  devel/sage/sage/structure/sage_object.pyx
 sage -t  "devel/sage/sage/structure/sage_object.pyx"
 **********************************************************************
 File "/sage/dev/devel/sage/sage/structure/sage_object.pyx", line 1001:
     sage: print "x"; sage.structure.sage_object.unpickle_all(std)
 Expected:
     x...
     Successfully unpickled ... objects.
     Failed to unpickle 0 objects.
 Got:
     x
     doctest:1: DeprecationWarning: Your word object is saved in an old
 file format since FiniteWord_over_OrderedAlphabet is deprecated and will
 be deleted in a future version of Sage (you can use FiniteWord_list
 instead). You can re-save your word by typing "word.save(filename)" to
 ensure that it will load in future versions of Sage.
     doctest:1: DeprecationWarning: Your word object is saved in an old
 file format since AbstractWord is deprecated and will be deleted in a
 future version of Sage (you can use FiniteWord_list instead). You can re-
 save your word by typing "word.save(filename)" to ensure that it will load
 in future versions of Sage.
     doctest:1: DeprecationWarning: Your word object is saved in an old
 file format since Word_over_Alphabet is deprecated and will be deleted in
 a future version of Sage (you can use FiniteWord_list instead). You can
 re-save your word by typing "word.save(filename)" to ensure that it will
 load in future versions of Sage.
     doctest:1: DeprecationWarning: Your word object is saved in an old
 file format since Word_over_OrderedAlphabet is deprecated and will be
 deleted in a future version of Sage (you can use FiniteWord_list instead).
 You can re-save your word by typing "word.save(filename)" to ensure that
 it will load in future versions of Sage.
     doctest:1: DeprecationWarning: ChristoffelWord_Lower is deprecated,
 use LowerChristoffelWord instead
     ** failed:  _class__sage_plot_misc_options__.sobj
     ** failed:  _class__sage_plot_misc_rename_keyword__.sobj
     Failed:
     _class__sage_plot_misc_options__.sobj
     _class__sage_plot_misc_rename_keyword__.sobj
     Successfully unpickled 584 objects.
     Failed to unpickle 2 objects.
 **********************************************************************
 1 items had failures:
    1 of   7 in __main__.example_23
 ***Test Failed*** 1 failures.
 For whitespace errors, see the file
 /home/rob/.sage//tmp/.doctest_sage_object.py
          [5.2 s]

 ----------------------------------------------------------------------
 The following tests failed:


         sage -t  "devel/sage/sage/structure/sage_object.pyx"
 }}}

 The deprecation-warnings are just garbage from words.py, so the important
 thing to note is the "** failed"-lines. The problem, as far as I can see,
 is simply that there is a standard pickle jar which has a pickle for all
 the functions etc. shipped with sage. This needs to be updated when
 applying this patch. Does anyone know how this is done and possibly
 included in the patch?

 Cheers, Johan

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