#15959: The hash function of combinatorial free module elements should depend on
the parent
-------------------------------------+-------------------------------------
       Reporter:  nthiery            |        Owner:
           Type:  enhancement        |       Status:  new
       Priority:  major              |    Milestone:  sage-6.2
      Component:  linear algebra     |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Travis Scrimshaw   |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/tscrim/kyoto_fix-15959           |  7bdb017e3e37d52a89d29e262d0b96f3fe831b48
   Dependencies:  #15931             |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by {'newvalue': u'Travis Scrimshaw', 'oldvalue': ''}):

 * commit:   => 7bdb017e3e37d52a89d29e262d0b96f3fe831b48
 * dependencies:   => #15931
 * branch:   => u/tscrim/kyoto_fix-15959
 * author:   => Travis Scrimshaw


Comment:

 I haven't merged in the latest #15931, but instead started with the
 a2bbe7f297 commit. The fix for the Kyoto path model is that I was passing
 in elements of the weight space, whereas the `Epsilon()/Phi()` methods
 returned an element of the weight lattice. I've added an additional check
 for robustness.
 ----
 New commits:
 
||[http://git.sagemath.org/sage.git/commit/?id=e8fe5eb8943debdb457c38d3264167d66e7b2b14
 e8fe5eb]||{{{Trac 15931: implement a proper hash function for
 (combinatorial) free module elements}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=e6ef605116edd69eb8630d80fc731eeae0316a25
 e6ef605]||{{{Trac 15931: add a @cached_method on the new hash function for
 (combinatorial) free module elements}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=a2bbe7f297caeb73bda1f8ce31af9ee21b90a26b
 a2bbe7f]||{{{Trac 15931: include the hash of the parent in the new hash
 function for (combinatorial) free module elements}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=13be6205f2aeaa0dd6658f96286f5eafbc99addf
 13be620]||{{{Merge branch 'develop' into test/15959}}}||
 
||[http://git.sagemath.org/sage.git/commit/?id=7bdb017e3e37d52a89d29e262d0b96f3fe831b48
 7bdb017]||{{{Fix for Kyoto path model with hash depending on the
 parent.}}}||

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