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

py3: make all tests pass #26212

Closed
fchapoton opened this issue Sep 7, 2018 · 437 comments
Closed

py3: make all tests pass #26212

fchapoton opened this issue Sep 7, 2018 · 437 comments

Comments

@fchapoton
Copy link
Contributor

This ticket is used to keep track of the progress towards python3.

As this ticket is too long, final steps are moved to #28298

release files having failing doctests more than 100 from 10 to 100 from 1 to 10 failing doctests
8.4.b3 915 42 251 622 17972
8.4.b4 803 23 175 605 10921
8.4.b5 657 17 145 495 8509
8.4.b6 594 17 127 450 7706
8.4.b7 587 17 128 442 7678
8.4.rc0 524 16 119 389 6976
8.5.b0 461 6 94 361 4206
8.5.b1 414 3 76 335 3206
8.5.b2 370 1 66 303 2465
8.5.b3 339 1 62 276 2245
8.5.b4 312 0 58 254 1947
8.5.b5 295 0 51 244 1741
8.5.b6 264 0 43 221 1514
8.5.rc0 252 0 39 213 1393
8.6.b0 234 0 30 204 1163
8.6.b1 231 0 30 201 1146
8.6.rc0 232 0 28 204 1124
8.6.rc1 236 0 30 206 1136
8.7.b0 240 1 38 201 1605
8.7.b1 200 0 23 177 866
8.7.b2 198 0 24 174 863
8.7.b3 166 0 20 146 717
8.7.b4 165 0 19 146 703
8.7.b5 143 0 12 131 562
8.7.b6 137 0 8 129 464
8.7.b7 122 0 8 114 447
8.7.rc0 105 0 7 98 384
8.8.b0 102 0 7 95 370

Now with python 3.7.3:

release files having failing doctests more than 100 from 10 to 100 from 1 to 10 failing doctests
8.8.b1 105 0 6 99 382
8.8.b2 94 0 6 88 324
8.8.b3 82 0 6 76 282
8.8.b4 77 0 2 75 246
8.8.b5 56 0 2 54 173
8.8.b6 49 0 1 48 151
8.8.b7 44 0 1 43 136
8.8.rc0 32 0 0 32 92
8.9.b0 22 0 0 22 71
8.9.b2 18 0 0 18 58
8.9.b3 15 0 0 15 47
8.9.b4 12 0 0 12 33
8.9.b5 8 0 0 8 18
8.9.b7 5 0 0 5 15
8.9.b9 4 0 0 4 14
8.9.rc0 3 0 0 3 7

Folders where all tests pass

Fragile folders (still fails on the reference machine)

These doctests pass when run on each file alone.

┗ /attach.py  # 4
┗ backends/generic_backend.pyx  # 2
┗ singular/function.pyx  # 1
┗ eclib/interface.py # 1 (line 597)
  • FLINT issue in src/sage/rings/polynomial/polynomial_rational_flint.pyx

Folders with work in progress

┣ non_symmetric_macdonald_polynomials.py  # 1
┣ weyl_characters.py  # 3
┗ weyl_group.py  # 7
see above for the remaining issues
┗ test.py  # 2
┣ nested_class.pyx  # 9
┗ expression.pyx  # 1 hash + 2 flaky in lines 11633 and 11635
┣ graphtheory_doctest.py  # 1
┗ sol/graphtheory_doctest.py  # 1

CC: @vinklein @embray @jm58660 @timokau

Component: python3

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

@fchapoton fchapoton added this to the sage-8.4 milestone Sep 7, 2018
@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@jhpalmieri

This comment has been minimized.

@jhpalmieri

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@vinklein

This comment has been minimized.

@vinklein

This comment has been minimized.

@vinklein

This comment has been minimized.

@vinklein

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@jhpalmieri

This comment has been minimized.

@mwageringel

This comment has been minimized.

@jplab

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton
Copy link
Contributor Author

comment:393

status on 8.9.beta4: remains 12 files with a total of 33 failing doctests:

'src/sage/combinat/root_system/non_symmetric_macdonald_polynomials.py  # 1 doctest failed',
 'src/sage/libs/singular/function.pyx  # 1 doctest failed',
 'src/sage/schemes/hyperelliptic_curves/hyperelliptic_finite_field.py  # 1 doctest failed',
 'src/sage/symbolic/expression.pyx  # 1 doctest failed',
 'src/sage/tests/books/computational-mathematics-with-sagemath/graphtheory_doctest.py  # 1 doctest failed',
 'src/sage/tests/books/computational-mathematics-with-sagemath/sol/graphtheory_doctest.py  # 1 doctest failed',
 'src/sage/doctest/test.py  # 2 doctests failed',
 'src/sage/numerical/backends/generic_backend.pyx  # 2 doctests failed',
 'src/sage/combinat/root_system/weyl_characters.py  # 3 doctests failed',
 'src/sage/repl/attach.py  # 4 doctests failed',
 'src/sage/combinat/root_system/weyl_group.py  # 7 doctests failed',
 'src/sage/misc/nested_class.pyx  # 9 doctests failed'

@dcoudert
Copy link
Contributor

comment:394

Yes, the 8 last failing doctests in generic_graph.py have been fixed with #27232. So, all doctests in the standard graphs module are now fixed !!!

We still have some failing doctests in optional packages like plantri #28108. I have now yet checked all other optional packages. Do we have a meta-ticket to record them ?

@jhpalmieri
Copy link
Member

comment:395

Is this a new one? All tests in schemes are supposed to pass.

'src/sage/schemes/hyperelliptic_curves/hyperelliptic_finite_field.py  # 1 doctest failed'

@fchapoton
Copy link
Contributor Author

comment:396

I think I have seen that already in previous betas, maybe since 8.9.beta3 or before.

EDIT: it was already there in 8.9.b0 (see above)

@jhpalmieri
Copy link
Member

comment:397

Is there a ticket for it? What's the error? (I don't see it on my OS X machines.)

@vinklein
Copy link
Mannequin

vinklein mannequin commented Jul 30, 2019

comment:398

I don't have it either on ubuntu 16.04.

@fchapoton
Copy link
Contributor Author

comment:399

There is no ticket for this error in schemes. The exact error can be seen in the patchbot log:

https://patchbot.sagemath.org/log/0/Ubuntu/16.04/x86_64/4.4.0-148-generic/atlas/2019-07-29%2008:49:23?short

This patchtbot is running on an ubuntu 16.04, using gcc 8.3.0.

Maybe we should start a new ticket for the final polishing ? This one is very long already.

@jhpalmieri
Copy link
Member

comment:400

Replying to @fchapoton:

Maybe we should start a new ticket for the final polishing ? This one is very long already.

Good idea. It's great seeing the progress!

@fchapoton

This comment has been minimized.

@fchapoton
Copy link
Contributor Author

comment:401

I have created #28298 for the few remaining final steps.

@fchapoton
Copy link
Contributor Author

comment:402

LARGE SCALE REGRESSION in 8.9.b5, with 241 failing doctests !!!!

EDIT: This could have been caused by a wrong version of gcc being used. Will retry.

@fchapoton

This comment has been minimized.

@jhpalmieri
Copy link
Member

comment:403

Replying to @fchapoton:

LARGE SCALE REGRESSION in 8.9.b5, with 241 failing doctests !!!!

EDIT: This could have been caused by a wrong version of gcc being used. Will retry.

For what it's worth, on OS X I only see one new failure, due to #28225, a proposed fix at #28320.

@fchapoton

This comment has been minimized.

@fchapoton
Copy link
Contributor Author

comment:404

Replying to @fchapoton:

LARGE SCALE REGRESSION in 8.9.b5, with 241 failing doctests !!!!

EDIT: This could have been caused by a wrong version of gcc being used. Will retry.

This was a false alert, due to a wrong gcc. There are 18 failing doctests.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

@fchapoton

This comment has been minimized.

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

10 participants