undefined_name/image_python
5
0 Fork 0
Code Issues 1 Pull Requests 2 Packages Projects 1 Releases Wiki Activity

Filtre de Canny terminé; #7

Open
Darkan wants to merge 1 commits from filter_canny into master
pull from: filter_canny
merge into: undefined_name:master
Conversation 2 Commits 1 Files Changed 4
+241 -59
Darkan commented 2022-11-11 09:18:11 +01:00
Member
Copy link
Copy source

Ajout des fonctions utiles au filtre de Canny dans usefull_func.py : calculGradient, copyNullMatrix, tranform_angle, dltNoMaxima, get_norm, norm_voisin, seuillageHysteresis;

Pas de modification important dans sobel.py

En tout cas il fonctionne et reste quelque ajust à faire mais pour plus tard... :-)

Ajout des fonctions utiles au filtre de Canny dans [usefull_func.py](https://git.herisson.ovh/undefined_name/image_python/src/branch/master/imageEngine/filters/usefull_func.py) : `calculGradient`, `copyNullMatrix`, `tranform_angle`, `dltNoMaxima`, `get_norm`, `norm_voisin`, `seuillageHysteresis`; Pas de modification important dans [sobel.py](https://git.herisson.ovh/undefined_name/image_python/src/branch/master/imageEngine/filters/sobel.py) En tout cas il fonctionne et reste quelque ajust à faire mais pour plus tard... :-)
Darkan added 1 commit 2022-11-11 09:18:19 +01:00
Filtre de Canny terminé; 4aa559772c 
Ajout des fonctions utiles au filtre de Canny dans 'usefull_func.py'
Pas de modification important dans 'sobel.py'
tonitch added the enhancement label 2022-11-11 12:37:45 +01:00
imageEngine/filters/canny.py
@@ -40,0 +16,4 @@
return contours
"""
tonitch commented 2022-11-11 12:41:34 +01:00
Owner
Copy link
Copy source

ca ne fonctionne pas que tu l'ais mis en commentaire ?

ca ne fonctionne pas que tu l'ais mis en commentaire ?
Darkan commented 2022-11-11 14:28:21 +01:00
Author
Member
Copy link
Copy source

Le code est bon mais ça marche pas, j'ai sûrement quelques trucs à régler dans mon calcul de convolution. Du coup je le laisse ça en attendant d'avoir corriger le truc.

Le code est bon mais ça marche pas, j'ai sûrement quelques trucs à régler dans mon calcul de convolution. Du coup je le laisse ça en attendant d'avoir corriger le truc.
tonitch marked this conversation as resolved
imageEngine/filters/usefull_func.py
@@ -21,0 +17,4 @@
for x in range(len(mat)):
for y in range(len(mat[0])):
coord_i = i - (len(mat) // 2) + x
corrd_j = j - (len(mat[0]) // 2) + y
tonitch commented 2022-11-11 12:43:22 +01:00
Owner
Copy link
Copy source

Petite typo ici ?

Petite typo ici ?
imageEngine/filters/usefull_func.py
@@ -52,3 +57,4 @@
def pixel(img, i, j, default=(0,0,0)):
#i la colone et j la ligne
tonitch commented 2022-11-11 12:45:40 +01:00
Owner
Copy link
Copy source

pourrais être x, y ou row, collumn pour être plus explicite?

pourrais être x, y ou row, collumn pour être plus explicite?
imageEngine/filters/usefull_func.py
@@ -72,2 +78,4 @@
def filtre_gaussien(mat_img):
mat_gauss = [
[2/159, 4/159, 5/159, 4/159,2/159],
tonitch commented 2022-11-11 13:05:22 +01:00
Owner
Copy link
Copy source

tricheur haha

tricheur haha
imageEngine/filters/usefull_func.py
@@ -106,0 +117,4 @@
def calculGradient(filtred_image):
mask_x = [[1, 0, -1]]
mask_y = [[1],[0],[-1]]
tonitch commented 2022-11-11 12:47:23 +01:00
Owner
Copy link
Copy source

tu peux pas appliquer une seule convolution en fesant une matrice normalement ?

tu peux pas appliquer une seule convolution en fesant une matrice normalement ?
Darkan commented 2022-11-11 14:22:19 +01:00
Author
Member
Copy link
Copy source

Non, j'ai besoin de la valeur que donne les 2 matrices pour calculer le radient de l'angle de chaque pixel, et aussi calculer la norme.

Non, j'ai besoin de la valeur que donne les 2 matrices pour calculer le radient de l'angle de chaque pixel, et aussi calculer la norme.
tonitch commented 2022-11-11 14:53:58 +01:00
Owner
Copy link
Copy source

ha ouai je confonds haha

ha ouai je confonds haha
imageEngine/filters/usefull_func.py
@@ -106,0 +136,4 @@
return norme_gradient, angle_normal_gradient
def copyNullMatrix(mat):
tonitch commented 2022-11-11 13:10:55 +01:00
Owner
Copy link
Copy source

peut être fait en une ligne

nullmat = [[0 for i in j] for j in mat]

Je pense que c'est plus éfficace que de faire une copie puis ensuite de passer dans toute la matrice pour tout transformer

pire des cas tu peux le faire de manière plus explicite aussi

nullmat = list()
for i in range(len(mat)):
	nullmat.append(list())
	for _ in mat[i]:
    	nullmat[i].append(0)
return nullmat

ca fait la même chôse

peut être fait en une ligne ```python nullmat = [[0 for i in j] for j in mat] ``` Je pense que c'est plus éfficace que de faire une copie puis ensuite de passer dans toute la matrice pour tout transformer pire des cas tu peux le faire de manière plus explicite aussi ```python nullmat = list() for i in range(len(mat)): nullmat.append(list()) for _ in mat[i]: nullmat[i].append(0) return nullmat ``` ca fait la même chôse
Darkan commented 2022-11-11 14:23:45 +01:00
Author
Member
Copy link
Copy source

Je suis pas encore assez à l'aise avec python pour savoir faire des trucs comme ça 😢

Je suis pas encore assez à l'aise avec python pour savoir faire des trucs comme ça 😢
tonitch commented 2022-11-11 14:54:56 +01:00
Owner
Copy link
Copy source

alors la deuxième méthode est pour toi, ce que tu fais c'est une copie de de la matrice alors que tout ce que tu as besoin c'est de la taille ^^ ca va améliorer pas mal de faire la deuxième méthode que je t'ai montré

alors la deuxième méthode est pour toi, ce que tu fais c'est une copie de de la matrice alors que tout ce que tu as besoin c'est de la taille ^^ ca va améliorer pas mal de faire la deuxième méthode que je t'ai montré
imageEngine/filters/usefull_func.py
@@ -106,0 +284,4 @@
return mx
def Minim(mat):
mn = 1000
tonitch commented 2022-11-11 13:01:24 +01:00
Owner
Copy link
Copy source

en vrai ca devrais pas poser de problème mais tu peux le faire de manière différente:

mn = mat[0][0]
for i in mat:
	for j in i:
    	if(i < mn):
        	mn = j
return mn

c'est un détail parce que je pense pas que tu vas avoir des valeurs supérieurs à 1000 dans tes matrice mais techniquement on sais jamais, ca peut être plus sur

en vrai ca devrais pas poser de problème mais tu peux le faire de manière différente: ```python mn = mat[0][0] for i in mat: for j in i: if(i < mn): mn = j return mn ``` c'est un détail parce que je pense pas que tu vas avoir des valeurs supérieurs à 1000 dans tes matrice mais techniquement on sais jamais, ca peut être plus sur
tonitch commented 2022-11-11 13:12:23 +01:00
Owner
Copy link
Copy source

Quelques petit trucs que j'ai remarqué... j'ai pas tout regardé tho ^^

Quelques petit trucs que j'ai remarqué... j'ai pas tout regardé tho ^^
Darkan was assigned by tonitch 2022-11-11 13:12:58 +01:00
tonitch added this to the Avancement project 2022-11-11 13:13:02 +01:00
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin filter_canny:filter_canny
git checkout filter_canny
Sign in to join this conversation.
Reviewers
No Reviewers
tonitch
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels enhancement
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Avancement
Uncategorized
Assignees
Clear assignees
tonitch (Debucquoy Anthony)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: undefined_name/image_python#7