#16239: ZZ in the wrong category
-------------------------------------+-------------------------------------
       Reporter:  tscrim             |        Owner:  sage-combinat
           Type:  defect             |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.3
      Component:  categories         |   Resolution:
       Keywords:  integer ring,      |    Merged in:
  category                           |    Reviewers:  Nicolas M. ThiƩry
        Authors:  Travis Scrimshaw   |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:                     |  8f67fb521e4a1baf0a78a65a1b4359698b3bf872
  public/categories/fix_ZZ-16239     |     Stopgaps:
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by tscrim):

 Replying to [comment:24 vdelecroix]:
 > It looks ok for me if you modify the doctest explaining what happened.
 For me it is a facade issue as a facade should be childless (i.e. there is
 no point in testing whether an element has a parent a given facade).

 It's not checking that it is a child of a facade parent, but that the
 facade parents `__call__` (and the `_element_constructor_`?) doesn't
 change the element. I'm wondering if facade parents should work with the
 coercion framework differently...

 Well, for now I will just add the warning to the doctest unless anyone
 objects.

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