Ticket #8426 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Add scipy to 3rdpartylibs-mac

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: critical Milestone: Release 3.1
Component: Python Keywords:
Cc: borreguero@… Blocked By:
Blocking: Tester: Michael Reuter

Description

We make scipy available on all other platforms (by packaging it on Windows & specifying it as a dependency on linux) so it makes sense to add it here too.

More appositely, Jose wants to use it in a new python fitting function he's trying to add.

Change History

comment:1 Changed 7 years ago by Russell Taylor

  • Status changed from new to inprogress

Re #8426. Add scipy version 0.13.0

This was built via easy_install on a Mountain Lion machine onto which I had installed gfortran 4.2.1. It is build against Apple Python 2.7.2 and numpy 1.6.1 (both of which come pre-installed on OS X).

Changeset: 4c6014aa08ae2ec770e92b1f9d1705b99bcb66a6

comment:2 Changed 7 years ago by Russell Taylor

I had to jump through a couple of hoops to make this work. First, I had to download gfortran 4.2.1 (found it at http://cran.us.r-project.org/bin/macosx/tools/old/) rather than the latest version so that the compiled code would depend on the same libgcc_s.1.dylib that comes pre-installed on OS X. Second, I moved the libgfortran dylibs out of the way to force static linking to those libraries.

comment:3 Changed 7 years ago by Russell Taylor

  • Status changed from inprogress to verify
  • Resolution set to fixed

To test: There's no branch as this is in the 3rdpartylibs-mac repo

This should probably be tested by someone who has some knowledge of scipy, and certainly must be tested on a Mac. I did successfully run the scipy test suite (see here) against the build, so I'm pretty confident it's working.

Build Mantid (making sure to update Third_Party), or take an install kit (can be a develop or master build; it doesn't matter) and have a play around with some scipy commands.

comment:4 Changed 7 years ago by Russell Taylor

  • Status changed from verify to reopened
  • Resolution fixed deleted

I think I'll remove the test files from the repo to save a little space.

comment:5 Changed 7 years ago by Jose Borreguero

  • Blocking 8373 removed

(In #8373)

comment:6 Changed 7 years ago by Russell Taylor

  • Status changed from reopened to inprogress

Re #8426. Remove test files to save a little space.

We don't need to carry them around.

Changeset: abe6e17e146bc6ba20b7650b412fa52305dc2227

comment:7 Changed 7 years ago by Russell Taylor

  • Status changed from inprogress to verify
  • Resolution set to fixed

See comment:3 for testing info.

comment:8 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:9 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Downloaded the latest kit and ran some things from the SciPy cookbook and things went fine.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9270

Note: See TracTickets for help on using tickets.