From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marius Bakke Subject: Re: [PATCH] gnu: Add dlib. Date: Wed, 24 Aug 2016 11:26:28 +0100 Message-ID: <87eg5e4g4r.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> References: <20160815222840.GA10735@jasmine> <4752bc68-5466-6c26-a7b4-e53aec400ff5@uq.edu.au> <8760r10z7n.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <20160816204736.GA25753@jasmine> <993034f9-ceae-525f-01b9-0b8af7a5aafe@uq.edu.au> <20160816234507.GA24224@jasmine> <87h9ajzc1z.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <20160818202353.GB2393@jasmine> <8737m1yqru.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <20160821201757.GA21038@jasmine> <6004b842-2434-9cdc-7a31-04231eac5941@uq.edu.au> <87k2f957wt.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> <8760qr5o8n.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:44082) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bcVOS-0001je-Ov for guix-devel@gnu.org; Wed, 24 Aug 2016 06:26:51 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bcVOO-0005Nh-HO for guix-devel@gnu.org; Wed, 24 Aug 2016 06:26:47 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:41138) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bcVOM-0005LV-26 for guix-devel@gnu.org; Wed, 24 Aug 2016 06:26:44 -0400 In-Reply-To: <8760qr5o8n.fsf@ike.i-did-not-set--mail-host-address--so-tickle-me> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Ben Woodcroft , Leo Famulari Cc: guix-devel , Alex Kost --=-=-= Content-Type: text/plain Marius Bakke writes: >> Without OpenBLAS dlib will use an internal BLAS implementation. I'm >> fairly certain that will at least fix the crash on x86_64, which was >> a segfault in libopenblasp-r0.2.15.so when we had LAPACK in inputs, but >> seems to consistently trigger on Hydra regardless. >> >> I got busy this weekend, but will try to reproduce the i686 errors this >> week; and also check if the newer openblas in core-updates solves the >> x86_64 segfault. >> >> Stay tuned... > > Short update: I can successfully reproduce the i686 test failures simply > by `guix build --system=i686-linux` on x86_64. Removing OpenBLAS from > inputs had no effect. Now to figure out what's going on.. There are a couple of things going on in this thread: 1. Segfault on x86_64. This seems to have been resolved simply by updating OpenBLAS. At least, I'm no longer able to reproduce it even with LAPACK in inputs. So, that should fix the Hydra x86_64 build. Can the OpenBLAS update be cherry-picked to master? 2. i686 test failures. Updating OpenBLAS fixed 1/5 errors. The remaining four are reproducible on 32-bit Ubuntu, so they do not seem Guix related. Upstream has been notified. 3. ARM failures. I don't have ARM hardware to test on, but I'm guessing it's similar to i686 (i.e. not directly Guix related). Adding "#:parallel-build? #f" had no effect on tests, indeed the check phase does not seem to use the previously built dlib; it builds it again without parallel-build. I will try reproducing the non-reproducibility on some higher end hardware, hopefully this week. I've also found that FFTW is no longer used, apparently due to thread safety issues. So I'd appreciate if the following patch can be added. Apologies for not catching the missing reference earlier, I will be more careful in the future (fftw was added in the last minute..). --=-=-= Content-Type: text/x-patch Content-Disposition: inline; filename=0001-gnu-dlib-Remove-unused-fftw-from-inputs.patch >From 714f38b31996e014ed0cc56391e379e2241ee26e Mon Sep 17 00:00:00 2001 From: Marius Bakke Date: Tue, 23 Aug 2016 21:17:14 +0100 Subject: [PATCH] gnu: dlib: Remove unused fftw from inputs. * gnu/packages/machine-learning.scm (dlib)[inputs]: Remove fftw. (define-module): Don't include algebra.scm. --- gnu/packages/machine-learning.scm | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm index 4332045..7669702 100644 --- a/gnu/packages/machine-learning.scm +++ b/gnu/packages/machine-learning.scm @@ -28,7 +28,6 @@ #:use-module (guix build-system gnu) #:use-module (guix build-system r) #:use-module (gnu packages) - #:use-module (gnu packages algebra) #:use-module (gnu packages autotools) #:use-module (gnu packages boost) #:use-module (gnu packages compression) @@ -515,8 +514,7 @@ single hidden layer, and for multinomial log-linear models.") (native-inputs `(("pkg-config" ,pkg-config))) (inputs - `(("fftw" ,fftw) - ("giflib" ,giflib) + `(("giflib" ,giflib) ;("lapack" ,lapack) XXX lapack here causes test failures in some setups. ("libjpeg" ,libjpeg) ("libpng" ,libpng) -- 2.9.3 --=-=-=--