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

Symbolic hurwitz_zeta function #15095

Closed
eviatarbach opened this issue Aug 24, 2013 · 13 comments
Closed

Symbolic hurwitz_zeta function #15095

eviatarbach opened this issue Aug 24, 2013 · 13 comments

Comments

@eviatarbach
Copy link
Contributor

There should be a symbolic hurwitz_zeta function to replace the current one, which can only do numerical evaluation with Maxima.

Depends on #15118

CC: @burcin

Component: symbolics

Author: Eviatar Bach

Branch/Commit: u/chapoton/15095 @ ebb1e08

Reviewer: Frédéric Chapoton

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

@eviatarbach eviatarbach added this to the sage-6.1 milestone Aug 24, 2013
@eviatarbach
Copy link
Contributor Author

Dependencies: #15118

@eviatarbach
Copy link
Contributor Author

comment:2

This ticket depends on #15118, since otherwise mpmath returns nonsensical values for singularities.

@eviatarbach
Copy link
Contributor Author

comment:3

Attachment: trac15095.patch.gz

@eviatarbach
Copy link
Contributor Author

comment:4

Patchbot apply trac15095.patch​

@fchapoton
Copy link
Contributor

Branch: u/chapoton/15095

@fchapoton
Copy link
Contributor

comment:5

I have made a git branch.


New commits:

ff52c79Trac #15095: Symbolic hurwitz_zeta function
805fc1dtrac #15095 minor import changes

@fchapoton
Copy link
Contributor

Commit: 805fc1d

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 6, 2014

Changed commit from 805fc1d to ebb1e08

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jan 6, 2014

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

ebb1e08trac #15095 minor review patch

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:7

Ok, looks good to me. If you are happy with my small changes, you can set a positive review (and fill you name as Author).

@eviatarbach
Copy link
Contributor Author

comment:8

Looks good, as long as those imports weren't needed. Thank you for reviewing!

@eviatarbach
Copy link
Contributor Author

Author: Eviatar Bach

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