# Python Forum

Full Version: Image convoluion
You're currently viewing a stripped down version of our content. View the full version with proper formatting.
Hi, im a student of software engineering and i've been tasked to apply convolutions to images.
I have to use the basic convolution kernels that are on wikipedia, im mainly focusing on sharpen which is giving me strange artifacts but i do now know why since i think my code is good. Do you have any tips?

Convolution is simply an element-wise multiplication of two matrices followed by a sum.

- Take two matrices (which both have the same dimensions).
- Multiply them, element-by-element (i.e., not the dot-product, just a simple multiplication).
- Sum the elements together

Enjoy
Webchefz Infotech
Yeah, im doing that as you can see in the Image posted bellow and im trying to clip the values to 0, 255 so it wont mess things up, but they still od.
Maybe I'm missing something. Once you have the outputImage array, why not use np.max() to find the maximum value, then scale all pixels by multiplying by 255/max?
That sounds like a good idea, will try that, Thanks!

But yeah, that did not work out... Still some strange errors.
This is my code, as you can see i have some other functions but they work just fine, only sharpen is giving me some errors.
```def sharpen(image):
kernel =  np.array([
[0, -1, 0],
[-1, 5, -1],
[0, -1, 0],
])

outputImage = np.zeros_like(image)

if (image.ndim == 2):

for i in range(image.shape[0]):
for j in range(image.shape[1]):

elif(image.ndim == 3):

for i in range(image.shape[0]):
for j in range(image.shape[1]):

return outputImage

args = sys.argv[1:]
args = ["--sharpen", "LennaBnW.png", "LennaSharpen.png"]
inputFilePath = args[-2]
outputFilePath = args[-1]

image1 = Image.open(inputFilePath)
imgArray = np.asarray(image1, dtype = np.uint8);

for arg in args:
if (arg == "--rotate"):
imgArray = rotate(imgArray)
if (arg == "--mirror"):
imgArray = mirror(imgArray)
if (arg == "--inverse"):
imgArray = inverse(imgArray)
if (arg == "--bw"):
imgArray = bw(imgArray)
if (arg == "--lighten"):
index = args.index("--lighten")
imgArray = lighten(imgArray, int(args[index + 1]))
if (arg == "--darken"):
index = args.index("--darken")
imgArray = darken(imgArray, int(args[index + 1]))
if (arg == "--sharpen"):
imgArray = sharpen(imgArray)

imgArray = (255/np.max(imgArray)) * imgArray
np.clip(imgArray, 0, 255)

if (imgArray.ndim == 3):
image = Image.fromarray(imgArray.astype(np.uint8), 'RGB')
elif (imgArray.ndim == 2):
image = Image.fromarray(imgArray.astype(np.uint8), 'L')

image.save(outputFilePath)
```
So if you can find error in that i would be really glad.
I found the solution, which increased the time from 7 seconds to 30, but the results are correct.
All that needs to be done is to add clipping, but not after the output image is calculated, the clipping needs to be added to the lines 18, 31, 32 and 33 after the sum() function.
I repeat, it increases the computational time like hell, but it works.
Thanks to everybody for trying to help me!