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

fix doctest failures in doc/en/constructions/linear_algebra.rst due to upgrade to Maxima 5.19.1 #6783

Closed
sagetrac-drkirkby mannequin opened this issue Aug 20, 2009 · 5 comments

Comments

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Aug 20, 2009

On Solaris (SPARC), the following tests failed. Both ECL and Maxima were updated - ECL version 9.8.4, Maxima version 5.19.1


----------------------------------------------------------------------
| Sage Version 4.1.1, Release Date: 2009-08-14                       |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
Thu Aug 20 20:02:37 BST 2009
dsage-trial tmp directory doesn't exist - creating ...
This script will run the unit tests for DSage
sage -t  "devel/sage/doc/en/constructions/linear_algebra.rst"
**********************************************************************
File "/export/home/drkirkby/sage/sage-4.1.1/devel/sage/doc/en/constructions/linear_algebra.rst", line 276:
    sage: eig
Expected:
    [[[-sqrt(3)*%i,sqrt(3)*%i],[1,1]],[1,(sqrt(3)*%i+1)/4],[1,-(sqrt(3)*%i-1)/4]]
Got:
    [[[-sqrt(3)*%i,sqrt(3)*%i],[1,1]],[[[1,(sqrt(3)*%i+1)/4]],[[1,-(sqrt(3)*%i-1)/4]]]]
**********************************************************************
File "/export/home/drkirkby/sage/sage-4.1.1/devel/sage/doc/en/constructions/linear_algebra.rst", line 291:
    sage: A.eigenvectors()
Expected:
    [[[2,11],[1,2]],[0,0,1],[0,1,1/3]]
Got:
    [[[2,11],[1,2]],[[[0,0,1]],[[0,1,1/3]]]]
**********************************************************************
File "/export/home/drkirkby/sage/sage-4.1.1/devel/sage/doc/en/constructions/linear_algebra.rst", line 294:
    sage: A.eigenvectors()
Expected:
     [[[-1,2],[2,1]],[0,1,-1],[0,0,1]]
Got:
    [[[-1,2],[2,1]],[[[0,1,-1]],[[0,0,1]]]]
**********************************************************************
2 items had failures:
   1 of   5 in __main__.example_11
   2 of   6 in __main__.example_12
***Test Failed*** 3 failures.
For whitespace errors, see the file /export/home/drkirkby/sage/sage-4.1.1/tmp/.doctest_linear_algebra.py
         [19.2 s]
sage -t  "devel/sage/doc/en/constructions/number_theory.rst"

Component: linear algebra

Keywords: maxima

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

@sagetrac-drkirkby sagetrac-drkirkby mannequin added this to the sage-4.1.2 milestone Aug 20, 2009
@sagetrac-drkirkby

This comment has been minimized.

@sagetrac-drkirkby sagetrac-drkirkby mannequin changed the title Doctest failure in sage-4.1.1/devel/sage/doc/en/constructions/linear_algebra.rst 3 doctest failures in sage-4.1.1/devel/sage/doc/en/constructions/linear_algebra.rst Aug 20, 2009
@aghitza
Copy link

aghitza commented Aug 20, 2009

Changed keywords from none to maxima

@aghitza
Copy link

aghitza commented Aug 20, 2009

comment:2

This is due to changes in Maxima's formatting of the output for eigenvectors. See attached patch.

@aghitza aghitza changed the title 3 doctest failures in sage-4.1.1/devel/sage/doc/en/constructions/linear_algebra.rst fix doctest failures in doc/en/constructions/linear_algebra.rst due to upgrade to Maxima 5.19.0 Aug 20, 2009
@aghitza
Copy link

aghitza commented Aug 20, 2009

Attachment: trac_6783.patch.gz

apply after the spkg's at #6564 and #6699

@aghitza aghitza assigned aghitza and unassigned williamstein Aug 20, 2009
@sagetrac-drkirkby sagetrac-drkirkby mannequin changed the title fix doctest failures in doc/en/constructions/linear_algebra.rst due to upgrade to Maxima 5.19.0 fix doctest failures in doc/en/constructions/linear_algebra.rst due to upgrade to Maxima 5.19.1 Aug 21, 2009
@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Sep 2, 2009

comment:6

This is fixed by #6699.

@sagetrac-mvngu sagetrac-mvngu mannequin removed the s: needs review label Sep 2, 2009
@sagetrac-mvngu sagetrac-mvngu mannequin closed this as completed Sep 2, 2009
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

2 participants