#4395 closed defect (fixed)
[with patch; positive review] Sage 3.1.4: magma related optional doctest failure in sage/rings/quotient_ring.py
Reported by: | mabshoff | Owned by: | was |
---|---|---|---|
Priority: | major | Milestone: | sage-3.2 |
Component: | doctest coverage | Keywords: | |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
sage -t -long -optional devel/sage/sage/rings/quotient_ring.py ********************************************************************** File "/home/mabshoff/build-3.2.a1/sage-3.2.alpha1-iras/tmp/quotient_ring.py", line 647: sage: Q._magma_() # optional requires Magma Expected: Affine Algebra of rank 2 over GF(2) Graded Reverse Lexicographical Order Variables: x, y Quotient relations: [ x^2 + x, y^2 + y ] Got: Affine Algebra of rank 2 over GF(2) Graded Reverse Lexicographical Order Variables: x, y Quotient relations: [ 0, 0 ] ********************************************************************** 1 items had failures: 1 of 5 in __main__.example_23 ***Test Failed*** 1 failures. For whitespace errors, see the file /home/mabshoff/build-3.2.a1/sage-3.2.alpha1-iras/tmp/.doctest_quotient_ring.py [5.4 s] exit code: 1024
Attachments (1)
Change History (7)
comment:1 Changed 11 years ago by
- Summary changed from Sage 3.1.4: magma related optional doctest failure in sage/rings/quotient_ring.py to [with patch; needs review] Sage 3.1.4: magma related optional doctest failure in sage/rings/quotient_ring.py
comment:2 Changed 11 years ago by
Where is the patch?
Cheers,
Michael
comment:3 Changed 11 years ago by
This patch (one it is here :)) also likely fixes the following problem:
sage -t -long -optional devel/sage/sage/rings/polynomial/pbori.pyx ********************************************************************** File "/scratch/mabshoff/release-cycle/sage-3.1.3.final/tmp/pbori.py", line 988: sage: B._magma_() # optional requires magma Expected: Affine Algebra of rank 3 over GF(2) Lexicographical Order Variables: x, y, z Quotient relations: [ x^2 + x, y^2 + y, z^2 + z ] Got: Affine Algebra of rank 3 over GF(2) Lexicographical Order Variables: x, y, z Quotient relations: [ 0, 0, 0 ] ********************************************************************** File "/scratch/mabshoff/release-cycle/sage-3.1.3.final/tmp/pbori.py", line 1024: sage: B._magma_() # optional requires magma, indirect doctest Expected: Affine Algebra of rank 3 over GF(2) Lexicographical Order Variables: x, y, z Quotient relations: [ x^2 + x, y^2 + y, z^2 + z ] Got: Affine Algebra of rank 3 over GF(2) Lexicographical Order Variables: x, y, z Quotient relations: [ 0, 0, 0 ] **********************************************************************
Changed 11 years ago by
comment:4 Changed 11 years ago by
- Milestone changed from sage-3.2.1 to sage-3.2
- Summary changed from [with patch; needs review] Sage 3.1.4: magma related optional doctest failure in sage/rings/quotient_ring.py to [with patch; positive review] Sage 3.1.4: magma related optional doctest failure in sage/rings/quotient_ring.py
Positive review. It fixes the original problem reported, but not as I suspected the issue in
devel/sage/sage/rings/polynomial/pbori.pyx
That issue is now #4482.
Cheers,
Michael
comment:5 Changed 11 years ago by
- Resolution set to fixed
- Status changed from new to closed
Merged in Sage 3.2.rc0
comment:6 Changed 5 years ago by
- Description modified (diff)
- Report Upstream set to N/A
Note: See
TracTickets for help on using
tickets.
Apply this after #4394