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

5 internet doctests failing in findstat.py #26997

Closed
seblabbe opened this issue Jan 2, 2019 · 11 comments
Closed

5 internet doctests failing in findstat.py #26997

seblabbe opened this issue Jan 2, 2019 · 11 comments

Comments

@seblabbe
Copy link
Contributor

seblabbe commented Jan 2, 2019

On Ubuntu 16.04, running sage-8.5.beta1, the command

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

gives

...
3 items had failures:
   3 of  16 in sage.databases.findstat
   1 of   4 in sage.databases.findstat.FindStatStatistic.__repr__
   1 of   9 in sage.databases.findstat.FindStatStatistic._find_by_values
    [247 tests, 5 failures, 107.27 s]
----------------------------------------------------------------------
sage -t --long src/sage/databases/findstat.py  # 5 doctests failed
----------------------------------------------------------------------

Full failure log is long and is pasted here: https://framabin.org/p/?b9dacda8324c7295#71YTYesLKEKBCNg1LtnDUhKkWDrcm+xiZGDI9a8bjfg=

CC: @vinklein

Component: doctest coverage

Author: Martin Rubey

Branch: 6079bf0

Reviewer: Frédéric Chapoton

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

@seblabbe seblabbe added this to the sage-8.6 milestone Jan 2, 2019
@mantepse
Copy link
Contributor

mantepse commented Jan 2, 2019

comment:1

The problem is that the semantics of FancyTuple changed, and I think in a bad way:

sage: FancyTuple([(1,2,3),(4,5,6)])
0: 0: 1
1: 2
2: 3
1: 0: 4
1: 5
2: 6

sage: FancyTuple([[1,2,3],(4,5,6)])
0: [1, 2, 3]
1: 0: 4
1: 5
2: 6

I think this comes from #26704 but I didn't check.

@mantepse
Copy link
Contributor

mantepse commented Jan 2, 2019

@mantepse
Copy link
Contributor

mantepse commented Jan 2, 2019

comment:3

easy fix, according to the specification of FancyTuple.__getitem__.


New commits:

6079bf0fix `__getitem__` of FancyTuple to preserve tuple-items

@mantepse
Copy link
Contributor

mantepse commented Jan 2, 2019

Commit: 6079bf0

@mantepse
Copy link
Contributor

mantepse commented Jan 2, 2019

Author: Martin Rubey

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:6

seems to be ok on python3

@embray
Copy link
Contributor

embray commented Jan 15, 2019

comment:7

Retarging tickets optimistically to the next milestone. If you are responsible for this ticket (either its reporter or owner) and don't believe you are likely to complete this ticket before the next release (8.7) please retarget this ticket's milestone to sage-pending or sage-wishlist.

@embray embray modified the milestones: sage-8.6, sage-8.7 Jan 15, 2019
@vbraun
Copy link
Member

vbraun commented Jan 22, 2019

Changed branch from u/mantepse/5_internet_doctests_failing_in_findstat_py to 6079bf0

@seblabbe
Copy link
Contributor Author

comment:9

5 new other failing internet doctests in findstat at #27838

@seblabbe
Copy link
Contributor Author

Changed commit from 6079bf0 to none

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

5 participants