Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

MAINT: Fix deprecated non-integer index to numpy arrays #390

Merged
merged 1 commit into from Dec 21, 2012

Conversation

Projects
None yet
2 participants
Contributor

jseabold commented Dec 21, 2012

I'm not sure why but these DeprecationWarnings from numpy were raising errors in nose for me.

Contributor

jseabold commented Dec 21, 2012

Reported errors in this thread

@rgommers rgommers commented on an outdated diff Dec 21, 2012

scipy/cluster/hierarchy.py
if aa > n:
+ aa = aa
@rgommers

rgommers Dec 21, 2012

Owner

No-op, remove

@rgommers rgommers commented on an outdated diff Dec 21, 2012

scipy/cluster/hierarchy.py
@@ -2324,6 +2325,7 @@ def _dendrogram_calculate_info(Z, p, truncate_mode, \
na = 1
da = 0.0
if ab > n:
+ ab = int(ab)
@rgommers

rgommers Dec 21, 2012

Owner

indentation?

@rgommers rgommers commented on an outdated diff Dec 21, 2012

scipy/stats/mstats_extras.py
@@ -352,6 +352,7 @@ def _idf(data):
if n < 3:
return [np.nan,np.nan]
(j,h) = divmod(n/4. + 5/12.,1)
+ j = int(j)
@rgommers

rgommers Dec 21, 2012

Owner

indentation?

Contributor

jseabold commented Dec 21, 2012

Oops. Thanks for checking, must've been sitting on my keyboard before I committed this. Testing now with those errors removed and will force push a new commit.

Contributor

jseabold commented Dec 21, 2012

It was an editor tabs/spaces problem. Should be fixed now.

@rgommers rgommers added a commit that referenced this pull request Dec 21, 2012

@rgommers rgommers Merge pull request #390 from jseabold/fix-int-idx
MAINT: Fix deprecated non-integer index to numpy arrays
785491c

@rgommers rgommers merged commit 785491c into scipy:master Dec 21, 2012

Owner

rgommers commented Dec 21, 2012

Merged, thanks Skipper.

@jseabold jseabold deleted the jseabold:fix-int-idx branch Dec 21, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment