PythonとOpen CVを使用し、画像を二値化後、黒の面積の割合を条件として使用したいです。

Question

プログラミング初心者です。PythonとOpen CVを使用して画像を二値化し、黒と白の面積の割合を求め、求めた面積の割合を条件として使えないかと思い、下記のプログラムを試作してみました。

import RPi.GPIO as GPIO             #GPIO用のモジュールをインポート
import time                         #時間制御用のモジュールをインポート
import cv2

def Black_Area():
    print('Black_Area='+str(black_area/whole_area*100)+'%')

#画像の読み込み
img=cv2.imread('photo.jpg',0)
#特定の範囲のGaussian分布から閾値を自動で決めて二値化
ret1,img_th=cv2.threshold(img,0,255,cv2.THRESH_OTSU)
#全体の画素数
whole_area=img_th.size
#白部分の画素数
white_area=cv2.countNonZero(img_th)
#黒部分の画素数
black_area=whole_area-white_area

#それぞれの割合を表示
print('White_Area='+str(white_area/whole_area*100)+'%')
print('Black_Area='+str(black_area/whole_area*100)+'%')


#画像の表示
cv2.imshow("OpenCV_th",img_th)
cv2.destroyAllWindows()

#GPIOの設定
GPIO.setmode(GPIO.BOARD)              #GPIOのモードを"GPIO.BOARD"に設定
GPIO.setup(15, GPIO.OUT)       #GPIO15を出力モードに設定

#GPIOの電圧を制御
if 50 < Black_Area:                #黒の面積の割合が５０％より大きい場合
    GPIO.output(15, GPIO.HIGH)     #GPIO15の出力をHigh(3.3V)にする
    time.sleep(5)                       #5秒間待つ
    GPIO.output(15, GPIO.LOW)      #GPIO15の出力をLow(0V)にする

#GPIOをクリーンアップ
GPIO.cleanup()
```### 前提・実現したいこと
二値化した画像から白と黒の面積の割合を求め、今回は黒の面積の割合を条件として扱い、黒の面積の割合が50%よりも大きいときにLEDを点灯させたいです。ですが、うまくいかず以下のようなエラーメッセージが出ました。

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

```test,LED.py:30: RuntimeWarning: This channel is already in use, continuing anyway.  Use GPIO.setwarnings(False) to disable warnings.
  GPIO.setup(15, GPIO.OUT)       #GPIO15を出力モードに設定
Traceback (most recent call last):
  File "/home/pi/Desktop/並列処理/test,LED.py", line 33, in <module>
    if 50 < Black_Area:                #黒の面積の割合が５０％より大きい場合
TypeError: '<' not supported between instances of 'int' and 'function'

試したこと

最初は条件の部分で使用している「Black_Area」を定義せずに使用したため「NameError」と出ました。そのため、「def」を使用して「Black_Area」定義してみましたがうまくいきませんでした。どなたか解決策を教えていただけないでしょうか。

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

Python 3.7.2 , Open CV 4.5.3

Answer 1

TypeError: '<' not supported between instances of 'int' and 'function'

は、Google翻訳で翻訳すると

TypeError： '<'は 'int'と 'function'のインスタンス間ではサポートされていません

です。

def Black_Area():
    print('Black_Area='+str(black_area/whole_area*100)+'%')

を関数ではなく

Black_Area = black_area/whole_area*100

のように変数にしましょう。

Comments

[ryu-12]

ご回答ありがとうございます。これは、
def Black_Area():
print('Black_Area='+str(black_area/whole_area*100)+'%')
の部分を　Black_Area = black_area/whole_area*100　に変えるということでしょうか？
もしそうであれば、
Traceback (most recent call last):
File "/home/pi/Desktop/並列処理/test,LED.py", line 5, in <module>
Black_Area = 'Black_Area='+str(black_area/whole_area*100)+'%'
NameError: name 'black_area' is not defined
というエラーが出てしまいました。繰り返し質問することになり申し訳ございませんが、どうすれば解決できるでしょうか？

[ppaul]

いいえ、
def Black_Area():
print('Black_Area='+str(black_area/whole_area*100)+'%')
を消して、そのかわりに
Black_Area = black_area/whole_area*100
を入れるという意味です。

[jbpb0]

質問者さん

Black_Area = black_area/whole_area*100

を追加する場所は、

#黒部分の画素数
black_area=whole_area-white_area

と

#GPIOの電圧を制御
if 50 < Black_Area:

の間のどこか、です

[ryu-12]

皆さん
修正すべき点のご指摘、本当にありがとうございます。ご指摘の通りに、
def Black_Area():
Black_Area = black_area/whole_area*100
に変えた後、条件の部分を、
if 50 < Black_Area:
から、
if 50 < black_area/whole_area*100:
に変えたところ、成功しました。ご丁寧に教えてくださった皆様、本当にありがとうございました。
自己解決欄に成功したプログラムを貼っておきます。

Answer 2

import RPi.GPIO as GPIO         #GPIO用のモジュールをインポート
import time                     #時間制御用のモジュールをインポート
import cv2
GPIO.setwarnings(False)

def Black_Area():
     Black_Area = black_area/whole_area*100

#画像の読み込み
img=cv2.imread('photo.jpg',0)
#特定の範囲のGaussian分布から閾値を自動で決めて二値化
ret1,img_th=cv2.threshold(img,0,255,cv2.THRESH_OTSU)
#全体の画素数
whole_area=img_th.size
#白部分の画素数
white_area=cv2.countNonZero(img_th)
#黒部分の画素数
black_area=whole_area-white_area

#それぞれの割合を表示
print('White_Area='+str(white_area/whole_area*100)+'%')
print('Black_Area='+str(black_area/whole_area*100)+'%')


#画像の表示
cv2.imshow("OpenCV_th",img_th)
cv2.destroyAllWindows()

#GPIOの設定
GPIO.setmode(GPIO.BOARD)              #GPIOのモードを"GPIO.BOARD"に設定
GPIO.setup(15, GPIO.OUT)       #GPIO15を出力モードに設定

#GPIOの電圧を制御
if 50 < black_area/whole_area*100:                #黒の面積の割合が５０％より大きい場合
    GPIO.output(15, GPIO.HIGH)     #GPIO15の出力をHigh(3.3V)にする
    time.sleep(5)                       #5秒間待つ
    GPIO.output(15, GPIO.LOW)      #GPIO15の出力をLow(0V)にする

#GPIOをクリーンアップ
GPIO.cleanup()

Comments

[jbpb0]

def Black_Area():
Black_Area = black_area/whole_area*100

は要りません
その関数は使われてないので

[ryu-12]

jbpb0さん
ご指摘、ありがとうございます。ご指摘の通りにdefの部分を消しても問題なく動作しました。