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,
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 --
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
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,
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
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
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