Opened 3 years ago
Last modified 21 months ago
#26212 closed task
py3: make all tests pass — at Version 98
Reported by: | chapoton | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | python3 | Keywords: | |
Cc: | vklein, embray, jmantysalo, gh-timokau | Merged in: | |
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
This ticket is used to keep track of the progress towards python3.
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 |
Folders where all tests pass (8.5.b1):
- algebras/steenrod ✔ #26219 and #26220
- arith #26208 ✔
- combinat/sf (#26220 +) #26215
- docs ✔ #26310
- features ✔
- finance #26213 ✔
- games ✔
- gsl
- groups/matrix_gps #26400 ✔
- knots ✔
- lfunctions ✔
- logic #26214 ✔
- media ✔
- monoids ✔ #26294
- parallel ✔
- probability #26216 ✔
- quadratic_forms/ #26224 ✔
- quivers #26211 ✔
- rings/number_field #26263
- sat #26543 ✔
- server ✔
- stats #26401 ✔
- typeset ✔
Folders with work in progress:
- doc/en #26346
- algebras #26316 #26399
- categories #26545
- combinat/species #26328
- combinat/words #26362, #26458, #26473, #26590
- combinat/finite_state_machine #26147, #26468
- combinat/subset #26510
- combinat #26535
- crypto #26398
- data_structures #24852
- functions #24758, #26591
- game_theory #26429
- graphs #26431 #26447
- groups/lie_gps #26518
- interfaces #26314
- libs #24804
- manifolds #26475 (regression !)
- matrix/ #26236 (very partial fix)
- misc #26312, #26390
- modular #26382
- modules #26276
- numerical/backends #24740 (for
char *
arguments management) and #24741 - plot #26210 (partial fix only) #26299 #26301 #26321
- rings (power series) #26539
- sandpiles #26016
- schemes/elliptic_curves #26536, #26529, #26570, #26594
- structure #25694
- tensor #26295, #26601
- tests/french_book #26495
Change History (99)
comment:1 Changed 3 years ago by
- Description modified (diff)
comment:2 Changed 3 years ago by
- Description modified (diff)
comment:3 Changed 3 years ago by
- Description modified (diff)
comment:4 Changed 3 years ago by
- Description modified (diff)
comment:5 Changed 3 years ago by
- Description modified (diff)
comment:6 Changed 3 years ago by
- Description modified (diff)
comment:7 Changed 3 years ago by
- Description modified (diff)
comment:8 Changed 3 years ago by
- Description modified (diff)
comment:9 Changed 3 years ago by
- Description modified (diff)
comment:10 Changed 3 years ago by
- Description modified (diff)
comment:11 Changed 3 years ago by
- Description modified (diff)
comment:12 Changed 3 years ago by
- Description modified (diff)
comment:13 Changed 3 years ago by
- Description modified (diff)
comment:14 Changed 3 years ago by
- Description modified (diff)
comment:15 Changed 3 years ago by
- Cc vklein added
comment:16 Changed 3 years ago by
- Description modified (diff)
comment:17 Changed 3 years ago by
- Description modified (diff)
comment:18 Changed 3 years ago by
- Description modified (diff)
comment:19 Changed 3 years ago by
- Description modified (diff)
comment:20 Changed 3 years ago by
- Description modified (diff)
comment:21 Changed 3 years ago by
- Description modified (diff)
comment:22 Changed 3 years ago by
- Description modified (diff)
comment:23 Changed 3 years ago by
- Description modified (diff)
comment:24 Changed 3 years ago by
- Description modified (diff)
comment:25 Changed 3 years ago by
- Description modified (diff)
comment:26 Changed 3 years ago by
- Description modified (diff)
comment:27 Changed 3 years ago by
- Description modified (diff)
comment:28 Changed 3 years ago by
- Description modified (diff)
comment:29 Changed 3 years ago by
- Description modified (diff)
comment:30 Changed 3 years ago by
- Description modified (diff)
comment:31 Changed 3 years ago by
- Description modified (diff)
comment:32 Changed 3 years ago by
- Description modified (diff)
comment:33 Changed 3 years ago by
- Description modified (diff)
comment:34 Changed 3 years ago by
- Description modified (diff)
comment:35 Changed 3 years ago by
- Description modified (diff)
comment:36 Changed 3 years ago by
- Description modified (diff)
comment:37 Changed 3 years ago by
- Description modified (diff)
comment:38 Changed 3 years ago by
- Description modified (diff)
comment:39 Changed 3 years ago by
- Description modified (diff)
comment:40 Changed 3 years ago by
- Description modified (diff)
comment:41 Changed 3 years ago by
- Description modified (diff)
comment:42 Changed 3 years ago by
- Description modified (diff)
comment:43 Changed 3 years ago by
- Description modified (diff)
comment:44 Changed 3 years ago by
- Description modified (diff)
comment:45 Changed 3 years ago by
- Description modified (diff)
comment:46 Changed 3 years ago by
- Description modified (diff)
comment:47 Changed 3 years ago by
- Description modified (diff)
comment:48 Changed 3 years ago by
- Description modified (diff)
comment:49 Changed 3 years ago by
- Description modified (diff)
comment:50 Changed 3 years ago by
- Description modified (diff)
comment:51 Changed 3 years ago by
- Description modified (diff)
comment:52 Changed 3 years ago by
- Description modified (diff)
comment:53 Changed 3 years ago by
- Description modified (diff)
comment:54 Changed 3 years ago by
- Description modified (diff)
comment:55 Changed 3 years ago by
- Description modified (diff)
comment:56 Changed 3 years ago by
- Description modified (diff)
comment:57 Changed 3 years ago by
- Description modified (diff)
comment:58 Changed 3 years ago by
- Description modified (diff)
comment:59 Changed 3 years ago by
- Description modified (diff)
comment:60 Changed 3 years ago by
- Description modified (diff)
comment:61 Changed 3 years ago by
- Description modified (diff)
comment:62 Changed 3 years ago by
- Description modified (diff)
comment:63 Changed 3 years ago by
- Description modified (diff)
comment:64 Changed 3 years ago by
- Description modified (diff)
comment:65 Changed 3 years ago by
- Description modified (diff)
comment:66 Changed 3 years ago by
- Description modified (diff)
comment:67 Changed 3 years ago by
- Description modified (diff)
comment:68 Changed 3 years ago by
- Cc embray added
comment:69 Changed 3 years ago by
- Description modified (diff)
comment:70 Changed 3 years ago by
- Description modified (diff)
comment:71 Changed 3 years ago by
- Description modified (diff)
comment:72 Changed 3 years ago by
- Description modified (diff)
comment:73 Changed 3 years ago by
- Description modified (diff)
comment:75 Changed 3 years ago by
- Description modified (diff)
comment:76 Changed 3 years ago by
- Description modified (diff)
comment:77 Changed 3 years ago by
The six remaining files with many failures are
['sage -t --long src/sage/combinat/finite_state_machine.py # 189 doctests failed', 'sage -t --long src/sage/dynamics/arithmetic_dynamics/projective_ds.py # 105 doctests failed', 'sage -t --long src/sage/graphs/generic_graph.py # 110 doctests failed', 'sage -t --long src/sage/groups/lie_gps/nilpotent_lie_group.py # 143 doctests failed', 'sage -t --long src/sage/rings/padics/padic_lattice_element.py # 167 doctests failed', 'sage -t --long src/sage/rings/valuation/limit_valuation.py # 106 doctests failed']
comment:78 Changed 3 years ago by
- Description modified (diff)
comment:79 Changed 3 years ago by
- Description modified (diff)
comment:80 follow-up: ↓ 84 Changed 3 years ago by
For whoever wants to work on it, regarding
sage -t --long src/sage/combinat/finite_state_machine.py # 189 doctests failed
about 150 of the failures can be cured by replacing 'iteritems' by 'items' in lines like
elif hasattr(data, 'iteritems'): # data is a dict (or something similar),
I don't know if 'items' is the right attribute to check, or if it would actually be better to change to isinstance(data, dict)
. What other data structures are mean to be acceptable? What is "something similar" to a dict?
comment:81 Changed 3 years ago by
- Description modified (diff)
comment:82 Changed 3 years ago by
- Description modified (diff)
comment:83 Changed 3 years ago by
- Description modified (diff)
comment:84 in reply to: ↑ 80 Changed 3 years ago by
Replying to jhpalmieri:
For whoever wants to work on it, regarding ... I don't know if 'items' is the right attribute to check, or if it would actually be better to change to
isinstance(data, dict)
. What other data structures are mean to be acceptable? What is "something similar" to a dict?
There is some custom "Mapping types" in sage like FreeModuleElement_generic_sparse
, which implements items
and iteritems
but doesn't inherit from dict
.
comment:85 Changed 3 years ago by
- Description modified (diff)
comment:86 Changed 3 years ago by
- Description modified (diff)
comment:87 Changed 3 years ago by
- Description modified (diff)
comment:88 Changed 3 years ago by
- Description modified (diff)
comment:89 Changed 3 years ago by
- Description modified (diff)
comment:90 Changed 3 years ago by
- Description modified (diff)
comment:91 Changed 3 years ago by
- Description modified (diff)
comment:92 Changed 3 years ago by
- Description modified (diff)
comment:93 Changed 3 years ago by
There was a regression in the manifolds folder (sigh)..
The 3 remaining files with more than 100 failing doctests are
['sage -t --long src/sage/combinat/finite_state_machine.py # 182 doctests failed', 'sage -t --long src/sage/graphs/generic_graph.py # 109 doctests failed', 'sage -t --long src/sage/rings/padics/padic_lattice_element.py # 167 doctests failed']
The first one is already tackled in #26147 at least partially.
comment:94 Changed 3 years ago by
- Description modified (diff)
comment:95 Changed 3 years ago by
Regarding
'sage -t --long src/sage/rings/padics/padic_lattice_element.py # 167 doctests failed'
I can reproduce those failures when I run ./sage -t ...
, but (at least for the first few) not when I run Sage interactively. Odd.
comment:96 Changed 3 years ago by
- Description modified (diff)
comment:97 Changed 3 years ago by
- Description modified (diff)
comment:98 Changed 3 years ago by
- Description modified (diff)
Hadn't even seen this ticket; very nice.