Ticket #8430 (closed defect: fixed)
doctest failures with Sage 4.3.4.alpha0
| Reported by: | mvngu | Owned by: | hivert |
|---|---|---|---|
| Priority: | major | Milestone: | sage-4.3.4 |
| Component: | doctest coverage | Keywords: | |
| Cc: | sage-combinat | Work issues: | |
| Report Upstream: | N/A | Reviewers: | Marshall Hampton |
| Authors: | Florent Hivert | Merged in: | sage-4.3.4.alpha1 |
| Dependencies: | Stopgaps: |
Description (last modified by mvngu) (diff)
From sage-devel:
* The following tests failed on sage.math: sage -t -long local/lib/python2.6/site-packages/sagenb-0.7.5.1-py2.6.egg/sagenb/notebook/interact.py # 1 doctests failed sage -t -long local/lib/python2.6/site-packages/sagenb-0.7.5.1-py2.6.egg/sagenb/misc/sageinspect.py # 1 doctests failed sage -t -long devel/sage/sage/categories/finite_semigroups.py # 2 doctests failed sage -t -long devel/sage/sage/categories/examples/finite_semigroups.py # 1 doctests failed
The doctest
sage -t -long devel/sage/sage/categories/finite_semigroups.py # 2 doctests failed
also failed on SPARC Solaris 10 (the machine t2.math). The failure uncovered another bug relating to kpsewhich. See ticket #8445.
Attachments
Change History
comment:1 Changed 3 years ago by mvngu
- Summary changed from three doctest failures with Sage 4.3.4.alpha0 to doctest failures with Sage 4.3.4.alpha0
comment:2 Changed 3 years ago by jhpalmieri
I've tried to address the problem with sageinspect.py in ticket #8324.
comment:3 Changed 3 years ago by jhpalmieri
Note that the problem with sagenb/notebook/interact.py should be fixed by the sagenb patch at ticket #5601.
comment:4 in reply to: ↑ description Changed 3 years ago by hivert
- Cc sage-combinat added
- Status changed from new to needs_review
- Authors set to Florent Hivert
- The following tests failed on sage.math:
sage -t -long devel/sage/sage/categories/finite_semigroups.py # 2 doctests failed sage -t -long devel/sage/sage/categories/examples/finite_semigroups.py # 1 doctests failed
Those last two should be fixed by the submitted patch. A better fix together with improvements is being written on sage-combinat but I don't know how long finishing those improvements will take. So please review the patch.
By the way, should'nt I create a ticket for this patch ? It is certainly orthogonal to the two other one doctest failure.
Cheers,
Florent
comment:5 follow-up: ↓ 7 Changed 3 years ago by mhampton
Since the other issues have there own tickets I don't think another one is necessary.
There is one doctest failure still:
File "/Volumes/E/sage-4.3.4.alpha0/devel/sage-t1/sage/categories/finite_semigroups.py", line 229:
sage: sorted(S.j_transversal_of_idempotents())
Expected:
['a', 'ab', 'ac', 'acb', 'b', 'bc', 'c']
Got:
['a', 'ab', 'ac', 'acb', 'b', 'c', 'cb']
*********************************************
which is not covered by the patch. Seems like maybe the expected string is wrong, unless the sorted function has varying behavior.
comment:7 in reply to: ↑ 5 Changed 3 years ago by hivert
- Status changed from needs_work to needs_review
Replying to mhampton:
There is one doctest failure still:
File "/Volumes/E/sage-4.3.4.alpha0/devel/sage-t1/sage/categories/finite_semigroups.py", line 229:
sage: sorted(S.j_transversal_of_idempotents())
Expected:
['a', 'ab', 'ac', 'acb', 'b', 'bc', 'c']
Got:
['a', 'ab', 'ac', 'acb', 'b', 'c', 'cb']
*********************************************
which is not covered by the patch. Seems like maybe the expected string is wrong, unless the sorted function has varying behavior.
I'm not sure to understand:
- why I missed this one;
- why the result of sorted changed between 4.3.3 and 4.3.4;
- why despite of this change it seems to be stable on both version...
Anyway the newly uploaded patch passed the tests 10 times in a raw on the machine sage. Maybe it could be worth checking of it is also stable on other architectures.
Sorry for the mess and please review.
comment:9 Changed 3 years ago by mhampton
- Status changed from needs_review to positive_review
All tests in categories passed on a 10.6 mac and on 64-bit Ubuntu 9.10, so I think I can give this a positive review.
comment:10 Changed 3 years ago by mpatel
- Reviewers set to Marshall Hampton
SageNB 0.7.5.2 includes fixes for the first two failures in the description (see #8435).
An important note: It's best to test SageNB files with -force_lib, e.g.,
sage -t -long -force_lib local/lib/python2.6/site-packages/sagenb-0.7.5.1-py2.6.egg/sagenb/notebook/interact.py
Running sage -t -sagenb or make test and its friends will set it implicitly. If the option is set, SAGE_LOCAL/bin/sage-doctest will treat the file(s) as Sage library code, even if it does not live under SAGE_ROOT/devel. In particular, sage-doctest will not attempt the equivalent of from filebase import *, which can raise false errors or cause segfaults.
I suppose I should have called it -library_code, instead.
Please remember to fillin
comment:11 Changed 3 years ago by drkirkby
I'm not at home, and can't very easily start applying patches, but I can confirm I get the same results, based on Sage 4.3.4.alpha0 but with a few Solaris-specific patches on Solaris 10 (SPARC)
**********************************************************************
File "/export/home/drkirkby/32/sage-4.3.4.alpha0/devel/sage/sage/categories/fini
te_semigroups.py", line 232:
sage: sorted(S.j_transversal_of_idempotents())
Expected:
['a', 'ab', 'ac', 'acb', 'b', 'bc', 'c']
Got:
['a', 'ab', 'ac', 'acb', 'b', 'c', 'cb']
**********************************************************************
File "/export/home/drkirkby/32/sage-4.3.4.alpha0/devel/sage/sage/categories/fini
te_semigroups.py", line 198:
sage: S.j_classes()
Expected:
[['acb', 'cab', 'bca', 'abc', 'bac', 'cba'], ['ac', 'ca'],
['ab', 'ba'], ['bc', 'cb'], ['a'], ['c'], ['b']]
Got:
[['a'], ['acb', 'cba', 'bca', 'abc', 'bac', 'cab'], ['ac', 'ca'], ['ab', 'ba
'], ['cb', 'bc'], ['c'], ['b']]
**********************************************************************
2 items had failures:
So the problem is occuring on more that one architecuture. I hope to try the patch within the next 12 hours.
comment:13 Changed 3 years ago by mhansen
- Status changed from positive_review to closed
- Resolution set to fixed
- Merged in set to sage-4.3.4.alpha1

