Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-07-01 Thread Gianfranco Costamagna
this shouldn't be a big issue, just introduce a python3 package, gets it in new queue and ask a removal of the old python2 cruft package. Until this is only in experimental we can just don't care. G. Il Venerdì 1 Luglio 2016 17:30, Lumin ha scritto: Control: close

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-07-01 Thread Gianfranco Costamagna
we can't as said before. Il Venerdì 1 Luglio 2016 17:12, Lumin ha scritto: Control: reopen -1 Give me five minutes and about 5 minutes to mentors, Can we overwrite the -1 version ... There are too many changes between -1 and -2 ... On 1 July 2016 at 15:06,

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-07-01 Thread Lumin
Control: reopen -1 Give me five minutes and about 5 minutes to mentors, Can we overwrite the -1 version ... There are too many changes between -1 and -2 ... On 1 July 2016 at 15:06, Gianfranco Costamagna wrote: > ETOOLATE > > >Well ... I'm somewhat confused by how

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-07-01 Thread Gianfranco Costamagna
ETOOLATE >Well ... I'm somewhat confused by how old the uploaded package is ... it still >builds python2 ... >I'll comment octave package out and do a quick build, the package lying in NEW >should be overwriten by the coming updated package. please give me a fixed -2, and I'll upload it on

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-07-01 Thread Gianfranco Costamagna
Hi, >Is experimental available for this purpose? it is, I'm uploading shortly, I did a dget changed unstable to experimental in changelog, and if everything is good I'll upload BTW skimage didn't migrate https://packages.qa.debian.org/s/skimage.html I would prefer to avoid overriding of

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-06-30 Thread lumin
Hi guys, Some updates to the caffe package: (can be seen in git repo but no mentors upload) 1. added octave-caffe-cpu package, but there remains some lintian errors to be solved. 2. changed package caffe-cpu into metapackage, move tools to package caffe-tools-cpu. the metapackage

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-06-03 Thread Mo Zhou
Hi, I've updated the caffe package in the git repo, one of the major changes is that `python-caffe-cpu` was changed to `python3-caffe-cpu`. On 2 June 2016 at 07:00, Ghislain Vaillant wrote: > > > The build time testsuite, autopkgtest and piuparts serve different > purposes.

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-06-02 Thread Ghislain Vaillant
On 02/06/16 05:33, lumin wrote: On Thu, 2016-05-26 at 14:32 +0100, Ghislain Vaillant wrote: I don't agree. Regarding the testsuite, I believe most features should be tested at package build time, including the Python stuff. We want to fail early if something goes wrong. To me, the autopkgtest

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-06-01 Thread lumin
On Thu, 2016-05-26 at 14:32 +0100, Ghislain Vaillant wrote: > I don't agree. Regarding the testsuite, I believe most features should > be tested at package build time, including the Python stuff. We want to > fail early if something goes wrong. To me, the autopkgtest testsuite > serves a

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-26 Thread Ghislain Vaillant
On 23/05/16 09:18, Gianfranco Costamagna wrote: Hi Lumin, Ghislain I think a lot of good progress has been made on the package, bravo lumin. An update to this, according to https://people.debian.org/~mpitt/autopkgtest/README.package-tests.html Even if I want to add some testsuite for

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-23 Thread Gianfranco Costamagna
Hi Lumin, Ghislain >An update to this, according to >https://people.debian.org/~mpitt/autopkgtest/README.package-tests.html >Even if I want to add some testsuite for python-caffe, I don't need >to add those runtime deps in control, I can add them in tests/control >instead, by adding "Depends"

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-19 Thread lumin
On Thu, 2016-05-19 at 06:32 +, Gianfranco Costamagna wrote: > Hi Lumin, > >Why should runtime deps be added into build-dep, which are useless > >unless I provide python-caffe-* testsuite. > > > not sure then, it should be fine that way! An update to this, according to

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-19 Thread Gianfranco Costamagna
Hi Lumin, >debhelper will pick them up as python package dependencies: > dh_python2 --requires=python/requirements.txt ok >I don't remember why I put python-skimage there but I remember >that python-protobuf was put there as explicit remind for me, >indicating that protobuf is the blocker of

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-19 Thread lumin
Control: block -1 by 799262 one more functional blocker, python3-opencv opencv is (I guess) frequently used by caffe users, Debian should have python3-opencv if we provide python3-caffe. when is the EOL of python2.x? I forgot it. If it is not 2017, can we first upload python2-caffe-* ? I'll

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-18 Thread lumin
Control: block -1 by 795841 Control: block 788539 by 795841 On Wed, 2016-05-18 at 21:12 +, Gianfranco Costamagna wrote: > Hi Lumin, > > >Thank you James, I've solved this problem. > I don't want to do the final checks until Ghislain gives me his personal ack, > but > I see that the python3

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-18 Thread Gianfranco Costamagna
Hi Lumin, >Thank you James, I've solved this problem. I don't want to do the final checks until Ghislain gives me his personal ack, but I see that the python3 dependencies might be fixed with not-much effort. issues I would like to see fixed or answered: python/requirements.txt <-- please check

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-18 Thread lumin
> Title: 'Caffe: Convolutional Architecture for Fast Feature Embedding' Thank you James, I've solved this problem. Mentors, please check the latest caffe package on mentors: https://mentors.debian.net/package/caffe debomatic result should be the same as that obtained 1 hour ago. I think

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-18 Thread Jakub Wilk
* lumin , 2016-05-18, 12:44: * add debian/upstream/metadata , but lintian says W: caffe source: upstream-metadata-yaml-invalid Is there anything wrong with this file? I have no idea I've just commited fix for Lintian to include validation error in the output, so in

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-18 Thread lumin
debomatic result * build pass, * autopkgtest OK, according to log no error occurs * lintian remains 1 warning about that weird YAML issue * piuparts fails because of DoM problem updated package was uploaded to mentors https://mentors.debian.net/package/caffe

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-18 Thread James Clarke
> On 18 May 2016, at 14:15, James Clarke wrote: > >> On 18 May 2016, at 13:44, lumin wrote: >> * add debian/upstream/metadata , but lintian says >> >>> W: caffe source: upstream-metadata-yaml-invalid >> >> Is there anything wrong with this file? I have

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-18 Thread James Clarke
> On 18 May 2016, at 13:44, lumin wrote: > * add debian/upstream/metadata , but lintian says > >> W: caffe source: upstream-metadata-yaml-invalid > > Is there anything wrong with this file? I have no idea > > ``` > Homepage: http://caffe.berkeleyvision.org/ > Name: Caffe

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-17 Thread Jonathon Love
As said in Debian's python policy I'm aware of it. Let me clarify this point here. one of the build dependencies of python-caffe-* is python*-protobuf, and python3-protobuf is not possible for current protobuf version in Debian. In a word, build dependencies not satisfied for python3-caffe-*.

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-17 Thread lumin
Thank you for this careful and thorough review! On Tue, 2016-05-17 at 14:50 +0100, Ghislain Vaillant wrote: > Dear all, > > On 16/05/16 15:50, Gianfranco Costamagna wrote: > > Hi Lumin > > > > > >> Done. Updated package has been uploaded to mentors: > >> https://mentors.debian.net/package/caffe

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-17 Thread Ghislain Vaillant
Dear all, On 16/05/16 15:50, Gianfranco Costamagna wrote: Hi Lumin Done. Updated package has been uploaded to mentors: https://mentors.debian.net/package/caffe 1) did you try to enable the Debug build too? without CMAKE_BUILD_TYPE=RelWithDebInfo you won't be able to get automatic debug

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-16 Thread Gianfranco Costamagna
Hi Lumin >Done. Updated package has been uploaded to mentors: >https://mentors.debian.net/package/caffe 1) did you try to enable the Debug build too? without CMAKE_BUILD_TYPE=RelWithDebInfo you won't be able to get automatic debug packages I think 2) why the python3 support is disabled? note:

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-12 Thread lumin
On Sat, 2016-05-07 at 10:05 +0100, Ghislain Vaillant wrote: > > s/DEB_CPPFLAGS_MAINT_APPEND/DEB_CXXFLAGS_MAINT_APPEND in your d/rules. > Done. Updated package has been uploaded to mentors: https://mentors.debian.net/package/caffe

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-07 Thread Ghislain Vaillant
On 07/05/16 02:35, lumin wrote: Hi, I've split the caffe-cpu package and the caffe-cuda package, and I'd like to first handle the cpu version, leaving the CUDA version pending at debian/science/caffe-contrib. The updated cpu version has been uploaded to mentors:

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-06 Thread lumin
Hi, I've split the caffe-cpu package and the caffe-cuda package, and I'd like to first handle the cpu version, leaving the CUDA version pending at debian/science/caffe-contrib. The updated cpu version has been uploaded to mentors: https://mentors.debian.net/package/caffe This update involves

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-03 Thread Gianfranco Costamagna
Hi, >set(CFLAGS ...) which should be replaced by set(CFLAGS $(CFLAGS) ...) > >An upstream classic unfortunately. as upstream I did this once, and the side effect was something weird. when you run multiple times cmake .. the cflags gets appended multiple times, so you might end up in a really

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-03 Thread Ghislain Vaillant
On 03/05/16 03:08, lumin wrote: - d/*.install.in: no multi-arch install paths? why? I have no plan to make multiarch support for this package, because that makes no sense. In production environment Caffe is a computational intensive and memory-consuming application, and I believe no user will

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-02 Thread lumin
Hi mentors, I found the way to fix lintianI: no-fortify-functions, and I'll add multi-arch support as suggested by Aron, so please wait for my next upload.

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-02 Thread lumin
Hi mentors, I've fixed the issues you pointed out. New packages are rebuilt locally, and uploaded to mentors. https://mentors.debian.net/package/caffe Debomatic-amd64 is still building this updated package: http://debomatic-amd64.debian.net/distribution#experimental/caffe/1.0.0~rc3-1/buildlog

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-02 Thread Gianfranco Costamagna
control: owner -1 ! control: tags -1 moreinfo Hi Ghislain, thanks for the good review! I remember I looked at this package a while ago, so, if you (lumin) can fix what Ghislain pointed out, I'll be happy to do a new reivew and hopefully sponsor :) cheers, G. Il Lunedì 2 Maggio 2016 15:13,

Re: Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-02 Thread Ghislain Vaillant
On 01/05/16 12:44, lumin wrote: Package: sponsorship-requests Severity: wishlist X-Debbugs-CC: a...@debian.org, deb...@danielstender.com, deb...@onerussian.com, debian-de...@lists.debian.org, debian-science@lists.debian.org Dear mentors, I am looking for a sponsor for my package "caffe"

Bug#823140: RFS: caffe/1.0.0~rc3-1 -- a deep learning framework [ITP]

2016-05-01 Thread lumin
Package: sponsorship-requests Severity: wishlist X-Debbugs-CC: a...@debian.org, deb...@danielstender.com, deb...@onerussian.com, debian-de...@lists.debian.org, debian-science@lists.debian.org Dear mentors, I am looking for a sponsor for my package "caffe" * Package name: caffe