Python bindings for Earth mover Distance EMD() are missing (Bugfix #3406)

Added by Pradeep Kumar J over 11 years ago. Updated almost 10 years ago.

Status:Open Start date:2013-11-28
Priority:Normal Due date:
Assignee:Pradeep Kumar J % Done:

0%
Category:python bindings
Target version:-
Affected version:2.4.9 (latest release) Operating System:Any
Difficulty: HW Platform:x86
Pull request:

Description

Python bindings (and possibly for other languages) for Earth Mover Distance EMD are missing in master and latest release.
Can be fixed by using CV_EXPORTS_W instead of CV_EXPORTS in line 1352 of imgproc.hpp (in master-branch).

Change from:
CV_EXPORTS float EMD( InputArray signature1, InputArray signature2,
int distType, InputArray cost=noArray(),
float* lowerBound = 0, OutputArray flow = noArray() );
to:
CV_EXPORTS_W float EMD( InputArray signature1, InputArray signature2,
int distType, InputArray cost=noArray(),
float* lowerBound = 0, OutputArray flow = noArray() );

Associated revisions

Revision 4763d803
Added by Vadim Pisarevsky over 10 years ago

Merge pull request #3406 from shiquanwang:Fix-bug-#4003_build-python-binding-under-python3

History

#1
Updated by Nikita Manovich about 11 years ago

Hi,

Thank you for the bug report. Could you please help us and submit a pull request for the problem? It will help us a lot. You can find all necessary information here: http://www.code.opencv.org/projects/opencv/wiki/How_to_contribute

  • Assignee changed from Vadim Pisarevsky to Pradeep Kumar J
  • Status changed from New to Open

#2
Updated by Steven Puttemans almost 11 years ago

Pull request added but rebased to 2.4

  • Pull request set to https://github.com/Itseez/opencv/pull/2759
  • Affected version changed from 2.4.7 (latest release) to 2.4.9 (latest release)

#3
Updated by Steven Puttemans almost 11 years ago

@Pradeep Kumar : Can you reconfirm that this fix doesn't break the use of the lowerbound parameter? Was a remark suggested by Andrey Pavlenko in the pull request.

#4
Updated by Steven Puttemans over 10 years ago

Removed the pull request due to cleaning of git and author not responding and solution not being found yet.

  • Pull request deleted (https://github.com/Itseez/opencv/pull/2759)

#5
Updated by Robert Pollak almost 10 years ago

This is becoming more severe since the legacy function cv.CalcEMD2 is gone in OpenCV 3.0.0-beta.

Btw., let me re-link the pull request with the discussion.

Also available in: Atom PDF