exposed vl_phow

[simmimourya]

Fixed Errors.

[menpobot]

Can one of the admins verify this patch?

[patricksnape]

I would suggest that for methods like rgb2gray or equivalent where it is likely they will have more widespread use that we move them into a utils package or something.

Also, now would be a great time to add a test so we can check that everything is actually working correctly. The easiest thing to do is probably to run the phow function in Matlab on the Lena image. You will probably want to save the Lena image down from Python to ensure the pixel values are identical (and we don't get mixed up by Matlab vs Python image reading. Also, note that the Matlab output will be fortran ordered so you may need to do some transposing to get the shape correct.

[simmimourya]

To Do:

• [ ] Fix Line 82 descrs[si] = np.concatenate((d[:]), axis=0)
• [ ] Add test for vl_phow

[devkhan]

@simmimourya1 The CI builds are failing because of the cyvlfeat.sift.tests.sift_test.test_phow_frames_float_descriptors test. So you might want to take a look at it.

[simmimourya]

@devkhan Thanks for the suggestion. I'll check it.

[simmimourya]

• [ ] Add function doc-string in phow() and rgb2gray().
• [ ] Add more test cases.

[patricksnape]

@simmimourya1 Why did you delete this?

[menpobot]

Can one of the admins verify this patch?