type errorを解決したい．

Question

###前提・実現したいこと
pythonで各質量ごとに変化する２次元の関数を積分し，書き出すシステムを作っています。
書き出す機能を実装中に以下のエラーメッセージが発生しました。

###発生している問題・エラーメッセージ

<ipython-input-15-08f6886babba> in <module>()
      7     file.write( str(10**logm[i]) )
      8     file.write( '\t' )
      9     file.write( str(sigmath(x)) )<------(エラー)
     10     file.write( '\t' )
     11     file.write( str(sigmaga(x)) )
<ipython-input-14-b88dac1df230> in sigmath(x)
      1 def sigmath(x):
      2     return integrate.simps(f(x,y), x)<----------(エラー)
      3 def sigmaga(x):
      4     return integrate.simps(g(x,y), x)
      5 def sigmask(x):
TypeError: '_io.TextIOWrapper' object is not callable​

###該当コード

# In[1]:


# -- coding: utf-8 --
import numpy as np
from scipy import signal, integrate,interpolate


# In[2]:


vofk = np.loadtxt('myset_matterpower.dat', usecols=(0))
vofmp = np.loadtxt('myset_matterpower.dat', usecols=(1))
x = np.log10(vofk)  # vofkをxに格納
y = np.log10(vofmp)  # vofmpをyに格納


# In[3]:


# サンプルデータ作成
xnew = np.linspace(min(x),max(x),num=10*len(x)+123)
tt = xnew
#3spline
f0 = interpolate.InterpolatedUnivariateSpline(x, y,k=3)
y0 = f0(tt)


# In[4]:


x=10.0**tt # vofkを更新
y=10.0**y0# vofmpを更新
f = open('cdm_spline.dat','w')
f.write('#vofk \t vofmp\n')
for i in range(len(x)):
    f.write( str(x[i]) )
    f.write( '\t' )
    f.write( str(y[i]) )
    f.write( '\n' )

f.close()
# x=10.0**tt # vofkを更新
# y=10.0**y0# vofmpを更新


# In[5]:


def rho(z):
    h = 0.6780
    ob0 = 0.02225#h2
    ocdm0 = 0.1198#h2
    rho0 = 2.77536627e11
    return rho0 * (ob0 + ocdm0) * ((1.0 + z) ** 3.0)
def heaviside(x):
    return 0.5 * (np.sign(x) + 1)


# In[6]:


# 半径
def rth(m):
    return (m * 3.0 / (4.0 * np.pi * rho(z))) ** (1.0 / 3.0)
def rga(m):
    return ((2.0 * np.pi) ** (-1.0 / 2.0)) * ((m / (np.pi * rho(z))) ** (1.0 / 3.0))
def rsk(m):
    return (m / (6.0 * (np.pi ** 2.0) * rho(z))) ** (1.0 / 3.0)


# In[7]:


# 窓関数rth(m)
def wth(x):
    return (3.0 / ( rth(m)* x) ** 3.0) * (np.sin(rth(m) * x) - rth(m) * x * np.cos(rth(m) * x))
def wga(x):
    return np.e ** (((-(rga(m) * x) ** 2.0)) / 2.0)
def wsk(x):
    return heaviside(1.0 - (rsk(m) * x))


# In[8]:


# 被積分関数
def f(x,y):
    return (y / (2 * np.pi ** 2)) * (wth(x) ** 2) * x ** 2
def g(x,y):
    return (y / (2 * np.pi ** 2)) * (wga(x) ** 2) * x ** 2
def h(x,y):
    return (y / (2 * np.pi ** 2)) * (wsk(x) ** 2) * x ** 2


# In[9]:


def sigmath(x):
    return integrate.simps(f(x,y), x)
def sigmaga(x):
    return integrate.simps(g(x,y), x)
def sigmask(x):
    return integrate.simps(h(x,y), x)


# In[10]:


logm = np.linspace(0, 15, num=10000)
z = 0.0
file = open('cdm_variance.dat', 'w')
file.write('#vofm \t vofsigmath \t vofsigmaga \t vofsigmask\n')
for i in range(len(logm)):
    m = 10 ** logm[i]
    file.write(str(10 ** logm[i]))
    file.write('\t')
    file.write(str(sigmath(x)))
    file.write('\t')
    file.write(str(sigmaga(x)))
    file.write('\t')
    file.write(str(sigmask(x)))
    file.write('\n')

file.close()

###試したこと
9で定義せずに，10のループ内で直接積分を質量事に行った．
###補足情報(言語/FW/ツール等のバージョンなど)
anaconda3,pycharm,jupyter

Comments

[LouiS0616]

In[9]からIn[13]が抜けていますが、その間にファイル処理など行っていませんか？

[ktfs]

jupyterの更新忘れです．In番号更新しました．

Answer 1

[8]と[14]の間でfでファイルを開いていませんか？

---

fのような上書きされそうな関数名を使いたいのであればスコープをうまく使うとバグりにくくなります。

def sigmath(x):
    def f(x,y):
        return (y / (2 * np.pi ** 2)) * (wth(x) ** 2) * x ** 2
    return integrate.simps(f(x,y), x)

Comments

[ktfs]

jupyterの更新忘れです．In番号更新しました．

[mkgrei]

sigmathのreturnの前に
print(type(f))
を追加した場合、何が出力されますか？

[ktfs]

<class 'function'>が１００００行出ました．

[mkgrei]

TypeErrorは依然として出ていますか？

[ktfs]

消えました．

[ktfs]

print(type(f))を消しても動きました．

[mkgrei]

大量の出力を誘発して、失礼しました。

結局問題は他のファイルを開いてf関数を上書きしてしまったことにありました。
コード自体にはバグはありませんが、特定のセルの再実行を挟むとエラーが起こりやすい状況にあります。
fはファイルを開くときに無意識に使ってしまったりするので、関数名を他のものにするか、sigmath関数の中に定義すると事故は減るでしょう。

[ktfs]

一体全体コードの何がおかしかったのでしょうか．

[ktfs]

関数ｆとファイルfがかぶってしまったということでしょうか

[ktfs]

ありがとうございました