raspberrypiで近接センサを動かして、ADコンバータで出力を得たい

Question

raspberry piから給電して
・近接センサ_LBR-127HLD
・ADコンバータ_MCP3208
を動かしています。

近接センサのエミッタ側からADコンバータの入力を得ています。
ひとまずちゃんとした出力になっているか、Thonnyで以下のコードを書いて試してみました。
物を近づけたり遠ざけたりすると0~1の間で出力が返ってくるのですが、
30秒程度経つとフリーズしてしまいctrl+cも利かなくなってしまいます。
ただし、raspberry pi自体が操作できない訳ではなく、Interruptのボタンを押すとエラーは出ますがちゃんと終了できます。

コードか配線が間違っているのでしょうか。教えていただけないでしょうか。

from gpiozero import MCP3208
import RPi.GPIO as GPIO
import time

while True:
    time.sleep(1)
    print(MCP3208(channel = 0).value)

<2019/12/12/17:31_追記>
どこでエラーが発生しているか確認するためにあちこちにprint文を入れました。

from gpiozero import MCP3208
import RPi.GPIO as GPIO
import time

GPIO.setmode(GPIO.BCM)
GPIO.setup(21,GPIO.OUT)

while True:
    time.sleep(1)
    print(MCP3208(channel = 0).value)
    if MCP3208(channel = 0).value > 0.01:
        GPIO.output(21,GPIO.HIGH)
        print('low')
else:
    print('error')

GPIO.cleanup()
print('not_true')

shellには以下のように少しだけ動いてからエラーがでました

0.0006104260774020265
low
0.0006104260774020265
low
0.0006104260774020265
low
0.0006104260774020265
low
0.0023196190941277006
low
0.004517152972774996
low
0.0006104260774020265
low
0.0059821755585398595
low
0.015993163227933094
high
0.015016481504089851
high
0.0006104260774020265
low
0.0006104260774020265
low
0.0006104260774020265
low
0.0006104260774020265
low
0.0006104260774020265
low
0.0006104260774020265

ここでフリーズしてしまったのでctrl+cを打つと、

Exception ignored in: <bound method SharedMixin.__del__ of SPI(closed)>
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/gpiozero/mixins.py", line 137, in __del__
    super(SharedMixin, self).__del__()
  File "/usr/lib/python3/dist-packages/gpiozero/devices.py", line 122, in __del__
    self.close()
  File "/usr/lib/python3/dist-packages/gpiozero/devices.py", line 82, in close
    old_close()
  File "/usr/lib/python3/dist-packages/gpiozero/pins/local.py", line 102, in close
    self.pin_factory.release_all(self)
  File "/usr/lib/python3/dist-packages/gpiozero/pins/__init__.py", line 85, in release_all
    with self._res_lock:
  File "/usr/lib/python3/dist-packages/thonny/shared/thonny/backend.py", line 133, in signal_handler
    raise KeyboardInterrupt("Execution interrupted")
KeyboardInterrupt: Execution interrupted
KeyboardInterrupt: Forced reset
>>> 

と表示されました。

Answer 1

ysk_snnさん、

以下のコードは、どうでしょうか？

from gpiozero import MCP3208
import RPi.GPIO as GPIO
import time

ch = MCP3208(channel = 0)

while True:
    time.sleep(1)
    print(ch.value)

MCP3208のインスタンスを作りまくってるのが原因？

Comments

[ysk_snn]

ご回答ありがとうございます！
おっしゃるとおりのコードでは正常に動き続けています！

恥ずかしながら「インスタンス」について今回のご指摘をいただいてから調べたのですが、
私のコードのどこでインスタンスの過剰生成が行われているのか良く理解できませんでした...

よろしければ再度ご教示いただけないでしょうか。

[ysk_snn]

```python3
from gpiozero import MCP3208
import RPi.GPIO as GPIO
import time

while True:
time.sleep(1)
print(MCP3208(channel = 0).value)
```
の場合だと、print文において「MCP3208(channel = 0) = 何か値」というインスタンスを生成するのかな、と考えているのですが、

```python3
from gpiozero import MCP3208
import RPi.GPIO as GPIO
import time

ch = MCP3208(channel = 0)

while True:
time.sleep(1)
print(ch.value)
```
とした場合はprint文において「ch = 何か値」というインスタンスを生成することにはならないのでしょうか？

[mt08]

ysk_snnさん、

while True: 内に "MCP3208(channel = 0)" があるので、ループごとに、呼び出されてしまうのが、問題かと。

gpiozeroのサンプルのページ(https://gpiozero.readthedocs.io/en/stable/recipes.html)で、`MCP` で検索してみてください。
どのサンプルもループの前に、変数に代入して、それをループ内でつかってますよね。

ソースがgithubにあるので(https://github.com/gpiozero/gpiozero/blob/master/gpiozero/spi_devices.py)、追ってみるのはどうでしょう？

[ysk_snn]

mt08様

何度もありがとうございます。
サンプルコードのまねをして使ってみようと思います。

メモリやインスタンスの勉強になりました。
またよろしくお願いいたします。