#26348 closed defect (fixed)
3 other internet doctests failing in findstat.py
Reported by:  slabbe  Owned by:  

Priority:  major  Milestone:  sage8.5 
Component:  doctest coverage  Keywords:  
Cc:  Merged in:  
Authors:  Martin Rubey  Reviewers:  Sébastien Labbé, Frédéric Chapoton 
Report Upstream:  N/A  Work issues:  
Branch:  d35a6e3 (Commits)  Commit:  d35a6e3b9ff5ea1cd332dbb67aacc52d6f82d8fc 
Dependencies:  Stopgaps: 
Description (last modified by )
As reported in sagerelease for 8.4.beta6, the command
sage tp long optional=sage,optional,external src/sage/databases/findstat.py
gives
sage t long src/sage/databases/findstat.py ********************************************************************** File "src/sage/databases/findstat.py", line 106, in sage.databases.findstat Failed example: r = findstat(Permutations, lambda pi: pi.saliances()[0]); r # optional  internet, random (St000051: The size of the left subtree. , [Mp00069: complement, Mp00061: to increasing tree], 24) Exception raised: Traceback (most recent call last): File "/home/slabbe/GitBox/sage/local/lib/python2.7/sitepackages/sage/doctest/forker.py", line 659, in _run self.compile_and_execute(example, compiler, test.globs) File "/home/slabbe/GitBox/sage/local/lib/python2.7/sitepackages/sage/doctest/forker.py", line 1068, in compile_and_execute compiled = compiler(example) File "/home/slabbe/GitBox/sage/local/lib/python2.7/sitepackages/sage/doctest/forker.py", line 635, in compiler compileflags, 1) File "<doctest sage.databases.findstat[7]>", line 3 (St000051: The size of the left subtree. , [Mp00069: complement, Mp00061: to increasing tree], Integer(24)) ^ SyntaxError: invalid syntax ********************************************************************** File "src/sage/databases/findstat.py", line 110, in sage.databases.findstat Failed example: (s, list_f, quality) = next((a,b,c) for a,b,c in r if a.id() == 51) # optional  internet Exception raised: Traceback (most recent call last): File "/home/slabbe/GitBox/sage/local/lib/python2.7/sitepackages/sage/doctest/forker.py", line 659, in _run self.compile_and_execute(example, compiler, test.globs) File "/home/slabbe/GitBox/sage/local/lib/python2.7/sitepackages/sage/doctest/forker.py", line 1070, in compile_and_execute exec(compiled, globs) File "<doctest sage.databases.findstat[8]>", line 1, in <module> (s, list_f, quality) = next((a,b,c) for a,b,c in r if a.id() == Integer(51)) # optional  internet StopIteration ********************************************************************** File "src/sage/databases/findstat.py", line 130, in sage.databases.findstat Failed example: print(list_f[0].code() + "\r\n" + list_f[1].code()) # optional  internet, random Exception raised: Traceback (most recent call last): File "/home/slabbe/GitBox/sage/local/lib/python2.7/sitepackages/sage/doctest/forker.py", line 659, in _run self.compile_and_execute(example, compiler, test.globs) File "/home/slabbe/GitBox/sage/local/lib/python2.7/sitepackages/sage/doctest/forker.py", line 1070, in compile_and_execute exec(compiled, globs) File "<doctest sage.databases.findstat[12]>", line 1, in <module> print(list_f[Integer(0)].code() + "\r\n" + list_f[Integer(1)].code()) # optional  internet, random IndexError: list index out of range ********************************************************************** 1 item had failures: 3 of 16 in sage.databases.findstat 7 webbrowser tests not run [247 tests, 3 failures, 72.88 s]
Change History (17)
comment:1 Changed 2 years ago by
 Description modified (diff)
comment:2 Changed 2 years ago by
comment:3 Changed 2 years ago by
 Branch set to u/mantepse/3_other_internet_doctests_failing_in_findstat_py
comment:4 Changed 2 years ago by
 Commit set to 6ed089bf7cde0d10f3eca170db4caa35b3ee306c
 Status changed from new to needs_review
New commits:
6ed089b  fix doctests, remove a few "random" tags

comment:5 Changed 2 years ago by
We should rather keep the #random, so that we do not have to change the doctests every week or so.
comment:6 Changed 2 years ago by
 Reviewers set to Sébastien Labbé
 Status changed from needs_review to positive_review
I suggest we update doctests and/or readd # random
tag later in another ticket if needed. Positive review.
comment:7 followup: ↓ 9 Changed 2 years ago by
 Status changed from positive_review to needs_work
oups, the plugins "doctest continuation" seems unhappy. And sometimes pyflakes
too.
comment:8 Changed 2 years ago by
pyflakes (when run using python3) warns about an exception class called StandardError? that has been removed in Python 3
comment:9 in reply to: ↑ 7 Changed 2 years ago by
Replying to slabbe:
oups, the plugins "doctest continuation" seems unhappy. And sometimes
pyflakes
too.
I do not understand this. It is complaining about the doctest which I just changed:
sage: r = findstat(Permutations, lambda pi: pi.saliances()[0]); r # optional  internet 0: ... ... (St000051: The size of the left subtree of a binary tree., [Mp00069: complement, Mp00061: to increasing tree], 1000) ... sage: (s, list_f, quality) = next((a,b,c) for a,b,c in r if a.id() == 51) # optional  internet
I can't see what's wrong with it  any ideas?
Concerning StandardError
, it seems that this should be replaced with Exception
.
comment:10 Changed 2 years ago by
The plugin is not smart. If you think it's ok, just go on.
comment:11 Changed 2 years ago by
Can I use or look at the plugin without setting up a patchbot?
comment:12 Changed 2 years ago by
The code is here: https://github.com/sagemath/sagepatchbot
Patchbot is not so easy to use. Some instructions for usage inside an ipython session are here at the bottom: https://wiki.sagemath.org/patchbot
comment:13 Changed 2 years ago by
 Commit changed from 6ed089bf7cde0d10f3eca170db4caa35b3ee306c to d35a6e3b9ff5ea1cd332dbb67aacc52d6f82d8fc
Branch pushed to git repo; I updated commit sha1. New commits:
d35a6e3  StandardError > Exception, no ... at beginning of line followed by whitespace

comment:14 Changed 2 years ago by
 Status changed from needs_work to needs_review
comment:15 Changed 2 years ago by
 Reviewers changed from Sébastien Labbé to Sébastien Labbé, Frédéric Chapoton
 Status changed from needs_review to positive_review
ok, even if I would have prefered to keep "# random"
comment:16 Changed 2 years ago by
 Branch changed from u/mantepse/3_other_internet_doctests_failing_in_findstat_py to d35a6e3b9ff5ea1cd332dbb67aacc52d6f82d8fc
 Resolution set to fixed
 Status changed from positive_review to closed
comment:17 Changed 2 years ago by
 Milestone changed from sage8.4 to sage8.5
This should be retargeted for 8.5.
The first one seems to indicate that the meaning of "..." has changed: it fails because the next line is interpreted as part of the doctest instead of part of the output.
The other failures are a consequence of the first.