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

Doc typo in .is_similar() method #14831

Closed
sagetrac-tmonteil mannequin opened this issue Jun 26, 2013 · 5 comments
Closed

Doc typo in .is_similar() method #14831

sagetrac-tmonteil mannequin opened this issue Jun 26, 2013 · 5 comments

Comments

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Jun 26, 2013

In the doc of the method .is_similar() (located in in sage/sage/matrix/matrix2.pyx), we get this unreadable block:

      sage: A = matrix(QQ, [[ 19, -7, -29], ...
      [-16, 11,  30], ...                   [ 15, -7, -25]]) sage: B =
      matrix(QQ, [[-38, -63,  42], ...                   [ 14,  25,
      -14], ...                   [-14, -21,  18]]) sage: A.charpoly()
      == B.charpoly() True sage: A.jordan_form() [-3| 0  0] [--+-----]
      [ 0| 4  1] [ 0| 0  4] sage: B.jordan_form() [-3| 0| 0]
      [--+--+--] [ 0| 4| 0] [--+--+--] [ 0| 0| 4] sage:
      A.is_similar(B) False

This is due to a missing ::.

Component: documentation

Author: Frédéric Chapoton

Reviewer: Thierry Monteil

Merged: sage-5.12.beta1

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

@sagetrac-tmonteil sagetrac-tmonteil mannequin added this to the sage-5.12 milestone Jun 26, 2013
@fchapoton
Copy link
Contributor

Author: Frédéric Chapoton

@fchapoton
Copy link
Contributor

comment:1

Attachment: trac_14831.patch.gz

here is a patch, please review

@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Jul 8, 2013

Reviewer: Thierry Monteil

@sagetrac-tmonteil
Copy link
Mannequin Author

sagetrac-tmonteil mannequin commented Jul 8, 2013

comment:2

Looks good to me, though i am not sure you should consider yourself as a beginner ;)

@jdemeyer
Copy link

Merged: sage-5.12.beta1

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