First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 81520
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Python Gentoo Team <python@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Darren Dale <dsdale24@gmail.com>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
setup.py.diff patch for setup.py to recognize atlas on gentoo patch Darren Dale 2005-02-10 11:04 0000 213 bytes Details | Diff
numeric-23.7.atlas.patch patch for numeric to recognize atlas patch Darren Dale 2005-02-10 12:12 0000 470 bytes Details | Diff
numeric-23.7-r1.ebuild proposed ebuild for numeric-23.7-r1 text/plain Darren Dale 2005-02-10 12:17 0000 1020 bytes Details
setup.py.diff updated patch to play nice with blas-config and lapack-config patch Darren Dale 2005-02-11 06:55 0000 479 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 81520 depends on: Show dependency tree
Bug 81520 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2005-02-10 10:55 0000
I suggest that if atlas is listed in the USE flags, that Numeric should depend
on lapack-atlas and blas-atlas. I suggest that sci-libs/atlas is not a good
alternative, because it does not have a full lapack implementation.

The setup.py file should be modified, I'll post a patch.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

------- Comment #1 From Darren Dale 2005-02-10 11:04:56 0000 -------
Created an attachment (id=50930) [details]
patch for setup.py to recognize atlas on gentoo

Here is the setup.py patch for an ebuild to recognize lapack-atlas. I dont have
experience writing ebuilds, but I will try to propose a new one that depends on
lapack-atlas if USE="atlas". This patch would be applied instead of the
existing patch, again if USE="atlas".

------- Comment #2 From Darren Dale 2005-02-10 12:12:38 0000 -------
Created an attachment (id=50945) [details]
patch for numeric to recognize atlas

successfully built numeric with this patch, and the following ebuild...

------- Comment #3 From Darren Dale 2005-02-10 12:17:53 0000 -------
Created an attachment (id=50946) [details]
proposed ebuild for numeric-23.7-r1

I was able to link numeric-23.7 to atlas-tuned blas and lapack using this
ebuild and patch 50945. If USE="atlas", then lapack-atlas is required.

------- Comment #4 From Danny van Dyk (RETIRED) 2005-02-11 03:18:46 0000 -------
Hi Darren!

Ok, what I don't like on this idea is the following: We currently have a working
system to switch between different BLAS and LAPACK implementations. So it makes
no sense to apply this patch to circumvent the current system.

A much better patch would make it look like this:

library_dirs_list = ['/usr/lib/']
libraries_list = ['lapack', 'cblas', 'blas', 'atlas', 'g2c']

This way, any user of numeric could switch between the implementations using
blas-config and lapack-config.

------- Comment #5 From Darren Dale 2005-02-11 06:55:33 0000 -------
Created an attachment (id=50997) [details]
updated patch to play nice with blas-config and lapack-config

The use flag in the above ebuild is in question, but this patch is offerred as
suggested in the last comment.

------- Comment #6 From Danny van Dyk (RETIRED) 2005-02-16 04:05:09 0000 -------
[As a side note: please submit always patches against ebuilds you want to be
modified. This eases the devs job twice: 1st) he sees the actual differences,
2nd) he can commit w/o problems even if other parts of the ebuild have been
altered in between submission and commit. ;-) ]

Darren, as we already talked about on gentoo-science ML, we need to remove
the "atlas" USE flag from your ebuild submission...

@Python-herd: I'd like to see this in the tree. Mind if I commit it myself and
add the Scientific Herd to the metadata ?

--- numeric-23.7.ebuild 2005-02-16 13:01:25.403085728 +0100
+++ numeric-23.7-r1.ebuild      2005-02-16 13:01:08.422667144 +0100
@@ -11,19 +11,21 @@
 SRC_URI="mirror://sourceforge/numpy/${MY_P}.tar.gz"
 HOMEPAGE="http://www.pfdubois.com/numpy/"

-IUSE=""
+IUSE="lapack"
 SLOT="0"
 KEYWORDS="~x86 ~ppc ~sparc ~alpha ~hppa ~amd64"
 LICENSE="as-is"

 # 2.1 gave sandbox violations see #21
-DEPEND=">=dev-lang/python-2.2"
+DEPEND=">=dev-lang/python-2.2
+       lapack? ( virtual/lapack )"

 src_unpack() {
        unpack ${A}

        cd ${S}
        epatch ${FILESDIR}/${P}.patch
+       epatch ${FILESDIR}/${P}-lapack.patch
 }

 src_install() {

------- Comment #7 From Darren Dale 2005-02-16 07:01:20 0000 -------
Thats excellent, thank you Danny.

One final comment, the homepage for Numeric is now:

http://numeric.scipy.org/

It should be reflected in the ebuild.

------- Comment #8 From Sébastien Fabbro 2006-06-07 10:33:47 0000 -------
The Gentoo Scientific overlay in http://gentooscience.org contains an updated
ebuild of numeric allowing lapack and also makes some fixes.

------- Comment #9 From Sébastien Fabbro 2006-09-13 09:16:26 0000 -------
The ebuild in the gentooscience.org overlay fixes a decent number of numeric
problems. From the ChangeLog:

  +files/numeric-arrayobject.patch, +files/numeric-lapack.patch,
  +numeric-24.2-r1.ebuild: fixed a memory leak, and added the possibility
  to compile linear algebra with installed blas/cblas/lapack.
  Tested against gcc41 and gcc34 on amd64, x86. Added longdescription
  in metadata.xml. Closes bugs #81520 and #114885.

May be it would be worth a bump in the main tree?

------- Comment #10 From Sébastien Fabbro 2007-02-07 17:23:23 0000 -------
added lapack support in numeric-24.2-r2, now in cvs.

First Last Prev Next    No search results available      Search page      Enter new bug