pythonでThreadingを使ってナンバリング設定がうまくいきません。

pythonに詳しい方に質問です。
写真を撮る瞬間にLEDで点灯するために、LEDを光らせて、写真を撮るという二つを並行して行おうとしてるのですがナンバリング設定をしてくださいとエラーが出ます。

初心者ですので変なミスかもしれないんですが、助言をお願いします。
以下プログラムです。

import time
import picamera
import datetime
import RPi.GPIO as GPIO
import threading

USER_NAME = "pi/";
HOME_DIR = "/home/" + USER_NAME
SAVE_DIR = HOME_DIR + "cameradata/";

datetime = datetime.datetime.today()
datetime_formatted = datetime.strftime("%Y%m%d%H%M%S")

file_name ="IMG_" datetime_formatted".jpg"

def camera():

 with picamera.PiCamera() as camera:
  camera.start_preview()

  time.sleep(2)

  camera.capture(SAVE_DIR file_name)

  camera.stop_preview()


def LED():

 GPIO.setmode(GPIO.BOARD)
 GPIO.setup(11,GPIO.OUT)
 for i in range(1):

  GPIO.output(11,GPIO.HIGH)
  time.sleep(2)
  GPIO.output(11,GPIO.LOW)
  time.sleep(2)



if __name__ == "__main__":

 p1 = threading.Thread(target = camera)
 p2 = threading.Thread(target = LED)

 p1.start()
 p2.start()

GPIO.cleanup()

以下が表示されるエラーです。
Exception in thread Thread-2:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
self.run()
File "/usr/lib/python2.7/threading.py", line 754, in run
self.__target(*self.__args, **self.__kwargs)
File "camera3.py", line 38, in LED
GPIO.output(11,GPIO.LOW)
RuntimeError: Please set pin numbering mode using GPIO.setmode(GPIO.BOARD) or GPIO.setmode(GPIO.BCM)

hayataka2049

2018/11/07 05:41

コードが読めないので、再編集して<code>ボタンで挿入できるコードブロックの中に入れるようにしてください。https://teratail.com/tour　https://teratail.storage.googleapis.com/uploads/contributed_images/56957fe805d9d7befa7dba6a98676d2b.gif

surarinmon

2018/11/07 05:44

申し訳ありません、修正しました。

can110

2018/11/07 06:08

「'」が「&quot」に、またインデントもなくなっています。実行できるコードに修正ください。

surarinmon

2018/11/07 06:15

おそらく修正できたと思います、申し訳ありません

can110

2018/11/07 06:19

修正ありがとうございます。しかしfor i in range(1):のあとの文がインデントされていないので実行できないはずです。本当にそのコードは実際のコードでしょうか？

surarinmon

2018/11/07 06:21

ラズベリーパイを使って行っているのを別のパソコンにコピーしたものです、誤字が多くて申し訳ないです。

行動規範の内容に同意します

回答2件

ベストアンサー

スレッドが実行中なのにメインスレッドでGPIO.cleanupを呼び出しているからです。
GPIO.cleanupをLED関数の最後で呼ぶようにしてみてください。
それから、LEDスレッドかcameraスレッドはメインスレッド上での実行でも問題ないと思います。

投稿2018/11/08 03:54

編集2018/11/08 03:57

frodo821

総合スコア322

もしかして以下のようなインデントになっていませんか？

Python
1# 略
2
3def LED():
4    print('GPIO.setmode(GPIO.BOARD)') # 確認！
5    GPIO.setmode(GPIO.BOARD)
6    GPIO.setup(11,GPIO.OUT)
7    for i in range(1):
8        print('GPIO.output(11,GPIO.HIGH)') # 確認！
9        GPIO.output(11,GPIO.HIGH)
10        time.sleep(2)
11print('GPIO.output(11,GPIO.LOW)') # 確認！
12GPIO.output(11,GPIO.LOW) # ここ！
13time.sleep(2)
14
15# 略