OpenCV : AttributeError: 'list' object has no attribute 'queryIdx'

Essex picture Essex · Jan 6, 2017 · Viewed 7.3k times · Source

I'm using OpenCV, SIFT and Homography in order to detect all objects in a picture.

My global picture looks like :

enter image description here

And I want to detect all lamps on the picture, even if the orientation is not exactly the same between each one.

My model picture looks like :

enter image description here

I wrote this script with Python :

#-*- coding: utf-8! -*-

import os, shutil
import numpy as np
import cv2

#########################
# SIFT descriptors part #
#########################

img1 = cv2.imread('/Users/test/Desktop/SIFT/Ville/ville.jpg',0)
img2 = cv2.imread('/Users/test/Desktop/SIFT/Ville/lampe.jpg',0)

##########################
# Initiate SIFT detector #
##########################

sift = cv2.xfeatures2d.SIFT_create()

kp1, des1 = sift.detectAndCompute(img1,None)
kp2, des2 = sift.detectAndCompute(img2,None)

FLANN_INDEX_KDTREE = 0
index_params = dict(algorithm = FLANN_INDEX_KDTREE, trees = 5)
search_params = dict(checks = 50)

flann = cv2.FlannBasedMatcher(index_params, search_params)
#bf = cv2.BFMatcher()
matches = flann.knnMatch(des1,des2,k=2)

good = []
for m,n in matches :
    if m.distance < 0.7*n.distance :
        good.append([m])

MIN_MATCH_COUNT = 3

if len(good)>MIN_MATCH_COUNT:
    src_pts = np.float32([ kp1[m.queryIdx].pt for m in good ]).reshape(-1,1,2)
    dst_pts = np.float32([ kp2[m.trainIdx].pt for m in good ]).reshape(-1,1,2)

    M, mask = cv2.findHomography(src_pts, dst_pts, cv2.RANSAC,5.0)
    matchesMask = mask.ravel().tolist()

    h,w = img1.shape
    pts = np.float32([ [0,0],[0,h-1],[w-1,h-1],[w-1,0] ]).reshape(-1,1,2)
    dst = cv2.perspectiveTransform(pts,M)

    img2 = cv2.polylines(img2,[np.int32(dst)],True,255,3, cv2.LINE_AA)

else:
    print "Not enough matches are found - %d/%d" % (len(good),MIN_MATCH_COUNT)
    matchesMask = None

draw_params = dict(matchColor = (0,255,0), # draw matches in green color
                   singlePointColor = None,
                   matchesMask = matchesMask, # draw only inliers
                   flags = 2)

img3 = cv2.drawMatches(img1,kp1,img2,kp2,good,None,**draw_params)

plt.imshow(img3, 'grayfinal.jpg'),plt.show()
cv2.imwrite('matches.jpg',img3)

I get this error :

Traceback (most recent call last):
  File "image.py", line 34, in <module>
    src_pts = np.float32([ kp1[m.queryIdx].pt for m in good ]).reshape(-1,1,2)
AttributeError: 'list' object has no attribute 'queryIdx'

Do you have an idea ?

EDIT :

With assumed solutions, I got something which looks right. But How I could detect the others lamps on the picture ?

enter image description here

Answer

wencakisa picture wencakisa · Jan 6, 2017

You should append only m to good, instead of doing good.append([m]).

Because right now each element in good is a list with one element (m), and you try to access its queryIdx. That's why you receive this AttributeError: 'list' object has no attribute 'queryIdx'