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

Update scipy to 0.16.1 #17643

Closed
jpflori opened this issue Jan 16, 2015 · 47 comments
Closed

Update scipy to 0.16.1 #17643

jpflori opened this issue Jan 16, 2015 · 47 comments

Comments

@jpflori
Copy link

jpflori commented Jan 16, 2015

Upgrade to scipy 0.16.1 this is intended to be done at the same time as numpy 1.10.1 and matplotlib 1.5.0 as noted in the ticket dependencies.

http://www.lmona.de/files/sage/scipy-0.16.1.tar.gz

Component: packages: standard

Author: François Bissey

Branch/Commit: cc6accc

Reviewer: Volker Braun

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

@jpflori jpflori added this to the sage-6.5 milestone Jan 16, 2015
@kiwifb
Copy link
Member

kiwifb commented Jan 16, 2015

comment:1

Didn't know it was out. Not in the Gentoo tree yet so I have no idea so far.

@kiwifb
Copy link
Member

kiwifb commented Aug 5, 2015

comment:2

scipy 0.16.0 is out, one doctest broken:

sage -t --long usr/share/doc/sage/en/faq/faq-usage.rst
Running doctests with ID 2015-08-05-11-07-08-20037f0b.
Using --optional=optional,sage
Doctesting 1 file.
sage -t --long --warn-long 64.8 usr/share/doc/sage/en/faq/faq-usage.rst
**********************************************************************
File "usr/share/doc/sage/en/faq/faq-usage.rst", line 351, in faq-usage
Failed example:
    stats.ttest_ind(list([1,2,3,4,5]),list([2,3,4,5,.6]))
Expected:
    (array(0.07675295564533369), 0.94070490247380478)
Got:
    Ttest_indResult(statistic=array(0.07675295564533369), pvalue=0.94070490247380478)
**********************************************************************
1 item had failures:
   1 of  36 in faq-usage
    [24 tests, 1 failure, 0.68 s]
----------------------------------------------------------------------
sage -t --long --warn-long 64.8 usr/share/doc/sage/en/faq/faq-usage.rst  # 1 doctest failed
----------------------------------------------------------------------

this is on top of 6.9.beta0.

@kiwifb

This comment has been minimized.

@kiwifb kiwifb modified the milestones: sage-6.5, sage-6.10 Nov 8, 2015
@kiwifb kiwifb changed the title Update scipy to 0.15 Update scipy to 0.16.1 Nov 8, 2015
@kiwifb
Copy link
Member

kiwifb commented Nov 8, 2015

Branch: u/fbissey/scipy_0.16.1

@kiwifb
Copy link
Member

kiwifb commented Nov 8, 2015

Commit: f3ff139

@kiwifb
Copy link
Member

kiwifb commented Nov 8, 2015

Author: François Bissey

@kiwifb
Copy link
Member

kiwifb commented Nov 8, 2015

New commits:

05c84c1upgrade to numpy 1.10.1, updating the package including a few cherrypicked patch from master
de8f740Merge branch 'develop' into numpy-1.10
ebdf899Fix to get rid of the problem with result_type. Pending upstream approval or suggestion of an alternative.
d12591fUpgarde matplotlib to 1.5.0
f3ff139Fix the doctest warning by adding it to the expected out. I am expecting the warning to originate from matplotlib and to be

@kiwifb
Copy link
Member

kiwifb commented Nov 8, 2015

Dependencies: #17642 #19556

@kiwifb

This comment has been minimized.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 9, 2015

Changed commit from f3ff139 to 31669f2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 9, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

31669f2Correct further warnings

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 9, 2015

Changed commit from 31669f2 to b21be98

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 9, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

b21be98Removing line numbers from doctest

@kiwifb
Copy link
Member

kiwifb commented Nov 10, 2015

comment:9

New branch compatible with the new push at #19556 coming soon.

@kiwifb
Copy link
Member

kiwifb commented Nov 10, 2015

New commits:

7177516Upgrade matplotlib to 1.5.0
7ce23d0Fix deprecation warnings in doctests.
c7ac0a7Updated numpy patch according to feedback from upstream
52a45f4Merge branch 'numpy-1.10' into mpl-1.5.0
a733692remove test for a limitation that doesn't exist anymore.
6d59badUpgrade to scipy 0.16.1 and fix affected doctest

@kiwifb
Copy link
Member

kiwifb commented Nov 10, 2015

Changed commit from b21be98 to 6d59bad

@kiwifb
Copy link
Member

kiwifb commented Nov 10, 2015

Changed branch from u/fbissey/scipy_0.16.1 to u/fbissey/scipy-0.16.1

@videlec
Copy link
Contributor

videlec commented Nov 13, 2015

comment:12

Just a confirmation that with the three tickets applied all tests pass (debian 64bits).

Could you replace

if np.array_equal(vert1, tpath.vertices) and np.array_equal(code1, tpath.codes):
     return True
else:
     return False

with

return np.array_equal(vert1, tpath.vertices) and np.array_equal(code1, tpath.codes)

Or there is a reason to let it as you did?

@kiwifb
Copy link
Member

kiwifb commented Nov 13, 2015

comment:13

No reason I didn't do that. I was so focused in finding a replacement for that particular line (it took some effort for me) that I didn't realise the code could be streamlined.
I will make the change your suggest in a few hours. Would you otherwise be happy to review positively the tickets?

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 14, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

04997f6Simplify flow in arrow.py
8b383deMerge branch 'mpl-1.5.0' into scipy0.16.1

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 14, 2015

Changed commit from 6d59bad to 8b383de

@kiwifb
Copy link
Member

kiwifb commented Nov 14, 2015

comment:15

Replying to @kiwifb:

I will make the change your suggest in a few hours. Would you otherwise be happy to review positively the tickets?

All done.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 23, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

d40c0e9Adding "config_fc --noarch" to setup.py call so we don't get miscompilation on ARM.

@vbraun
Copy link
Member

vbraun commented Nov 23, 2015

comment:24

Can you report it somewhere upstream?

@kiwifb
Copy link
Member

kiwifb commented Nov 23, 2015

comment:25

I'll look into it. I am guessing it is technically a bug in numpy's distutils rather than scipy but I have no idea how to fix it or how it happens.

@kiwifb
Copy link
Member

kiwifb commented Nov 23, 2015

comment:26

It looks like someone already acted in upstream numpy, I guess an option is to try to backport the commit in the numpy we ship.
numpy/numpy@a08437e

@jpflori
Copy link
Author

jpflori commented Nov 23, 2015

comment:27

I think I might have removed the no_arch stuff when I began to clean up the spkg mess.
Backporting the fix looks a good idea for a further ticket.
Or we just wait for a numpy version bump.

@kiwifb
Copy link
Member

kiwifb commented Nov 23, 2015

comment:28

Replying to @jpflori:

Backporting the fix looks a good idea for a further ticket.
Or we just wait for a numpy version bump.

The danger with those two scenario is that the removal will be forgotten. However considering that everyone else has it with some variations (Gentoo has noopt and fedora has fcompiler=gunf95 [or something]) I guess it doesn't hurt to keep it.

@vbraun
Copy link
Member

vbraun commented Nov 23, 2015

Changed branch from u/fbissey/scipy-0.16.1 to d40c0e9

@kiwifb
Copy link
Member

kiwifb commented Nov 23, 2015

comment:30

I must say I spent a bit of my night worrying that it was the wrong fix. Good that it really works.

@kiwifb
Copy link
Member

kiwifb commented Nov 23, 2015

Changed commit from d40c0e9 to none

@vbraun
Copy link
Member

vbraun commented Nov 23, 2015

comment:31

Argh still doesn't work... http://build.sagedev.org/release/builders/%20%20slow%20Oxford%20ARM%20%28Ubuntu%2012.10%29%20incremental/builds/330/steps/compile/logs/scipy


Last 10 new commits:

ebdf899Fix to get rid of the problem with result_type. Pending upstream approval or suggestion of an alternative.
7177516Upgrade matplotlib to 1.5.0
7ce23d0Fix deprecation warnings in doctests.
c7ac0a7Updated numpy patch according to feedback from upstream
52a45f4Merge branch 'numpy-1.10' into mpl-1.5.0
a733692remove test for a limitation that doesn't exist anymore.
6d59badUpgrade to scipy 0.16.1 and fix affected doctest
04997f6Simplify flow in arrow.py
8b383deMerge branch 'mpl-1.5.0' into scipy0.16.1
d40c0e9Adding "config_fc --noarch" to setup.py call so we don't get miscompilation on ARM.

@vbraun
Copy link
Member

vbraun commented Nov 23, 2015

Changed branch from d40c0e9 to u/fbissey/scipy-0.16.1

@vbraun
Copy link
Member

vbraun commented Nov 23, 2015

Commit: d40c0e9

@vbraun vbraun reopened this Nov 23, 2015
@kiwifb
Copy link
Member

kiwifb commented Nov 23, 2015

comment:32

OK so I was right to worry it was the wrong fix. I'll remove the last commit on the scipy branch and add the upstream commit in numpy. I guess if I had gone for noopt instead of noarch it would have worked.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 23, 2015

Changed commit from d40c0e9 to 93a8c3d

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 23, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

93a8c3dRevert "Adding "config_fc --noarch" to setup.py call so we don't get miscompilation on ARM."

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 23, 2015

Changed commit from 93a8c3d to cc6accc

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 23, 2015

Branch pushed to git repo; I updated commit sha1. New commits:

9c6e94cAdd upstream patch to reverse inclusion of mingwin64 support. It cause ARM to be taken for win64!
414fccfMerge branch 'numpy-1.10' into mpl-1.5.0
cc6acccMerge branch 'mpl-1.5.0' into scipy0.16.1

@vbraun
Copy link
Member

vbraun commented Nov 24, 2015

Changed branch from u/fbissey/scipy-0.16.1 to cc6accc

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

4 participants