Opened 6 years ago

Closed 6 years ago

#15095 closed enhancement (fixed)

Symbolic `hurwitz_zeta` function

Reported by: eviatarbach Owned by:
Priority: major Milestone: sage-6.1
Component: symbolics Keywords:
Cc: burcin Merged in:
Authors: Eviatar Bach Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: u/chapoton/15095 Commit: ebb1e08a233dba1221fb1a6dfbd881b999d14cff
Dependencies: #15118 Stopgaps:

Description

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

Attachments (1)

trac15095.patch​ (7.5 KB) - added by eviatarbach 6 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 6 years ago by eviatarbach

  • Cc burcin added

comment:2 Changed 6 years ago by eviatarbach

  • Dependencies set to #15118

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

Changed 6 years ago by eviatarbach

comment:3 Changed 6 years ago by eviatarbach

  • Status changed from new to needs_review

comment:4 Changed 6 years ago by eviatarbach

Patchbot apply trac15095.patch​

comment:5 Changed 6 years ago by chapoton

  • Branch set to u/chapoton/15095
  • Commit set to 805fc1d86dfd2423fac0044f99b11e85d8b4280c

I have made a git branch.

New commits:

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

comment:6 Changed 6 years ago by git

  • Commit changed from 805fc1d86dfd2423fac0044f99b11e85d8b4280c to ebb1e08a233dba1221fb1a6dfbd881b999d14cff

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

​ebb1e08trac #15095 minor review patch

comment:7 Changed 6 years ago by chapoton

  • Reviewers set to Frédéric Chapoton

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).

comment:8 Changed 6 years ago by eviatarbach

  • Authors set to Eviatar Bach
  • Status changed from needs_review to positive_review

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

comment:9 Changed 6 years ago by vbraun

  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.