edge detection opencv python

Question

it is my code and I'm trying to edge detection using the sobel method. But I'm getting neither error nor image.If I have a mistake, can you explain where it is?

import cv2
import numpy as np

img=cv2.imread("lena.jpg")
img=cv2.cvtColor(img,cv2.COLOR_BGR2GRAY)
m,n=img.shape


kernelx=np.array([[-1, -2, -1], [0, 0, 0], [1, 2, 1]])
kernely=np.array([[-1, 0, 1], [-2, 0, 2], [-1, 0, 1]])


gx=np.zeros((m,n), dtype=np.int32)
gy=np.zeros((m,n), dtype=np.int32)


for x in range(1,m-1):
    for y in range(1,n-1):
        for a in range(3):
            for b in range(3):
                xn = x + a - 1
                yn = y + b - 1
                gx=gx+(kernelx[a][b]*img[xn][yn])
                gy=gy+(kernely[a][b]*img[xn][yn])






        final_image=np.sqrt(pow(gx,2.0)+pow(gy,2.0))



cv2.imshow("a",final_image)
cv2.waitKey(0)
cv2.destroyAllWindows()

Are you able to write a program using imshow that displays any image (without trying to do edge detection on top)? — mkrieger1
– mkrieger1, Commented May 22, 2021 at 21:05
Can you write a working program that loads an image and displays it? — mkrieger1
– mkrieger1, Commented May 22, 2021 at 21:18

TzviLederer · Accepted Answer · 2021-05-23 07:22:09Z

2

You have an error in your code.
Try to use cv2.filter2D() instead of your for loop.

import cv2
import numpy as np

img = cv2.imread(r"lena.jpg")
img = cv2.cvtColor(img, cv2.COLOR_BGR2GRAY)
m, n = img.shape

kernelx = np.array([[-1, -2, -1], [0, 0, 0], [1, 2, 1]])
kernely = np.array([[-1, 0, 1], [-2, 0, 2], [-1, 0, 1]])

img_x = cv2.filter2D(img, ddepth=cv2.CV_8U, kernel=kernelx)
img_y = cv2.filter2D(img, ddepth=cv2.CV_8U, kernel=kernely)

final_image = np.sqrt(pow(img_x, 2.0) + pow(img_y, 2.0))

cv2.imshow("a", final_image)
cv2.waitKey(0)
cv2.destroyAllWindows()

if you want to do it with a for loop, the right way of doing it is:
I add a tqdm progress bar for you ;)

import cv2
import numpy as np
from tqdm import tqdm

img = cv2.imread(r"lena.jpg")
img = cv2.cvtColor(img, cv2.COLOR_BGR2GRAY)
m, n = img.shape

kernelx = np.array([[-1, -2, -1], [0, 0, 0], [1, 2, 1]])
kernely = np.array([[-1, 0, 1], [-2, 0, 2], [-1, 0, 1]])

out_im = np.zeros((m-2, n-2))
g_x = np.zeros((m-2, n-2))
g_y = np.zeros((m-2, n-2))
for m_i in tqdm(range(m-2)):
    for n_i in range(n-2):
        im_patch = img[m_i:m_i+3, n_i:n_i+3]
        g_x[m_i, n_i] = sum([i * k for i, k in zip(im_patch.flatten(), kernelx.flatten())])
        g_y[m_i, n_i] = sum([i * k for i, k in zip(im_patch.flatten(), kernely.flatten())])

out_im = np.sqrt(g_x ** 2 + g_y ** 2)

out_im = np.uint8(out_im/out_im.max() * 255)

cv2.imshow("", out_im)
cv2.waitKey(0)
cv2.destroyAllWindows()

Note:

You may need to normalize the image before display.

edited May 23, 2021 at 7:22

answered May 22, 2021 at 21:30

TzviLederer

1371 silver badge9 bronze badges

Sign up to request clarification or add additional context in comments.

11 Comments

izmirlikezzap Over a year ago

Actually,ı know this method but ı want to write like my code but ı dont know how it is

TzviLederer Over a year ago

The for loop does not do convolution or 2d filtering as it should do.

TzviLederer Over a year ago

Try to read the definition of convolution

mkrieger1 Over a year ago

But why does the program not respond anymore?

izmirlikezzap Over a year ago

your code is working.but I can't say the same for mine

|

Collectives™ on Stack Overflow

edge detection opencv python

1 Answer 1

Note:

11 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Note:

11 Comments

Your Answer

Sign up or log in

Post as a guest

Related