二重ループの計算結果をリストに保管する方法

Question

画像をグレースケール化し、差分を取った後にリストに計算結果をまとめて表示させ要素しているのですがうまく行きません。良い方法はないでしょうか

import cv2
import numpy as np

agazou = cv2.imread("a.jpg",0)
bgazou = cv2.imread("b.jpg",0)

rows, cols = agazou.shape[:2]

list =[[0 for i in range(rows)] for j in range(cols)]

for x in range(rows):

    for y in range(cols):
       
       g1 = agazou [x,y]
       g2 = bgazou [x,y]
       
       r = g1 - g2
       
       list.append(r)

print(list[10][10])

Answer 1

list[y][x] = rということでしょうか。

なお、変数名listは基本的に使うのを避けてください。str, tuple, dictなども同様です。

Pythonで変数名をつけるときには予約語だけでなく組み込み関数との衝突も気を付けましょう - Qiita

また、numpy配列の差分はそのままベクトル演算で取ることも可能なので、実際はforループすら不要で

result = agazou - bgazou

でいいと思います。こちらの方が速度もはるかに速いはずです。

Comments

[surarinmon]

ご指摘ありがとうございます。list[y][x] = rとしてy,xの値から結果が出せるようなプログラムを作成しようとしていました。

ベクトル演算の件も存じ上げなかったので助かりました。もしlistに値を保存する場合はlist[y][x] = r

をfor文のなかにいれて　print(list[y][x])とすればよいのでしょうか

[hayataka2049]

良いはずだけど、それで望みの結果になるかどうかはご自身で確認してください。