Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hash collisions for derivatives of symbolic functions - act 3 #7508

Closed
burcin opened this issue Nov 21, 2009 · 7 comments
Closed

hash collisions for derivatives of symbolic functions - act 3 #7508

burcin opened this issue Nov 21, 2009 · 7 comments

Comments

@burcin
Copy link

burcin commented Nov 21, 2009

Reported by Alex Raichev on sage-support:

----------------------------------------------------------------------
| Sage Version 4.2.1, Release Date: 2009-11-14                       |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: X= var('x,y,z')
sage: f= function('f',*X); f
f(x, y, z)
sage: d= {}
sage: for l in [1..2]:
....:     for s in UnorderedTuples(X,l):
....:         print diff(f,s)
....:         d[diff(f,s)]= 69
....:
D[0](f)(x, y, z)
D[1](f)(x, y, z)
D[2](f)(x, y, z)
D[0, 0](f)(x, y, z)
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call
last)
...
/Applications/sage/local/lib/python2.6/site-packages/sage/symbolic/
expression_conversions.py in derivative(self, ex, operator)
    344             NotImplementedError: derivative
    345         """
--> 346         raise NotImplementedError, "derivative"  
    347
    348     def arithmetic(self, ex, operator):

NotImplementedError: derivative

This is another form of the problem I couldn't fix in #6243 and #6851.

Component: symbolics

Keywords: pynac

Author: Burcin Erocal

Reviewer: Karl-Dieter Crisman

Merged: sage-4.3.rc0

Issue created by migration from https://trac.sagemath.org/ticket/7508

@burcin burcin added this to the sage-4.3 milestone Nov 21, 2009
@burcin burcin self-assigned this Nov 21, 2009
@burcin
Copy link
Author

burcin commented Nov 22, 2009

add doctests

@burcin
Copy link
Author

burcin commented Nov 22, 2009

Author: Burcin Erocal

@burcin
Copy link
Author

burcin commented Nov 22, 2009

comment:1

Attachment: trac_7508-fderivative_hash_collision_doctest.patch.gz

This is fixed (hopefully, for good) in the new pynac package here:

http://sage.math.washington.edu/home/burcin/pynac/pynac-0.1.10.spkg

attachment: trac_7508-fderivative_hash_collision_doctest.patch adds doctests for the fix.

Note that the new pynac version also contains fixes for #7264 and #7406. Tests should be run with the patches from those tickets also applied in this order:

This ticket now depends on #7490.

@kcrisman
Copy link
Member

kcrisman commented Dec 5, 2009

comment:2

Positive review.

@kcrisman
Copy link
Member

kcrisman commented Dec 5, 2009

comment:3

I should point out that #7264 has a problem, so the spkg should not be merged until that is resolved.

@mwhansen
Copy link
Contributor

Merged: sage-4.3.rc0

@mwhansen
Copy link
Contributor

Reviewer: Karl-Dieter Crisman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants