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

Revert ATLAS package #14753

Closed
jdemeyer opened this issue Jun 17, 2013 · 12 comments
Closed

Revert ATLAS package #14753

jdemeyer opened this issue Jun 17, 2013 · 12 comments

Comments

@jdemeyer
Copy link

There are some problems with generic binaries (SAGE_FAT_BINARY=yes) in the new ATLAS spkg. In order not to delay the release of Sage 5.10 any longer, I will revert #10508.

Fixing the issues below (and upgrading ATLAS to 3.10.x) is #14754.

On cicero (Fedora 16 i686), the build fails: http://build.sagemath.org/sage/builders/Skynet%20cicero%20%28Fedora%2016%20i686%29%20bin/builds/90/steps/shell_5/logs/atlas

Note the use of -DATL_3DNow and lots of "Illegal instruction" messages, proving that this likely is not a generic binary, unlike what it should be.

On arando (Ubuntu 13.04 i686), the build works but there is doctest failure:

sage -t devel/sage/sage/matrix/matrix_double_dense.pyx
**********************************************************************
File "devel/sage/sage/matrix/matrix_double_dense.pyx", line 3823, in sage.matrix.matrix_double_dense.Matrix_double_dense.is_positive_definite
Failed example:
    H.is_positive_definite()
Expected:
    True
Got:
    False
**********************************************************************

Component: packages: standard

Author: Jeroen Demeyer

Reviewer: Jeroen Demeyer

Merged: sage-5.10

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

@jdemeyer
Copy link
Author

Attachment: 14753_revert_atlas.patch.gz

@jdemeyer
Copy link
Author

comment:1

Attachment: 14753_revert_atlas_docs.patch.gz

@jdemeyer

This comment has been minimized.

@jdemeyer jdemeyer added this to the sage-5.10 milestone Jun 17, 2013
@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Author

Merged: sage-5.10

@jdemeyer
Copy link
Author

Author: Jeroen Demeyer

@jdemeyer
Copy link
Author

Reviewer: Jeroen Demeyer

@jpflori
Copy link

jpflori commented Jun 17, 2013

comment:5

The fact that 3DNow is used was discussed at #13706.

Of course we can remove that.

Jeroen: could you try to modify spkg-install: replace

isa_ext = ('3DNow',)

by

isa_ext = ()

on line 404 (in some version of the p0 spkg I have access to right now.)
It's in the configure_base function, the part about "Generic configuration on Intel i386 compatible CPUs"; and then try to rebuild on cicero? (I have no access to it)

@jdemeyer

This comment has been minimized.

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Author

comment:8

Replying to @jpflori:

The fact that 3DNow is used was discussed at #13706.

Of course we can remove that.

Jeroen: could you try to modify spkg-install: replace

isa_ext = ('3DNow',)

by

isa_ext = ()

on line 404 (in some version of the p0 spkg I have access to right now.)
It's in the configure_base function, the part about "Generic configuration on Intel i386 compatible CPUs"; and then try to rebuild on cicero? (I have no access to it)

I posted a spkg at #14754 doing that, I'll try it on cicero (a single-core Pentium 4 machine).

In any case, this will not change my decision to revert #10508. I want to release Sage 5.10 as soon as possible.

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