#17658: Incorrect densification of polynomial matrix
-------------------------------------+-------------------------------------
       Reporter:  gagern             |        Owner:
           Type:  defect             |       Status:  new
       Priority:  critical           |    Milestone:  sage-6.5
      Component:  linear algebra     |   Resolution:
       Keywords:                     |    Merged in:
        Authors:                     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/gagern/ticket/17658              |  84f7248720bb3ab274ed97ac00aa0bc56c072924
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by gagern):

 * commit:   => 84f7248720bb3ab274ed97ac00aa0bc56c072924


Comment:

 Found the problem: ETuple won't convert to regular tuple, and won't have
 the same hash either. For this reason, we should make sure that our dict
 keys are regular tuples. Preferably of Python integers.

 I left the doctest with the method I used to observe this problem. If you
 would prefer it in the module-level docstring of the
 `matrix_generic_sparse.pyx` file, feel free to move it there or tell me
 that I should move it. If you have some other place to suggest, let me
 know.
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=84f7248720bb3ab274ed97ac00aa0bc56c072924
 84f7248]||{{{Convert keys of generic sparse matrix to pair of ints.}}}||

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