Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix sign of infstr in exceltools.rec2exel #1722

Merged
merged 1 commit into from

2 participants

@daler

(I couldn't find any existing tests for exceltools.py)

@WeatherGod
Collaborator

Wow, must have been a typo back in 2008... Perhaps we should cherry-pick this to the maintenance branch if it is worth the hassle?

@WeatherGod WeatherGod merged commit 7ee7337 into matplotlib:master

1 check failed

Details default The Travis build failed
@WeatherGod
Collaborator

Merged to master, and cherry-picked to the maintenance branch as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 30, 2013
  1. @daler
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/mpl_toolkits/exceltools.py
View
2  lib/mpl_toolkits/exceltools.py
@@ -124,7 +124,7 @@ def rec2excel(r, ws, formatd=None, rownum=0, colnum=0, nanstr='NaN', infstr='Inf
ws.write(rownum, colnum+i, nanstr)
elif mlab.safe_isinf(val):
sgn = np.sign(val)
- if sgn<0: s = infstr
+ if sgn>0: s = infstr
else: s = '-%s'%infstr
ws.write(rownum, colnum+i, s)
elif format.xlstyle is None:
Something went wrong with that request. Please try again.