#15156: Pickling with __reduce__() loops creates invalid pickles
-------------------------------------------------+-------------------------
       Reporter:  vbraun                         |        Owner:
           Type:  defect                         |       Status:
       Priority:  major                          |  needs_info
      Component:  misc                           |    Milestone:  sage-6.3
       Keywords:                                 |   Resolution:
        Authors:  SimonKing, jkeitel, novoselt,  |    Merged in:
  nbruin                                         |    Reviewers:
Report Upstream:  N/A                            |  Work issues:
         Branch:                                 |       Commit:
   Dependencies:                                 |     Stopgaps:
-------------------------------------------------+-------------------------
Changes (by saraedum):

 * status:  new => needs_info


Comment:

 `pickle.dumps` seems to detect such invalid references. I added it to
 `_test_pickle` and it detects the following:
 {{{
 sage -t src/sage/geometry/fan_morphism.py  # 1 doctest failed
 sage -t src/sage/combinat/affine_permutation.py  # 3 doctests failed
 sage -t src/sage/combinat/root_system/ambient_space.py  # 1 doctest failed
 sage -t src/sage/modular/abvar/homspace.py  # 1 doctest failed
 sage -t src/sage/combinat/root_system/type_affine.py  # 1 doctest failed
 sage -t src/sage/tests/cmdline.py  # 3 doctests failed
 sage -t src/sage/combinat/root_system/weight_space.py  # 2 doctests failed
 sage -t src/sage/combinat/root_system/root_space.py  # 1 doctest failed
 sage -t src/sage/modules/free_module.py  # 5 doctests failed
 sage -t src/sage/combinat/rigged_configurations/rigged_configurations.py
 # 3 doctests failed
 sage -t src/sage/combinat/alternating_sign_matrix.py  # 1 doctest failed
 sage -t src/sage/combinat/gelfand_tsetlin_patterns.py  # 6 doctests failed
 sage -t
 src/sage/combinat/rigged_configurations/rigged_configuration_element.py  #
 2 doctests failed
 sage -t src/sage/quivers/homspace.py  # 1 doctest failed
 sage -t src/sage/combinat/integer_list.py  # 1 doctest failed
 sage -t
 
src/sage/algebras/finite_dimensional_algebras/finite_dimensional_algebra_morphism.py
 # 1 doctest failed
 sage -t src/sage/combinat/root_system/cartan_type.py  # 1 doctest failed
 }}}

 I pushed one commit which contains an example on how I think, the problem
 can be fixed.

 Is this how this problem could be addressed? What do you think?

--
Ticket URL: <http://trac.sagemath.org/ticket/15156#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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to