Opened 9 years ago
Last modified 6 weeks ago
#16477 needs_info enhancement
implement Dirichlet series
Reported by: | rws | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | |
Component: | number theory | Keywords: | moebius, zeta, sigma, euler_phi, euler, Dirichlet series |
Cc: | kcrisman, mkamalakshya, jonhanke, slelievre | Merged in: | |
Authors: | Jonathan Hanke, Ralf Stephan | Reviewers: | Kamalakshya Mahatab, Vincent Delecroix, Jeroen Demeyer, Jonas Jermann, Ralf Stephan |
Report Upstream: | N/A | Work issues: | |
Branch: | public/ticket/16477 (Commits, GitHub, GitLab) | Commit: | eb2006785b5c456048ba4773fd58be652a409d5c |
Dependencies: | Stopgaps: |
Description (last modified by )
kcrisman in comment:20:ticket:8383:
Did somebody say defining Dirichlet series? Here is an implementation that I haven't had time to try out but which might be a good basis for that. This sage-support thread may also be relevant, though I don't know how advanced that psage code is.
The following file lists some product forms and their corresponding generating functions, it should be used to pick doctests for the g.f. input part of this ticket. The resp. OEIS entries are partly conjectural. https://drive.google.com/file/d/0B4PmRyK1JXgHY29aa00zZnhib1k/
Followup: #18102
Attachments (1)
Change History (85)
comment:1 Changed 9 years ago by
Cc: | kcrisman added |
---|
comment:2 Changed 9 years ago by
Description: | modified (diff) |
---|
Changed 9 years ago by
Attachment: | Dirichlet_series.sage added |
---|
comment:3 Changed 9 years ago by
comment:4 Changed 9 years ago by
Milestone: | sage-6.3 → sage-6.4 |
---|
comment:5 Changed 8 years ago by
Cc: | mkamalakshya added |
---|
comment:6 Changed 8 years ago by
Branch: | → u/mkamalakshya/16477 |
---|---|
Commit: | → c0037b25193d56e87d03eabd9db705f60ef7d3d9 |
New commits:
c0037b2 | Made the patch in to branch.
|
comment:7 Changed 8 years ago by
Milestone: | sage-6.4 → sage-wishlist |
---|
comment:8 Changed 8 years ago by
See also how to do it in Pari. Indeed, it has multiplication of Dirichlet series and so forth. Maybe that could be used somehow, since it already exists and is presumably well-tested.
comment:9 Changed 8 years ago by
This is not correct. The commit author (contrary to the committer) should show as Jonathan Hanke
. This is done by using the --author
option of git ci
. I will change that.
comment:10 Changed 8 years ago by
Branch: | u/mkamalakshya/16477 → public/dirichlet-series |
---|
comment:11 Changed 8 years ago by
Authors: | → Jonathan Hanke |
---|---|
Cc: | jonhanke added |
Commit: | c0037b25193d56e87d03eabd9db705f60ef7d3d9 → cfcd3aa3626df3f54cbbfa37b21083dcb4976758 |
Status: | new → needs_review |
I also moved the file under modular/
.
New commits:
cfcd3aa | 16477: original draft of Dirichlet series
|
comment:12 Changed 8 years ago by
Status: | needs_review → needs_work |
---|
comment:13 Changed 8 years ago by
Commit: | cfcd3aa3626df3f54cbbfa37b21083dcb4976758 → d1f4d30faa64ca38c2fbdd9c728e39bffc60d01a |
---|
Branch pushed to git repo; I updated commit sha1. New commits:
d1f4d30 | 16477: adapt to Sage; fix most doctest failures
|
comment:14 Changed 8 years ago by
Authors: | Jonathan Hanke → Jonathan Hanke, Ralf Stephan |
---|---|
Dependencies: | → #18038 |
Work issues: | → use pari, g.f. input |
comment:15 Changed 8 years ago by
Dependencies: | #18038 → #18038, #18041 |
---|
comment:16 Changed 8 years ago by
About PARI: dirmul
and dirdiv
already work in Sage. direuler
is difficult because of the syntax direuler(p=a,b,expr)
involving a loop variable and an expression to be evaluated.
comment:17 Changed 8 years ago by
Commit: | d1f4d30faa64ca38c2fbdd9c728e39bffc60d01a → 949082ca407de7df7ae2ce31ecfad4f5d21f3ffa |
---|
comment:18 Changed 8 years ago by
Description: | modified (diff) |
---|
comment:19 Changed 8 years ago by
Hi
Just an idea:
Shouldn't it be possible to define/implement Dirichlet series by specifying a function (from Z/ideals/whatever) for their coefficients?
I guess that would require some base class (which ideally would accept/parse/recognize functions from Z/whatever) for the coefficient functions. It would do parsing tasks, printing(!) of functions, arithmetic operations/dirichlet convolutions/etc... (Completely) multiplicative functions would be a "nice" subclass. Ideally the implementation would at least support basic Dirichlet series example.
Again: Just an idea (for either an extension or a new implementation/ticket).
Regards, Jonas
comment:20 follow-up: 23 Changed 8 years ago by
You are quite right - so, does Pari support this? If not, then it's probably a fairly big job and should be a different ticket (while you should make sure that this ticket remains agnostic with respect to your questions so the backend can be changed). If yes, hopefully we could just hook into their stuff for anything nontrivial.
comment:21 Changed 8 years ago by
At the moment this is implemented:
sage: Lminus4 = dirichlet_series([kronecker(-4,n) for n in range(1,21)]); Lminus4 1 + -1/(3^s) + 1/(5^s) + -1/(7^s) + 1/(9^s) + -1/(11^s) + 1/(13^s) + -1/(15^s) + 1/(17^s) + -1/(19^s) + O(21^(-s)) sage: zeta = dirichlet_series([1]*20); zeta 1 + 1/(2^s) + 1/(3^s) + 1/(4^s) + 1/(5^s) + 1/(6^s) + 1/(7^s) + 1/(8^s) + 1/(9^s) + 1/(10^s) + 1/(11^s) + 1/(12^s) + 1/(13^s) + 1/(14^s) + 1/(15^s) + 1/(16^s) + 1/(17^s) + 1/(18^s) + 1/(19^s) + 1/(20^s) + O(21^(-s)) sage: zeta * Lminus4^(-1) 1 + 1/(2^s) + 2/(3^s) + 1/(4^s) + 2/(6^s) + 2/(7^s) + 1/(8^s) + 2/(9^s) + 2/(11^s) + 2/(12^s) + 2/(14^s) + 1/(16^s) + 2/(18^s) + 2/(19^s) + O(21^(-s))
Please give a concrete example what you like to see!
comment:22 Changed 8 years ago by
I just found this: https://github.com/williamstein/psage/blob/master/psage/lseries/eulerprod.py#L1128
EDIT: This has much more generality and lets this ticket look like a child's toy, so there 8P
comment:23 Changed 8 years ago by
comment:24 follow-up: 26 Changed 8 years ago by
rws:
I'm in particular hoping for an exact implementation of arithmetic/multiplicative functions:
id = ArithmeticFunctions().id() sigma0 = ArithmeticFunction(lambda n: sigma0(n)) id*id == sigma0 // Dirichlet convolution
...
f = DirichletSeries(id)*DirichletSeries(id) f == DirichletSeries(sigma0) f[91234154] f // nice representations: => sum_{n in Z} (sum_{d|n} 1) or: sum_{n in Z} sigma0(n)
...
Those certainly form a nice subset of possible coefficient functions / dirichlet series. More generally I was hoping for the possibility to specify Dirichlet series / coefficient functions exactly, not only up to some precision. E.g. (bad example) DirichletSeries?(lambda n: sin(n)) or e.g. by specifying an (exact) generating series as e.g. a rational function
Calculating the n'th coefficient from these to get a DirichletSeries? as now is at least rather simple.
Other ideas: exact verification of identities (not only up to precision), nice representations/output of coefficient functions after operations (hard), support functions from ideals (TODO: figure out the correct category for the domain of coefficient functions).
I am aware that this is not a small task and might not be easily possible in python. But I hope you get the idea of what I had in mind...
comment:25 Changed 8 years ago by
That is probably a bit beyond this ticket, though maybe not. It would be nice to have such "symbolic" ones too.
comment:26 follow-up: 27 Changed 8 years ago by
Replying to jj:
f = DirichletSeries(id)*DirichletSeries(id) f == DirichletSeries(sigma0) f[91234154] f // nice representations: => sum_{n in Z} (sum_{d|n} 1) or: sum_{n in Z} sigma0(n)nice representations/output of coefficient functions after operations (hard),
That would first need conversion of the g.f. into divisor sum form. Another ticket.
id = ArithmeticFunctions().id() sigma0 = ArithmeticFunction(lambda n: sigma0(n)) id*id == sigma0 // Dirichlet convolution
That would need ability to compute a g.f. from the coefficients. I think this is possible but hard.
support functions from ideals (TODO: figure out the correct category for the domain of coefficient functions).
And a third ticket.
So this ticket would prepare for the others by providing creation from expressions of a limited form (polynomial fractions with generator from zeta(a*s+b)
and L(c)
).
comment:27 follow-up: 28 Changed 8 years ago by
Replying to rws:
Replying to jj:
f = DirichletSeries(id)*DirichletSeries(id) f == DirichletSeries(sigma0) f[91234154] f // nice representations: => sum_{n in Z} (sum_{d|n} 1) or: sum_{n in Z} sigma0(n)nice representations/output of coefficient functions after operations (hard),
That would first need conversion of the g.f. into divisor sum form. Another ticket.
There are (at least) two different (exact) approaches:
- Coefficient functions
- Generating functions (as q-series?), I am not entirely sure what you mean by it...
I was emphasizing on the coefficient function part. Clearly it is possible to define dirichlet series by specifying the coefficient function (as a normal function). It is even relatively simple to implement all kind of operations. Individual coefficients could also be calculated (up to some precision). What is hard is getting a "nice representation" (e.g. a divisor sum form as you mentioned above).
- One idea might be to somehow "decompile" a function into source code / some at least nicer form.
- One idea might be to use symbolic expression (rather limited).
- Another idea might be to implement very basic coefficient functions and allow all kind of operations with them (using ast(?)/whatever) to be as flexible as possible with the combinations (include as many of the popular examples as possible).
- Finally we could simply use the old representation by just calculating the coefficients up to some precision. The advantage would be that "precision" is no longer part of the series, only of the representation.
id = ArithmeticFunctions().id() sigma0 = ArithmeticFunction(lambda n: sigma0(n)) id*id == sigma0 // Dirichlet convolutionThat would need ability to compute a g.f. from the coefficients. I think this is possible but hard.
The relation between coefficient functions and generating series might be a challenge. I wonder what approach is more suitable. Maybe the two concepts both have interesting cases that cannot easily be translated to the other way. So maybe both ways should be supported (?).
support functions from ideals (TODO: figure out the correct category for the domain of coefficient functions).
And a third ticket.
So this ticket would prepare for the others by providing creation from expressions of a limited form (polynomial fractions with generator from
zeta(a*s+b)
andL(c)
).
I was thinking in very general terms ("any" function from Z). That should be fairly simple (even with operations on series) and allow the calculation of the coefficients up to an arbitrary precision. Example:
(F+G).cf = lambda n: F.cf(n) + G.cf(n) (F*G).cf = lambda n: sum(F.cf(d)*G.cf(n/d) for d in divisors(n))
However it will be very hard to get "nice" representation of the coefficient functions.
Maybe "polynomial fractions with genertore from zeta(a*s+b) and L(c)" is more powerful, I don't know.
Side remark: *Dirichlet series* could/should be viewed as something like a *Fourier transform* of an *Arithmetic function*.
comment:28 follow-up: 29 Changed 8 years ago by
Replying to jj:
- Generating functions (as q-series?), I am not entirely sure what you mean by it...
See https://en.wikipedia.org/wiki/Generating_function#Dirichlet_series_generating_functions
comment:29 follow-up: 30 Changed 8 years ago by
Replying to rws:
Replying to jj:
- Generating functions (as q-series?), I am not entirely sure what you mean by it...
See https://en.wikipedia.org/wiki/Generating_function#Dirichlet_series_generating_functions
Yes but that doesn't explain how the g.f. is implemented (in an exact way). For some families of Dirichlet series the corresponding ordinary (or exponential) generating series could be stored as a rational function (e.g. 1/(1-q)).
I assumed you meant this when refering to g.f. How else could you store the g.f. in an exact matter?
comment:30 Changed 8 years ago by
Replying to jj:
How else could you store the g.f. in an exact matter?
Nearly all D.g.f.'s I have seen (and certainly all that generate your arithmetic functions) are polynomials in A. zeta(as+b)
, B. L(chi(c,d), s)
, C. (1-e^(-s+f))
, with a,b,c,d,e,f integer. Okay, this is not a proof, but well supported by data in the OEIS. So I want to store that polynomial.
comment:31 follow-up: 32 Changed 8 years ago by
I agree that both of these representations are useful and desired. Would it be possible to implement the basics here - esp. since it turns out Pari has a lot of the functionality we need - and then put the generating function stuff (which I would also use) in a separate ticket? I'm just concerned that this will never happen despite a lot of enthusiasm.
comment:32 Changed 8 years ago by
Unfortunately experience shows that this concern is well founded. So yes, I agree.
comment:33 follow-up: 34 Changed 8 years ago by
Just for your information:
I started some preliminary (poc) test implementation of arithmetic functions (no tests/documentation etc). The first idea was to use symbolic expressions but they don't work well together with sums which are absolutely crucial for operations on arithmetic functions...
The test implementation is available on u/jj/arith_functions:
http://git.sagemath.org/sage.git/commit/?id=526e1ae40f94cff95e5dc6926208b831643edfb4
Here is a simple example:
sigma3 = ArithmeticFunction(lambda n: sigma(n,3), lambda n,_: "sigma({},3)".format(n), lambda n,_: "\\sigma_{{3}}\\left({}\\right)".format(n)) sigma3^2 show(sigma3^2) (sigma3^2)(5)
comment:34 follow-up: 35 Changed 8 years ago by
Replying to jj:
I started some preliminary (poc) test implementation of arithmetic functions (no tests/documentation etc).
http://git.sagemath.org/sage.git/commit/?id=526e1ae40f94cff95e5dc6926208b831643edfb4
I think you should open an enhancement ticket for this branch, just to make it possible to discuss specifics. Note: I am supporting your effort of formulating the algebraic side of these functions. Note too that Dirichlet series represent a superset of these functions, so your code cannot serve as the parent ring implementation of this ticket, and a different ticket is needed.
comment:35 Changed 8 years ago by
Ok, I created a ticket: http://trac.sagemath.org/ticket/18102
Regarding Dirichlet series: It seems that rational functions (over what?) might provide a nice (internal) basis for how (exact) Dirichlet series are stored. The coefficients can be calculated and it should be possible to automatically define a corresponding coefficient function. This would still benefit from a nice base class for arithmetic functions (in particular since we know to construct/represent the coefficient function, this additional information could be "passed on" to / used by the arithmetic function element).
comment:36 Changed 8 years ago by
Branch: | public/dirichlet-series → u/rws/16477-1 |
---|
comment:37 Changed 8 years ago by
Commit: | 949082ca407de7df7ae2ce31ecfad4f5d21f3ffa → 5698ef17c2be57ebe3826737cdba43f12c6bb8d6 |
---|---|
Dependencies: | #18038, #18041 |
Milestone: | sage-wishlist → sage-6.6 |
Status: | needs_work → needs_review |
This version now supports generating functions of a certain form. Please see the documentation. This should suffice as a first version. Further development possibilities were already discussed in this ticket.
Please review.
New commits:
05c8cba | Merge branch 'develop' into tmp9
|
5698ef1 | 16477: adapt code to Sage; rewrite to allow generated series
|
comment:38 follow-up: 39 Changed 8 years ago by
Status: | needs_review → needs_work |
---|
Hello,
- there is no need to add method
dirmul
anddirdiv
to the parigen
object. This is taken care automatically since #17631. Doctesting is cool thoug. But move it somewhere else.
- Why is this new module in
sage.modular
? It makes no sense.
- You should try harder to use
pari
and notgp
. Since #17631, most functions are available directly inpari
.
DirichletSeries
would better inherit fromRingElement
. In that case, you must not define__add__
,__mul__
,__div__
etc but_add_
,_mul_
,_div_
. That way you will get benefits from coercion (e.g. multiplication by scalars). You might have a look to polynomials (insage.rings.polynomials.*
or power series (insage.rings.power_series_*
).
- What is the purpose of the commented code?
- The error must be new style ones. Please use
raise MyError("my message")
instead ofraise MyError, "my message"
Vincent
comment:39 follow-up: 40 Changed 8 years ago by
Replying to vdelecroix:
Thanks, I accept points 1 and 6 but:
- Why is this new module in
sage.modular
? It makes no sense.
The Dirichlet characters are there too. I don't see why not.
- You should try harder to use
pari
and notgp
. Since #17631, most functions are available directly inpari
.
But not those needing closures, see comment:16 and #18038
DirichletSeries
would better inherit fromRingElement
. In that case, you must not define__add__
,__mul__
,__div__
etc but_add_
,_mul_
,_div_
. That way you will get benefits from coercion (e.g. multiplication by scalars). You might have a look to polynomials (insage.rings.polynomials.*
or power series (insage.rings.power_series_*
).
This is clearly a separate ticket.
- What is the purpose of the commented code?
I didn't want to erase useful ideas of the original author.
comment:40 follow-up: 41 Changed 8 years ago by
Hello,
Replying to rws:
Replying to vdelecroix:
- Why is this new module in
sage.modular
? It makes no sense.The Dirichlet characters are there too. I don't see why not.
Because sage.modular
is intended for modular forms. A Dirichlet serie is not necessarily a modular form. To me, modules and submodules should work by inclusion. Or at least try to.
I do not see Dirichlet series as a subset of modular forms. I perfectly understand the reason why it is here, but it would be better in sage.functions
or sage.rings
. The directory sage.modular
really contains too much stuff. This is historical as people that initiated Sage were very interested in modular forms.
- You should try harder to use
pari
and notgp
. Since #17631, most functions are available directly inpari
.But not those needing closures, see comment:16 and #18038
I see! Do you mind adding a comment in the docstring saying that as soon as #18038 is ready, this can be modified?
- What is the purpose of the commented code?
I didn't want to erase useful ideas of the original author.
Why letting it commented then?
Vincent
comment:41 follow-up: 42 Changed 8 years ago by
Replying to vdelecroix:
Hello,
Replying to rws:
Replying to vdelecroix:
- Why is this new module in
sage.modular
? It makes no sense.The Dirichlet characters are there too. I don't see why not.
Because
sage.modular
is intended for modular forms. A Dirichlet serie is not necessarily a modular form. To me, modules and submodules should work by inclusion. Or at least try to.I do not see Dirichlet series as a subset of modular forms. I perfectly understand the reason why it is here, but it would be better in
sage.functions
orsage.rings
. The directorysage.modular
really contains too much stuff. This is historical as people that initiated Sage were very interested in modular forms.
What do you think of sage.lfunctions
?
Vincent
comment:42 follow-up: 43 Changed 8 years ago by
Replying to vdelecroix:
What do you think of
sage.lfunctions
?
What else would you put in there (I have no idea myself)?
comment:43 Changed 8 years ago by
Replying to rws:
Replying to vdelecroix:
What do you think of
sage.lfunctions
?What else would you put in there (I have no idea myself)?
You can first have a look to what is already in.
comment:44 Changed 8 years ago by
I vaguely remember having had a look at some time but now I see you're right.
comment:45 follow-up: 46 Changed 8 years ago by
Replying to rws:
I vaguely remember having had a look at some time but now I see you're right.
Note that L-function / L-series usual refer to functions / series satisfying some very specific (more restrictive) set of axioms (AC/FE/etc) - in contrast to Dirichlet series or abstract Dirichlet series. So there is definitely some justification for introducing a new class (at least if it supports exact representation).
Also if I remember correctly the L-function is not an exact representation but also restricted by precision. What I would like to see is support for exact abstract Dirichlet series, e.g. by using symbolic coefficient functions or by using exact generating series in terms of rational functions.
If they are not implemented in an exact way I am curious about the background: How / in what context will they be used? Why is a new class required?
In any case I agree that Dirichlet series don't belong to sage.modular.
Best
Jonas
comment:46 Changed 8 years ago by
Hello Jonas,
Replying to jj:
Thanks for your input. After reading your comment I think that one reasonable option is to create a new module dirichlet_series
and move lfunctions
in there. What do you think?
Vincent
comment:47 follow-up: 48 Changed 8 years ago by
I'm not sure that would be as useful of a top-level module, and might break some existing code that imports them. I agree it would be confusing to have this in lfunctions
, but not as confusing as having it somewhere else!
comment:48 Changed 8 years ago by
Replying to kcrisman:
I'm not sure that would be as useful of a top-level module, and might break some existing code that imports them. I agree it would be confusing to have this in
lfunctions
, but not as confusing as having it somewhere else!
It is not a problem to keep the old lfunctions
module as an alias for dirichlet_series.lfunctions
(if we really care, there can be a deprecated alias). But for matter of code organisation that would be better to have lfunctions
included in dirichlet_series
.
Vincent
comment:49 Changed 8 years ago by
Commit: | 5698ef17c2be57ebe3826737cdba43f12c6bb8d6 → c595740c4b6e92da37f2ed90579f4a3f5664923b |
---|
comment:50 Changed 8 years ago by
Status: | needs_work → needs_review |
---|---|
Work issues: | use pari, g.f. input |
That addresses all discussed issues save the new module, which IMHO should be a separate ticket and discussed on sage-devel.
comment:51 Changed 8 years ago by
Hello,
Thanks for moving it! You need to add an entry to your new file to get it included in the documentation (this is one line in $SAGE_ROOT/doc/en/reference/lfunctions/index.rst
).
Vincent
comment:52 follow-up: 54 Changed 8 years ago by
Status: | needs_review → needs_work |
---|---|
Work issues: | → documentation, elementary operations |
Hello,
6 more questions/issues.
- Why are you importing the alias
dirichlet_series
and notDirichletSeries
in the global namespace?
- How do I get the coefficients of the Dirichlet L functions obtained with the command
dirichlet_L(3,2,s)
?
- There are several issues with the documentation:
- you forgot
::
at line 107 (inDirichletSeries
class documentation). - the
INPUT
block does not follow the convention (see the developer guide, section Documentation String) - doctests are missing from many important functions (e.g.
base_ring
,is_Eulerian
,has_infinite_precision
,list
)
- you forgot
- In
__mul__
I do not understandR = self.base_ring() try: _ = RR(other)
Did you want to writeR(other)
? If so it is very wrong since if you multiply2.0
with a Dirichlet serie with rational coefficients you would end up with rational coefficients assage: QQ(3.2314) 16157/5000
You should really use coercion here (i.e. the item 4 in comment:38) or do stronger type check such asparent(other) is self.base_ring()
.
- This should work
sage: D1 = dirichlet_series([1,1,1]) sage: D1 + 1 Traceback (most recent call last): ...
(also related to item 4 in comment:38).
- What is the purpose of this nested
MaskFunction
class?
Vincent
comment:53 Changed 8 years ago by
Commit: | c595740c4b6e92da37f2ed90579f4a3f5664923b → 7f254972105d6555d1a1d209fe17e65beec557db |
---|
Branch pushed to git repo; I updated commit sha1. New commits:
7f25497 | 16477: improve +/*, documentation
|
comment:54 Changed 8 years ago by
Replying to vdelecroix:
- How do I get the coefficients of the Dirichlet L functions obtained with the command
dirichlet_L(3,2,s)
?
Analogous to polynomial coefficients, now added as example:
sage: D=dirichlet_series(dirichlet_L(3,2,s)) sage: D.list() [1, -1, 0, 1, -1, 0, 1, -1, 0, 1, -1, 0, 1, -1, 0, 1, -1, 0, 1, -1] sage: D.list(5) [1, -1, 0, 1, -1]
- In
__mul__
I do not understandR = self.base_ring() try: _ = RR(other)Did you want to writeR(other)
?
I haven't looked extensively at Jon's code as long as it worked.
You should really use coercion here (i.e. the item 4 in comment:38) or do stronger type check such as
parent(other) is self.base_ring()
.
- This should work
sage: D1 = dirichlet_series([1,1,1]) sage: D1 + 1 Traceback (most recent call last): ...
No, this is a feature request. With this ticket you can do as well:
sage: D1 = dirichlet_series([1,1,1]) sage: one = dirichlet_series(1) sage: D1 + one 2 + 1/(2^s) + 1/(3^s) + O(4^(-s))
- What is the purpose of this nested
MaskFunction
class?
As this is only of developer interest I added at l.156:
# the specific form of the factors returned by Maxima # (see ticket #18081) makes masking necessary arg = dirichlet_series._mask_exp_factors(arg)
The rest should be addressed with the latest commit.
comment:55 Changed 8 years ago by
Status: | needs_work → needs_review |
---|
comment:56 Changed 8 years ago by
Reviewers: | → Kamalakshya Mahatab, Vincent Delecroix, Jeroen Demeyer, Jonas Jermann |
---|
comment:57 Changed 8 years ago by
Description: | modified (diff) |
---|
comment:58 Changed 8 years ago by
Commit: | 7f254972105d6555d1a1d209fe17e65beec557db → ae323c3bd7dfb925191dba9239b434fc02c930a1 |
---|
comment:59 Changed 7 years ago by
Commit: | ae323c3bd7dfb925191dba9239b434fc02c930a1 → 8c2640114ced6b5a6ad82ed0486303034f0ea78a |
---|
comment:60 Changed 7 years ago by
Milestone: | sage-6.6 → sage-7.3 |
---|---|
Work issues: | documentation, elementary operations |
comment:61 Changed 7 years ago by
What is the status of review on this? I.e., what does the latest reviewer/author (rws) believe still needs review? I personally dearly wish for convolution/DP but I think I won't get that on this ticket and better to have this in than not.
comment:62 Changed 6 years ago by
Cc: | slelievre added |
---|---|
Keywords: | Dirichlet series added |
comment:63 Changed 6 years ago by
Branch: | u/rws/16477-1 → u/rws/16477-2 |
---|
comment:64 Changed 6 years ago by
Commit: | 8c2640114ced6b5a6ad82ed0486303034f0ea78a → f3ce7ad3e95642f3474e595dcadbd31cdd3db5ff |
---|---|
Milestone: | sage-7.3 → sage-8.0 |
Squashed, used lazy import, and completed coverage.
New commits:
f3ce7ad | 16477: implement Dirichlet series
|
comment:65 follow-up: 66 Changed 6 years ago by
Curious whether this supports e.g. a Dirichlet series from the Möbius function (I couldn't find an example like that, other than the evident one where you take the appropriate version of zeta - but that is a theorem, not a series).
It looks quite nice. Ralf, what piece of this would you say still needs review? After all, I think at least some of it you have in essence reviewed Jon Hanke's work. So it would be nice to know what would still need a good hard look.
Final note - I'm confused by the dummy
. Is that something from pari or Sage?
comment:66 follow-up: 67 Changed 6 years ago by
Replying to kcrisman:
Curious whether this supports e.g. a Dirichlet series from the Möbius function (I couldn't find an example like that, other than the evident one where you take the appropriate version of zeta - but that is a theorem, not a series).
I don't understand. The Dirichlet g.f. of the Moebius function is 1/zeta(s)
via Sum_{n >= 1} mu(n)/n^s = 1/zeta(s)
so you can get it via:
sage: var('s') s sage: dirichlet_series(1/zeta(s)) 1 + -1/(2^s) + -1/(3^s) + -1/(5^s) + 1/(6^s) + -1/(7^s) + 1/(10^s) + -1/(11^s) + -1/(13^s) + 1/(14^s) + 1/(15^s) + -1/(17^s) + -1/(19^s) + O(20^(-s)) sage: _.list() [1, -1, -1, 0, -1, 1, -1, 0, 0, 1, -1, 0, -1, 1, 1, 0, -1, 0, -1, 0]
It looks quite nice. Ralf, what piece of this would you say still needs review? After all, I think at least some of it you have in essence reviewed Jon Hanke's work. So it would be nice to know what would still need a good hard look.
The conversion from g.f. to the form Pari uses for generation is unpublished research. If I insist on a review it will never go into Sage so what do I say? I tested it on all known D.g.f.s collected from the OEIS. But there are surely future cases where it will fail.
Final note - I'm confused by the
dummy
. Is that something from pari or Sage?
It is only used internally for pattern matching. If there is a next version pattern matching will be done in Pynac.
comment:67 follow-up: 68 Changed 6 years ago by
Reviewers: | Kamalakshya Mahatab, Vincent Delecroix, Jeroen Demeyer, Jonas Jermann → Kamalakshya Mahatab, Vincent Delecroix, Jeroen Demeyer, Jonas Jermann, Ralf Stephan |
---|
Replying to rws:
Replying to kcrisman:
Curious whether this supports e.g. a Dirichlet series from the Möbius function (I couldn't find an example like that, other than the evident one where you take the appropriate version of zeta - but that is a theorem, not a series).
I don't understand. The Dirichlet g.f. of the Moebius function is
1/zeta(s)
viaSum_{n >= 1} mu(n)/n^s = 1/zeta(s)
so you can get it via:
What I meant was precisely that I knew you could do that, but one might want to just plug Moebius in if (say) one didn't know this fact. (And not just for finitely many values.) But apparently not, which is fine for now.
The conversion from g.f. to the form Pari uses for generation is unpublished research. If I insist on a review it will never go into Sage so what do I say? I tested it on all known D.g.f.s collected from the OEIS. But there are surely future cases where it will fail.
Sure. I just meant that since this is currently "needs review", what things would still need to be looked at? You can presumably vouch for quite a bit not due to your self.
comment:68 follow-up: 69 Changed 6 years ago by
Replying to kcrisman:
Sure. I just meant that since this is currently "needs review", what things would still need to be looked at? You can presumably vouch for quite a bit not due to your self.
I think it's ready but I can't set positive, can I?
comment:69 Changed 6 years ago by
Sure. I just meant that since this is currently "needs review", what things would still need to be looked at? You can presumably vouch for quite a bit not due to your self.
I think it's ready but I can't set positive, can I?
Typically not, unless you really didn't add anything beyond window-dressing. Which is why I'm asking what you did add - I guess a lot of the examples, right? Any actual code? (Since you squashed I guess that won't be visible in the log.) Then anyone (hopefully me) who is ready can just review the pieces that are "new since you", if that makes sense.
comment:70 Changed 6 years ago by
Jon's code was completely rewritten, so you would have to review it all.
comment:71 Changed 6 years ago by
Okay, that's a bigger job, but good for us to know. I didn't realize that. No, typically we don't review external Pari code, though, so no worries there.
comment:72 Changed 5 years ago by
Status: | needs_review → needs_work |
---|
Import arith errors from patchbot.
comment:73 Changed 5 years ago by
Commit: | f3ce7ad3e95642f3474e595dcadbd31cdd3db5ff → f1a1adac9dab287cd9e68a480d8e37914a3afc84 |
---|
comment:74 Changed 5 years ago by
Milestone: | sage-8.0 → sage-8.2 |
---|---|
Status: | needs_work → needs_review |
comment:75 Changed 4 years ago by
Branch: | u/rws/16477-2 → public/ticket/16477 |
---|---|
Commit: | f1a1adac9dab287cd9e68a480d8e37914a3afc84 → 9da86c7c3f81b216defe8d9f9a3c3ba665f8349b |
comment:76 follow-up: 77 Changed 4 years ago by
Status: | needs_review → needs_info |
---|
Why Dirichlet series derive from SageObject
? Don't you want the Dirichlet series to form an algebra?
comment:77 Changed 4 years ago by
Why Dirichlet series derive from
SageObject
? Don't you want the Dirichlet series to form an algebra?
Yes, since these are formal Dirichlet series, ideally they would be, see e.g. Shapiro Chapter 4.6.
comment:78 Changed 3 years ago by
See also #26098 which possibly might overlap this (or even supersede it?).
comment:79 Changed 19 months ago by
Commit: | 9da86c7c3f81b216defe8d9f9a3c3ba665f8349b → eb2006785b5c456048ba4773fd58be652a409d5c |
---|
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
eb20067 | 16477: implement Dirichlet series
|
comment:82 Changed 18 months ago by
Beware that there seem to be typos in the list given at https://drive.google.com/file/d/0B4PmRyK1JXgHY29aa00zZnhib1k/
For example,
(1+pX)/(1-X) A048250 sum of squfr. div. Z(s)Z(s-1)/Z(2s-1)
but A048250 has Dirichlet g.f.: zeta(s)*zeta(s-1)/zeta(2*s-2).
comment:83 Changed 18 months ago by
Here is a better reference: http://dx.doi.org/10.35834/mjms/1316032830
comment:84 Changed 6 weeks ago by
Milestone: | sage-8.2 |
---|
I've attached the relevant file as well.