python マルチスレッド終了時のエラーメッセージ

Question

前提・実現したいこと

arduino側より電圧値を送信し、cvsファイルに書き込むプログラムを書いています。
[開始]ボタンで測定値読み見込みはじめ[終了]ボタンで測定を終了させます。
[終了]ボタンを押すとdialogは消えて、再度起動すれば測定可能ですが、[終了]ボタンで以下エラーメッセージが出力されます。

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

[終了]させると、下記エラーメッセージが出力されます。
動作そのものは、問題なく動いているようです。
これを解消させるためにはどのようにすればよろしいでしょうか?

エラーメッセージ
Exception in thread Thread-7:
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\threading.py", line 926, in _bootstrap_inner
self.run()
File "C:\ProgramData\Anaconda3\lib\threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "D:/Python/STAIC-SK1000/STAIC-SK1000_04.py", line 39, in serial_data_recv
fl_line = float(w_line)
ValueError: could not convert string to float:

該当のソースコード

import tkinter as tk
import datetime
import serial
import time
import threading

ser = serial.Serial("COM6",9600)

def serial_data_recv():
    global stop_flag
    stop_flag = True

    
     # 通信最初の50要素から読み(通信のゴミを読み飛ばす) 
    print("通信バッファクリア中……")
    for i in range(50):
       ser.readline().rstrip()                   #\r\n削除

    print("測定開始")       
    
    f_now = datetime.datetime.now()                    #現在時刻取得
    #　ファイル名を時刻.csvに設定
    recv_data = f_now.strftime('%Y%m%d_%H%M') + ".csv"
    with open(recv_data,'w') as f:                    #ファイルオープン
        while True:
            if stop_flag == False:
                break
            else:
                line = ser.readline().rstrip()         #\r\n削除
                s_line = str(line)                     #文字列化
                w_line = s_line.strip("b'")            #受信文字から"b、'"を削除

                fl_line = float(w_line)
                SEN_OUT = (fl_line * 5.0)/1023	　　　# 0-5vに変換
                SEN_OUT2 = round(SEN_OUT,2)            
                now = datetime.datetime.now()             #現在時刻取得
            
                #ファイルフォーマットを整えて書き込み
                file_time0 = now.strftime('%H')
                f.write(str(file_time0))
                f.write(":")
                file_time1 = now.strftime('%M')
                f.write(str(file_time1))
                file_time2 = now.strftime('%S')
                f.write(":") 
                f.write(str(file_time2))
                f.write(",")
                f.write(str(SEN_OUT2))                    #受信データ書き込み
                f.write(",")
                f.write("V")
                f.write(",\n")                          #","と改行挿入

                print("  " + (str(SEN_OUT2)) + "  V")

    f.close()
    ser.close()
    print("測定終了") 
    time.sleep(5)
    root.destroy()    

def start():
    global stop_flag
    global thread

    # スレッドが無いなら生成してstart()する
    if not thread:
        thread = threading.Thread(target = serial_data_recv)
        stop_flag = False
        thread.start()
    
def stop():
    global stop_flag
    global thread      

    ser.close()
    root.destroy()    

    # スレッドがある場合停止してjoin()する
    if thread:
        stop_flag = True
        thread.join()
        thread = None


thread = None
 
root = tk.Tk()
root.title("検査  電圧計")
root.geometry("250x220+1000+10")

#thread_1 = threading.Thread(target = start)
#thread_2 = threading.Thread(target = stop)

button1 = tk.Button(root, text = "測定開始", font=("MSゴシック", 14),
        command = start)
button1.place(x = 80, y = 30)

button2 = tk.Button(root, text = "測定終了", font=("MSゴシック", 14),
        command = stop)
button2.place(x = 80, y = 70)

root.mainloop()

試したこと

ネットで調べましたが、わからないでいます。

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

開発ツールは、Windows10でspyder3を使用しています。

Answer 1

fl_line = float(w_line)
ValueError: could not convert string to float: 

float に変換できないエラーなので、変数 w_line の値を疑いましょう。

                line = ser.readline().rstrip()         #\r\n削除
                s_line = str(line)                     #文字列化
                w_line = s_line.strip("b'")            #受信文字から"b、'"を削除

怪しい・・・

print('w_line':, w_line)

してみたら何が表示されますか？

Comments

[sasamata]

ご教授、ありがとうございます。
w_line： 632
3.09 V
と、表示されます。計算すると正しい値のようです。

[shiracamus]

変数 w_line には "632\n3.09V" という複数行の文字列が入っているのではありませんか？
print('w_line':, repr(w_line))
に変えて表示してみてください。

[sasamata]

ご教授、ありがとうございます。
複数行の文字列は入っていないようです。

さらに、色々と試してみたところ、期待の動作をするようになりましたが、
再起動時、「カーネル停止しました。再起動中」と表示され、その後、通常に動作するようです。

import tkinter as tk
import datetime
import serial
import time
import threading

ser = serial.Serial("COM6",9600)

def serial_data_recv():
global stop_flag
global thread
stop_flag = True

num = 0
# SEN_OUT = 0

# 通信最初の50要素から読み(通信のゴミを読み飛ばす)
print("通信バッファクリア中……")
for i in range(50):
ser.readline().rstrip() #\r\n削除

print("測定開始")

f_now = datetime.datetime.now() #現在時刻取得
#　ファイル名を時刻.csvに設定
recv_data = f_now.strftime('%Y%m%d_%H%M') + ".csv"
with open(recv_data,'w') as f: #ファイルオープン
while True:
if stop_flag == False:
break
else:
line = ser.readline().rstrip() #\r\n削除
s_line = str(line) #文字列化
w_line = s_line.strip("b'") #受信文字から"b、'"を削除

fl_line = float(w_line)
SEN_OUT = (fl_line * 5.0)/1023 # 0-5vに変換
SEN_OUT2 = round(SEN_OUT,2)
now = datetime.datetime.now() #現在時刻取得

#ファイルフォーマットを整えて書き込み
file_time0 = now.strftime('%H')
f.write(str(file_time0))
f.write(":")
file_time1 = now.strftime('%M')
f.write(str(file_time1))
file_time2 = now.strftime('%S')
f.write(":")
f.write(str(file_time2))
f.write(",")
f.write(str(SEN_OUT2)) #受信データ書き込み
f.write(",")
f.write("V")
f.write(",\n") #","と改行挿入

print(" " + (str(SEN_OUT2)) + " V")
num += 1

f.close()
ser.close()
print("測定終了")
time.sleep(5)
root.destroy()

def stop():
global stop_flag
global thread
stop_flag = False

root = tk.Tk()
root.title("検査 静電容量")
root.geometry("250x220+1000+10")

thread_1 = threading.Thread(target = serial_data_recv)
thread_2 = threading.Thread(target = stop)

button1 = tk.Button(root, text = "測定開始", font=("MSゴシック", 14),
command = thread_1.start)
button1.place(x = 80, y = 30)

button2 = tk.Button(root, text = "測定終了", font=("MSゴシック", 14),
command = thread_2.start)
button2.place(x = 80, y = 70)

root.mainloop()

[shiracamus]

コメントにソースコードを貼ってもインデントがずれてしまうので、質問欄に追記するようにしてください。

[sasamata]

申し訳ありません。
再度、そのようにします。

Answer 2

stop()内で、スレッドの終了を待ってから、以下を実施してはいかがでしょうか？
順番を入れ替えるだけですが。

    ser.close()
    root.destroy()    

---

追記 -->

コメント内容から、こちらの想定と違ったのかと思ったのですが、その後にベストアンサーとなったので、一応補足しておきます。

当初のコードでは、stopメソッドを呼び出すと、すぐにserをクローズしています。その後にスレッドの終了待ちを行なっているため、serクローズ直後に読み出したデータが不正データになるのではないかと思いました。
これを、きちんとデータを読み終え、スレッドが終了するのを待ってから、serをクローズ、rootをdestroyすればと思った次第です。
ただ、serクローズ状態でデータを読もうとすると、例外が発生するかもしれず、今回のような状況になるのかはわからなかったので、まずは試していただいた方がと考えました。
なんとなく複合的な要因のようにも思いますが、本問題だけでなく、スレッド処理の際には、タイミングについても気をつけた方が良いという点を認識して貰えたらと思い、追記しました。

# original code
def stop():
    global stop_flag
    global thread      

    ser.close()
    root.destroy()    

    # スレッドがある場合停止してjoin()する
    if thread:
        stop_flag = True
        thread.join()
        thread = None

# modified code
def stop():
    global stop_flag
    global thread      

    # スレッドがある場合停止してjoin()する
    if thread:
        stop_flag = True
        thread.join()
        thread = None

    ser.close()
    root.destroy()    

また、シリアルからのデータ読み込みの際に、以下のようにバイト列を文字列にしていますが、

line = ser.readline().rstrip()
s_line = str(line) #文字列化
w_line = s_line.strip("b'") #受信文字から"b、'"を削除

python3の場合は、以下のようにした方がよろしいかと思います。

line = ser.readline().rstrip()
w_line = line.decode() # あるいは w_line = line.decode('utf-8')

Comments

[sasamata]

ありがとうございます。
変更してみましたが、同じエラーメッセージが出力されます。