Opened 5 years ago

Closed 4 years ago

#25471 closed defect (fixed)

OEIS update (database format change, new entries, incorrect warning handling)

Reported by: Sébastien Labbé Owned by:
Priority: major Milestone: sage-8.3
Component: combinatorics Keywords: days94
Cc: Merged in:
Authors: Thierry Monteil Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: 9889222 (Commits, GitHub, GitLab) Commit: 9889222e7915e75d3a0ebe4a6e156e3b8e207087
Dependencies: Stopgaps:

Status badges

Description (last modified by Thierry Monteil)

This ticket fixes the 34 doctest failures that happen when internet is enabled, see this sage-release thread.

There are 3 kinds or errors:

  • the oeis database gets updated with new data (this is a normal update),
  • the oeis database changed its internal format, and does not make difference between signed and unsigned sequences anymore (in particular, for signed sequences, the V,W,X fields disapear, and are replaced with the S,T,U fields),
  • the warnings triggered when a dead sequence is met were not correctly doctested.

The 3 commits corresponds to these 3 kind of errors

To test the branch:

sage -t --optional=sage,internet src/sage/databases/oeis.py 

Change History (17)

comment:1 Changed 5 years ago by Sébastien Labbé

Summary: sage -t --optional=sage,internet src/sage/databases/oeis.py has 34 doctests failed34 doctests failed in databases/oeis.py

shorter title

comment:2 Changed 5 years ago by Sébastien Labbé

Summary: 34 doctests failed in databases/oeis.py34 internet doctests failed in databases/oeis.py

comment:3 Changed 5 years ago by Sébastien Labbé

Description: modified (diff)

comment:4 Changed 5 years ago by Sébastien Labbé

Description: modified (diff)

comment:5 Changed 5 years ago by Sébastien Labbé

Description: modified (diff)

comment:6 Changed 5 years ago by Thierry Monteil

Hi,

i will be on this (some might be tricky, e.g. related to dead sequences).

You write:

Note that, as Sloane writes this week on the ​arxiv, the oeis is a now a Wiki which means that Sage doctests should depend as few as possible on the many modifications done on the OEIS database...

Are you suggesting to parse the wiki ???

Note that the issue of the database changing was discussed in previous tickets. Still, we should show part of the answers for having meaningful examples (not only for testing purpose, but also for the user).

comment:7 Changed 5 years ago by John Palmieri

By the way, several of us predicted this sort of situation (lots of broken doctests related to OEIS) in #16252.

comment:8 Changed 4 years ago by Thierry Monteil

Keywords: days94 added

comment:9 Changed 4 years ago by Thierry Monteil

Branch: u/tmonteil/34_internet_doctests_failed_in_databases_oeis_py

comment:10 Changed 4 years ago by Thierry Monteil

Branch: u/tmonteil/34_internet_doctests_failed_in_databases_oeis_py

comment:11 Changed 4 years ago by Thierry Monteil

Branch: u/tmonteil/oeis_update

comment:12 Changed 4 years ago by git

Commit: cf7a0aa9e1627db79a332def992da45fab7a5f0d

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

cf7a0aamerge

comment:13 Changed 4 years ago by Thierry Monteil

Branch: u/tmonteil/oeis_update
Commit: cf7a0aa9e1627db79a332def992da45fab7a5f0d

comment:14 Changed 4 years ago by Thierry Monteil

Branch: u/tmonteil/oeis_update

comment:15 Changed 4 years ago by Thierry Monteil

Authors: Thierry Monteil
Commit: 9889222e7915e75d3a0ebe4a6e156e3b8e207087
Description: modified (diff)
Status: newneeds_review
Summary: 34 internet doctests failed in databases/oeis.pyOEIS update (database format change, new entries, incorrect warning handling)

New commits:

e081bc5Trac 25471 : fix the 20 trivial upstream changes
6f15c06Trac 25471 : the OEIS database format changed and does not make difference between signed and unsigned sequences anymore
9889222Trac 25471 : fix warnings for dead sequences

comment:16 Changed 4 years ago by Frédéric Chapoton

Reviewers: Frédéric Chapoton
Status: needs_reviewpositive_review

ok (but this will need to be done again and again...)

comment:17 Changed 4 years ago by Volker Braun

Branch: u/tmonteil/oeis_update9889222e7915e75d3a0ebe4a6e156e3b8e207087
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.