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

32 vs. 64 bit issues in matrix1.pyx #8944

Closed
sagetrac-mvngu mannequin opened this issue May 10, 2010 · 5 comments
Closed

32 vs. 64 bit issues in matrix1.pyx #8944

sagetrac-mvngu mannequin opened this issue May 10, 2010 · 5 comments

Comments

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 10, 2010

From this sage-devel report:

The first on seems to be a trivial 32-bit vs. 64-bit
issue (probably occurs on any 32-bit machine, even without the "long"
option):

sage -t -long "devel/sage/sage/matrix/matrix1.pyx"
**********************************************************************
File "/Users/Shared/sage/test/sage-4.4.2.alpha0/devel/sage/sage/matrix/
matrix1.pyx", line 460:
    sage: b.dtype
Expected:
    dtype('int64')
Got:
    dtype('int32')
********************************************************************** 

This also happens on the Skynet machine cicero, a 32-bit Fedora 12 machine.

Component: doctest coverage

Author: Minh Van Nguyen

Reviewer: John Cremona

Merged: sage-4.4.2.rc0

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

@sagetrac-mvngu sagetrac-mvngu mannequin added this to the sage-4.4.2 milestone May 10, 2010
@sagetrac-mvngu sagetrac-mvngu mannequin added t: tests labels May 10, 2010
@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented May 10, 2010

Attachment: trac_8944.patch.gz

based on Sage 4.4.2.alpha0

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented May 10, 2010

Author: Minh Van Nguyen

@JohnCremona
Copy link
Member

comment:2

Works fine on both 32 and 64 bit machines.

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented May 12, 2010

Merged: sage-4.4.2.rc0

@sagetrac-mvngu
Copy link
Mannequin Author

sagetrac-mvngu mannequin commented May 12, 2010

Reviewer: John Cremona

@sagetrac-mvngu sagetrac-mvngu mannequin closed this as completed May 12, 2010
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

1 participant