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

Differentiation of conj/imag/real/abs functions #22209

Closed
rwst opened this issue Jan 19, 2017 · 14 comments
Closed

Differentiation of conj/imag/real/abs functions #22209

rwst opened this issue Jan 19, 2017 · 14 comments

Comments

@rwst
Copy link

rwst commented Jan 19, 2017

At the moment the result from differentiation of the functions conj, real, imag, abs does not take the argument domain into account.

sage: _ = var('x', domain='real')
sage: _ = var('z')

sage: (x^z).conjugate().diff(x)
x^(z - 1)*z*D[0](conjugate)(x^z)
sage: atan(x).imag_part().diff(x)
D[0](imag_part)(arctan(x))/(x^2 + 1)
sage: abs(log(z)).diff(z)
log(z)/(z*abs(log(z)))

GiNaC has functionality that is only now included in Pynac. The ticket should doctest it when it is implemented. It all benefits from recent domain logic improvements in Pynac as well.

pynac/pynac#45

Depends on #22219

Component: symbolics

Author: Ralf Stephan

Branch/Commit: 052e3cb

Reviewer: Travis Scrimshaw

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

@rwst rwst added this to the sage-7.6 milestone Jan 19, 2017
@rwst

This comment has been minimized.

@rwst
Copy link
Author

rwst commented Jan 20, 2017

@rwst
Copy link
Author

rwst commented Jan 20, 2017

Dependencies: pynac-0.7.4

@rwst
Copy link
Author

rwst commented Jan 20, 2017

Author: Ralf Stephan

@rwst

This comment has been minimized.

@rwst
Copy link
Author

rwst commented Jan 20, 2017

Commit: f96fe12

@rwst
Copy link
Author

rwst commented Jan 20, 2017

New commits:

f96fe1222209: Differentiation of conj/imag/real/abs functions

@rwst
Copy link
Author

rwst commented Feb 8, 2017

Changed dependencies from pynac-0.7.4 to #22219

@rwst
Copy link
Author

rwst commented Feb 8, 2017

comment:5

Trigger patchbot.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 20, 2017

Changed commit from f96fe12 to 052e3cb

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Feb 20, 2017

Branch pushed to git repo; I updated commit sha1. New commits:

052e3cbMerge branch 'develop' into t/22209/differentiation_of_conj_imag_real_abs_functions

@tscrim
Copy link
Collaborator

tscrim commented May 16, 2017

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented May 16, 2017

comment:7

LGTM.

@vbraun
Copy link
Member

vbraun commented May 18, 2017

Changed branch from u/rws/differentiation_of_conj_imag_real_abs_functions to 052e3cb

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