Pythonのマルチスレッド処理に関するエラー

Question

前提・実現したいこと

データセットがキーのアルファベット記号と要素の数字からなる辞書型で与えられており、
数字はリストに格納されています。リストの長さはキーによって様々です。
各キーの要素の数字間でそれぞれ、たすき掛けで掛け算し、最も大きいものを採用した上で、
配列の長さで割った値を２つのキー間の値として出力します。

これを大規模なデータセットで実行することを実現したいです。
関連する前回の質問

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

小規模なデータセットでは問題なく実行できますが、
このコードで、データセットのキーの数が大規模（例えば3万など）な場合、
１つのキーに対してたすき掛けの計算で比較する対象も膨大になるため、
現行のコードのまま実行すると膨大な時間がかかってしまいます。

そこで、並列処理を行うために、以下のコードを書いて、
辞書のキー別に平行で実行しようとしています。
現在のコードではキー別に実行する方法がわからないため、まず、全ての処理を３つのスレッドで平行して行う形式になっていますが、それでも以下のエラーが出てしまっている状況です。

#３つのスレッドで同じエラーが出ている状況です
Exception in thread Thread-8:
Traceback (most recent call last):
  File "/Users/username/anaconda/lib/python3.6/threading.py", line 916, in _bootstrap_inner
    self.run()
  File "/Users/username/anaconda/lib/python3.6/threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
TypeError: 'dict' object is not callable

Exception in thread Thread-6:
Traceback (most recent call last):
  File "/Users/username/anaconda/lib/python3.6/threading.py", line 916, in _bootstrap_inner
    self.run()
  File "/Users/username/anaconda/lib/python3.6/threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
TypeError: 'dict' object is not callable

Exception in thread Thread-7:
Traceback (most recent call last):
  File "/Users/username/anaconda/lib/python3.6/threading.py", line 916, in _bootstrap_inner
    self.run()
  File "/Users/username/anaconda/lib/python3.6/threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
TypeError: 'dict' object is not callable

該当のソースコード

def compute(data, output):
    for alph, nums in data.items():
        avg = {}
        my_list = data[alph]
        for target_alph, target_nums in data.items():
            target_list = data[target_alph]
            if alph == target_alph:
                continue
            max_nums = []

            for i in my_list:
                max_num = 0
                for j in target_list:
                    result = i * j
                    if result is not None and result > max_num:
                        max_num = result
                max_nums.append(max_num) 
            avg[target_alph] = sum(max_nums) / len(max_nums)
            output[alph] = avg
    return output
    
data = {
    'A':[1, 3, 5, 2, 1, 8, 9],
    'B':[9, 4, 3], 
    'C':[8, 5, 5, 6, 1]
}

output = {}

compute(data, output)

thA = threading.Thread(target=compute(data, output)) #data['A']
thB = threading.Thread(target=compute(data, output))#data['B']
thC = threading.Thread(target=compute(data, output)) #data['C']

thA.start(); print ('A started.')
thB.start(); print ('B started.')
thC.start(); print ('C started.')

thA.join(); print ('A completed.')
thB.join(); print ('B completed.')
thC.join(); print ('C completed.')

試したこと

試したことは、関数化した計算部分のコードと並列処理に関する簡単なコードです。
これらを組み合わせる段階でつまづいています。

関数化した計算部分のコード

def compute(data, output):
    for alph, nums in data.items():
        avg = {}
        my_list = data[alph]
        for target_alph, target_nums in data.items():
            target_list = data[target_alph]
            if alph == target_alph:
                continue
            max_nums = []

            for i in my_list:
                max_num = 0
                for j in target_list:
                    result = i * j
                    if result is not None and result > max_num:
                        max_num = result
                max_nums.append(max_num) 
            avg[target_alph] = sum(max_nums) / len(max_nums)
            output[alph] = avg
    return output
    
data = {
    'A':[1, 3, 5, 2, 1, 8, 9],
    'B':[9, 4, 3], 
    'C':[8, 5, 5, 6, 1]
}

output = {}

compute(data, output)

出力

{'A': {'B': 37.285714285714285, 'C': 33.142857142857146},
 'B': {'A': 48.0, 'C': 42.666666666666664},
 'C': {'A': 45.0, 'B': 45.0}}

並列処理に関する簡単なコード

import threading
 
def compute():
    x = 0
    times = 2 ** 26
    while x < times:
        x += 1
 
th1 = threading.Thread(target=compute)
th2 = threading.Thread(target=compute)

th1.start(); print '1 started.'
th2.start(); print '2 started.'

th1.join(); print '1 completed.'
th2.join(); print '2 completed.'

補足情報（FW/ツールのバージョンなど）

Python3.6

Comments

[umyu]

Pythonのthreading.ThreadはIO律速な処理を高速化するためのものなので、今回のようなCPU律速の処理を行うにはmultiprocessing やconcurrent.futures.ProcessPoolExecutorを使ってくださいな。

Answer 1

内容の是非や意味は無視して、出ているエラーについて回答しておきます。

threading.Threadのtargetにはcallable（関数オブジェクトなど）を渡す必要があります。

threading.Thread(target=compute(data, output))

というコードは、compute(data, output)が評価されて結果の辞書になってtargetに渡されているので、無意味ですし間違っています。

引数を与えたければargsを使ってください。

threading.Thread(target=compute, args=(data, output))

---

以前の質問でコードの改良案を回答しましたが、どうなったのでしょうか。

Comments

[退会済みユーザー]

以前、コードの改良案をご回答いただきましてありがとうございました。
できる限り無駄を無くしたコードにしてもハードウェア側のスペックで計算が難しい状況です。