[Pywikipedia-bugs] [Maniphest] [Commented On] T128946: Move out opencv from pywikibot-compat

2017-07-11 Thread Multichill
Multichill added a comment. Declining again. We're not going to do anything hereTASK DETAILhttps://phabricator.wikimedia.org/T128946EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: MultichillCc: Dvorapa, Xqt, Multichill, DaBPunkt, pywikibot-bugs-list, jayvdb,

[Pywikipedia-bugs] [Maniphest] [Commented On] T128946: Move out opencv from pywikibot-compat

2016-03-05 Thread DrTrigon
DrTrigon added a comment. In https://phabricator.wikimedia.org/T128946#2091074, @jayvdb wrote: > This will be much harder to package properly, as it is c++. True, this is part of the reason why I never did it myself. I appreciate your effort! > Now the important question --

[Pywikipedia-bugs] [Maniphest] [Commented On] T128946: Move out opencv from pywikibot-compat

2016-03-05 Thread jayvdb
jayvdb added a comment. @Multichill , the intention is to organise the catimages dependencies, so that it can be ported to core without the old `externals` system that compat used, and that all dependencies are properly maintained packages with unit tests, etc. i.e. all dependencies should

[Pywikipedia-bugs] [Maniphest] [Commented On] T128946: Move out opencv from pywikibot-compat

2016-03-05 Thread Multichill
Multichill added a comment. Why are we wasting time on compat? TASK DETAIL https://phabricator.wikimedia.org/T128946 EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/panel/emailpreferences/ To: Multichill Cc: Multichill, DaBPunkt, pywikibot-bugs-list, jayvdb, DrTrigon,

[Pywikipedia-bugs] [Maniphest] [Commented On] T128946: Move out opencv from pywikibot-compat

2016-03-05 Thread jayvdb
jayvdb added a comment. @AbdealiJK , if you think the same functionality can be achieved with another library, then we do not need to do this task ;-) TASK DETAIL https://phabricator.wikimedia.org/T128946 EMAIL PREFERENCES

[Pywikipedia-bugs] [Maniphest] [Commented On] T128946: Move out opencv from pywikibot-compat

2016-03-05 Thread jayvdb
jayvdb added a comment. This will be much harder to package properly, as it is c++. Now the important question -- which bits of this repo (if any) are needed for `catimages.py`. There are two Python classes, in two different .cpp files. `bagofwords_classification_python.cpp` and

[Pywikipedia-bugs] [Maniphest] [Commented On] T128946: Move out opencv from pywikibot-compat

2016-03-05 Thread AbdealiJK
AbdealiJK added a comment. I was taking a look at this issue, and as discussed am making another issue to have an independent discussion on this. I'm wondering why we even have this package. It seems easier to just use the opencv bindings for python ? I've used the Pascal dataset