sagemath / sage

Main repository of SageMath
https://www.sagemath.org
Other
1.39k stars 473 forks source link

OEIS database install docstring is out of date #16599

Closed 9343d2e0-59ba-4406-bd4f-c78e4cf1230e closed 10 years ago

9343d2e0-59ba-4406-bd4f-c78e4cf1230e commented 10 years ago

At #10358, the function to install the database was updated to

def install(self, oeis_url="http://oeis.org/stripped.gz", names_url="http://oeis.org/names.gz", overwrite=False):
...

But the docstring still gives outdated defaults:

        INPUT:

        - ``oeis_url`` - string (default: "http://www.research.att.com...")
        ...

        - ``names_url`` - string (default: "http://www.research.att.com...")
        ...

So these urls should be corrected. This problem was noted on this ask.sagemath answer

CC: @nathanncohen @sagetrac-tmonteil

Component: combinatorics

Keywords: oeis

Author: Frédéric Chapoton

Branch/Commit: 1b35539

Reviewer: Niles Johnson

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

fchapoton commented 10 years ago

Branch: u/chapoton/16599

fchapoton commented 10 years ago

Author: Frédéric Chapoton

fchapoton commented 10 years ago

Commit: 1b35539

fchapoton commented 10 years ago

New commits:

1b35539trac #16599 changed obsolete http address
9343d2e0-59ba-4406-bd4f-c78e4cf1230e commented 10 years ago
comment:2

great -- thanks!

vbraun commented 10 years ago
comment:3

Reviewer name

9343d2e0-59ba-4406-bd4f-c78e4cf1230e commented 10 years ago

Reviewer: Niles Johnson

vbraun commented 10 years ago

Changed branch from u/chapoton/16599 to 1b35539