#14711: Weak references in the coercion graph
-------------------------------------+-------------------------------------
Reporter: jpflori | Owner: davidloeffler
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-6.1
Component: number fields | Resolution:
Keywords: memleak, number | Merged in:
field, QuadraticField | Reviewers: Jean-Pierre Flori
Authors: Simon King | Work issues:
Report Upstream: N/A | Commit:
Branch: | 0af59ea93689cb6abb9d3fae0f1cf11f2aee5cca
u/jpflori/ticket/14711 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by nbruin):
Replying to [comment:171 tscrim]:
> My 2 cents. I don't think that `coerce_map_from()` should return the map
which is used in the coercion system, but a copy of it.
Presently, that routine gets used internally as well. The internal routine
should of course NOT copy the map, so changing that would imply renaming
the present routine to an underscore method and writing a new wrapper that
does the copying ... and changing the internal code to use the new
underscore method.
> Also couldn't we use (some form of) #8878 get around some of the
backtracking issues?
This is outside the scope of this ticket. We would need to seriously
recalibrate the "coercion costs" to ensure that the "shortest path" would
indeed be a reasonable thing. Furthermore, a large part of the graph is
encoded programmatically, so not fully accessible for a backtracking
search. We could of course just trust the programmatic bits to yield
"shortest" paths, but those routines tend to do depth-first searches
themselves
--
Ticket URL: <http://trac.sagemath.org/ticket/14711#comment:172>
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/groups/opt_out.
