#14982: When a parent is equipped with an embedding, consider coercions that 
don't
go through the embedding
-------------------------------------+-------------------------------------
       Reporter:  mmezzarobba        |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.4
      Component:  coercion           |   Resolution:
       Keywords:  embedding          |    Merged in:
        Authors:  Marc Mezzarobba    |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/mmezzarobba/14982-coerce_embeddings|  
f32b52f4176684d57e4e6b83f80c2c553ffcc8eb
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by mmezzarobba):

 Replying to [comment:43 vdelecroix]:
 > In this comment
 > {{{
 > +            # As a consequence, a value of __an_element with the wrong
 class
 > +            # is cached during the call to has_coerce_map_from. We
 reset the
 > +            # cache afterwards.
 > }}}
 > you refer to `__an_element`. But you changed the attribute's name to
 `_cache_an_element`.

 Thanks, fixed.

 > The hack for quadratic number field is horrible...

 You are the one who introduced it, aren't you? `;-)` So I guess you
 understand what it is for better than I do. In any case I just tested the
 change you suggested (on top of my branch + the current master) and it
 doesn't seem to break anything.

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