teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

修正: 語弊がある表現だったのを修正

2021/06/27 05:18

投稿

teamikl
teamikl

スコア8817

answer CHANGED
@@ -1,4 +1,4 @@
1
- 問題点: スレッドが正しく使えてません
1
+ 問題点: スレッドが~~正しく使えてません~~ 用途に合った使い方になってません
2
2
 
3
3
  ```python
4
4
  while True:

2

追記: マルチ「プロセス」時のロギングの設定に言及

2021/06/27 05:18

投稿

teamikl
teamikl

スコア8817

answer CHANGED
@@ -59,9 +59,14 @@
59
59
  # if __name__ == "__main__": 内に置きたいが、なければファイル先頭でもok
60
60
  logging.basicConfig(level=logging.DEBUG, format="[%(threadName)-10s][%(levelname)-8s] %(message)s")
61
61
 
62
- # logger モジュールは、スレッドセーフなので、どこからでも安全に呼び出せます。
62
+ # logging モジュールは、スレッドセーフなので、どこからでも安全に呼び出せます。
63
63
  # print の代わりに利用 logger.info, logger.debug, logger.error, 等
64
64
  logger.debug(f"debug message: {fn=}") # f-string で変数 fn を表示
65
+
66
+
67
+ # ※ multiprocessing 利用があったので追記
68
+ # マルチ「プロセス」の場合は、この設定では不十分で、
69
+ # 追加でプロセス間のキューを用いた設定・構成が必要になります。
65
70
  ```
66
71
 
67
72
  ----

1

追記: スレッドの使い方について

2021/06/26 11:18

投稿

teamikl
teamikl

スコア8817

answer CHANGED
@@ -40,4 +40,54 @@
40
40
 
41
41
  これらのライブラリはイベントループを自分で書く利用法が主なので、
42
42
  同時利用自体は可能なのですが、注意が必要な点は、
43
- 双方のイベントループを止めないように配慮しないと、片方が応答なしになってしまいます。
43
+ 双方のイベントループを止めないように配慮しないと、片方が応答なしになってしまいます。
44
+
45
+ ---
46
+ 追記: コードの全貌をまだ把握しきれてないので、読みながら判明した点を追記します。
47
+
48
+ 問題点: スレッドの使い方が、ボタンを押したときにスレッド生成、
49
+ スレッド内ではキューへの出し入れしか仕事をしていない。
50
+
51
+ 確認方法: logging モジュールを使い、関数がどのスレッドで実行されているかを表示する。
52
+
53
+ ```python
54
+ # ファイルの先頭で
55
+ import logging
56
+ logger = logging.getLogger(__name__)
57
+
58
+ # ログにスレッド名を表示&ロギングを有効にする設定
59
+ # if __name__ == "__main__": 内に置きたいが、なければファイル先頭でもok
60
+ logging.basicConfig(level=logging.DEBUG, format="[%(threadName)-10s][%(levelname)-8s] %(message)s")
61
+
62
+ # logger モジュールは、スレッドセーフなので、どこからでも安全に呼び出せます。
63
+ # print の代わりに利用 logger.info, logger.debug, logger.error, 等
64
+ logger.debug(f"debug message: {fn=}") # f-string で変数 fn を表示
65
+ ```
66
+
67
+ ----
68
+ スレッドは、解りやすくリソース毎に割り当てるなら
69
+ - 音楽再生スレッド(portaudio)
70
+ - 波形プロットのスレッド(pygame?)
71
+ - メインスレッド(PySimpleGUI)
72
+ とします。
73
+ 必要なキューは、GUI -> 音楽再生停止の制御と、wavデータ読み出し→波形プロット
74
+
75
+ メイン: pygame+PySimpleGUI <=> サブスレッド: portaudio にする方法もありますが、
76
+ 分離した方が、バックエンドを別のライブラリに差し替え等といった時に都合の良い設計になります。
77
+
78
+ スレッドの解りやすい使い方は、
79
+ ボタンを押したときに毎回生成ではなく、スレッドは予め稼働させておいて
80
+ ボタンを押したときに動き出すように、queue や threading.Eevnt 等で制御します。
81
+ メインスレッド終了時に、スレッドは正しく終了させるように構成する。
82
+ (スレッド内でのループやブロック処理は抜けるように。→リソース解放処理を確実に呼び出す)
83
+
84
+ ----
85
+ pygame をメインスレッド以外で使えるかは、試してないので解りませんが、
86
+ 使えない場合は メインスレッドの利用をPySimpleGUI と入れ替え。
87
+
88
+ GUIライブラリではメインスレッドでしか使えない制限がある事があります。
89
+ (正確には、初期化したスレッドと同じスレッド内でないと誤動作の原因に)
90
+
91
+ 基本は、ウィンドウを閉じた時に終了したい場合が多いので、
92
+ GUI をメインスレッドにした方が都合が問いのですが
93
+ PySimpleGUI 内で使われている tkinter は別スレッドでも大丈夫です。