all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] gnu: Add dlib.
@ 2016-08-13 17:15 Marius Bakke
  2016-08-14 17:25 ` Leo Famulari
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-13 17:15 UTC (permalink / raw)
  To: guix-devel

[-- Attachment #1: Type: text/plain, Size: 67 bytes --]

Hi,

This adds the dlib machine learning toolkit.

Cheers,
Marius


[-- Attachment #2: 0001-gnu-Add-dlib.patch --]
[-- Type: text/x-patch, Size: 3834 bytes --]

From 9a24f0bfd8a0375928d7accdc0ef744f4fa304a0 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Sat, 13 Aug 2016 11:26:10 +0100
Subject: [PATCH] gnu: Add dlib.

* gnu/packages/machine-learning.scm (dlib): New variable.
---
 gnu/packages/machine-learning.scm | 55 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 54 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index f96672c..6db0333 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -1,6 +1,7 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015, 2016 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2016 Efraim Flashner <efraim@flashner.co.il>
+;;; Copyright © 2016 Marius Bakke <mbakke@fastmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -32,13 +33,15 @@
   #:use-module (gnu packages compression)
   #:use-module (gnu packages dejagnu)
   #:use-module (gnu packages gcc)
+  #:use-module (gnu packages image)
   #:use-module (gnu packages maths)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages python)
   #:use-module (gnu packages statistics)
   #:use-module (gnu packages swig)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages xorg))
 
 (define-public libsvm
   (package
@@ -467,3 +470,53 @@ geometric models.")
      "This package provides functions for feed-forward neural networks with a
 single hidden layer, and for multinomial log-linear models.")
     (license (list license:gpl2+ license:gpl3+))))
+
+(define-public dlib
+  (package
+    (name "dlib")
+    (version "19.0")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "http://dlib.net/files/dlib-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "0a4xilzhb78ffpcyq2fipri80bmlgrl5ngr7r3jygdr9p7v3z4dn"))
+              (modules '((guix build utils)))
+              (snippet
+               '(begin
+                  ;; Delete ~13MB of bundled dependencies.
+                  (delete-file-recursively "dlib/external")
+                  (delete-file-recursively "docs/dlib/external")))))
+    (build-system cmake-build-system)
+    (arguments
+     `(#:phases
+       (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
+         (modify-phases %standard-phases
+           (add-before 'check 'build-test-suite
+             ;; No test target, so we build and run the unit tests here.
+             (lambda _
+               (mkdir-p test-dir)
+               (chdir test-dir)
+               (zero? (system* "cmake" ".."))
+               (zero? (system* "cmake" "--build" "." "--config" "Release"))))
+           (replace 'check
+             (lambda _ (zero? (system* "./dtest" "--runall"))))
+           (add-after 'check 'ascend-to-build-directory
+             (lambda _ (chdir "../../../../build") #t))))))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (inputs
+     `(("lapack" ,lapack)
+       ("libjpeg" ,libjpeg)
+       ("libpng" ,libpng)
+       ("libx11" ,libx11)
+       ("openblas" ,openblas)
+       ("zlib" ,zlib)))
+    (synopsis "Toolkit for making machine learning and data analysis applications in C++")
+    (description
+     "Dlib is a modern C++ toolkit containing machine learning algorithms and tools.  It
+is used in both industry and academia in a wide range of domains including robotics,
+embedded devices, mobile phones, and large high performance computing environments.")
+    (home-page "http://dlib.net")
+    (license license:boost1.0)))
-- 
2.9.2


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-13 17:15 [PATCH] gnu: Add dlib Marius Bakke
@ 2016-08-14 17:25 ` Leo Famulari
  2016-08-14 19:52   ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-14 17:25 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

On Sat, Aug 13, 2016 at 06:15:59PM +0100, Marius Bakke wrote:
> * gnu/packages/machine-learning.scm (dlib): New variable.

Thanks!

> +    (version "19.0")

The released 19.1 yesterday. Will you send an updated patch using the
latest version?

> +    (arguments
> +     `(#:phases
> +       (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
> +         (modify-phases %standard-phases
> +           (add-before 'check 'build-test-suite
> +             ;; No test target, so we build and run the unit tests here.
> +             (lambda _
> +               (mkdir-p test-dir)
> +               (chdir test-dir)
> +               (zero? (system* "cmake" ".."))
> +               (zero? (system* "cmake" "--build" "." "--config" "Release"))))
> +           (replace 'check
> +             (lambda _ (zero? (system* "./dtest" "--runall"))))

I think we should combine these two phases, and use
with-directory-excursion (defined in guix build utils) instead of
chdir-ing back and forth.

> +           (add-after 'check 'ascend-to-build-directory
> +             (lambda _ (chdir "../../../../build") #t))))))

Then, this phase can be removed.

What do you think?

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-14 17:25 ` Leo Famulari
@ 2016-08-14 19:52   ` Marius Bakke
  2016-08-15  7:43     ` Alex Kost
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-14 19:52 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 1187 bytes --]

Leo Famulari <leo@famulari.name> writes:

>> +    (version "19.0")
>
> The released 19.1 yesterday. Will you send an updated patch using the
> latest version?

Wow, fresh out of the oven :)

>
>> +    (arguments
>> +     `(#:phases
>> +       (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
>> +         (modify-phases %standard-phases
>> +           (add-before 'check 'build-test-suite
>> +             ;; No test target, so we build and run the unit tests here.
>> +             (lambda _
>> +               (mkdir-p test-dir)
>> +               (chdir test-dir)
>> +               (zero? (system* "cmake" ".."))
>> +               (zero? (system* "cmake" "--build" "." "--config" "Release"))))
>> +           (replace 'check
>> +             (lambda _ (zero? (system* "./dtest" "--runall"))))
>
> I think we should combine these two phases, and use
> with-directory-excursion (defined in guix build utils) instead of
> chdir-ing back and forth.
>
>> +           (add-after 'check 'ascend-to-build-directory
>> +             (lambda _ (chdir "../../../../build") #t))))))
>
> Then, this phase can be removed.

That worked great, thanks!

New patch attached.


[-- Attachment #2: 0001-gnu-Add-dlib.patch --]
[-- Type: text/x-patch, Size: 3683 bytes --]

From 5e30eff1cf24b236a78cc5abed870992e84f443f Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Sat, 13 Aug 2016 11:26:10 +0100
Subject: [PATCH] gnu: Add dlib.

* gnu/packages/machine-learning.scm (dlib): New variable.
---
 gnu/packages/machine-learning.scm | 52 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 51 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index f96672c..bd52e8d 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -1,6 +1,7 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015, 2016 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2016 Efraim Flashner <efraim@flashner.co.il>
+;;; Copyright © 2016 Marius Bakke <mbakke@fastmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -32,13 +33,15 @@
   #:use-module (gnu packages compression)
   #:use-module (gnu packages dejagnu)
   #:use-module (gnu packages gcc)
+  #:use-module (gnu packages image)
   #:use-module (gnu packages maths)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages python)
   #:use-module (gnu packages statistics)
   #:use-module (gnu packages swig)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages xorg))
 
 (define-public libsvm
   (package
@@ -467,3 +470,50 @@ geometric models.")
      "This package provides functions for feed-forward neural networks with a
 single hidden layer, and for multinomial log-linear models.")
     (license (list license:gpl2+ license:gpl3+))))
+
+(define-public dlib
+  (package
+    (name "dlib")
+    (version "19.1")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "http://dlib.net/files/dlib-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "0p2pvcdalc6jhb6r99ybvjd9x74sclr0ngswdg9j2xl5pj7knbr4"))
+              (modules '((guix build utils)))
+              (snippet
+               '(begin
+                  ;; Delete ~13MB of bundled dependencies.
+                  (delete-file-recursively "dlib/external")
+                  (delete-file-recursively "docs/dlib/external")))))
+    (build-system cmake-build-system)
+    (arguments
+     `(#:phases
+       (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
+         (modify-phases %standard-phases
+           (replace 'check
+             ;; No test target, so we build and run the unit tests here.
+             (lambda _
+               (mkdir-p test-dir)
+               (with-directory-excursion test-dir
+                 (zero? (system* "cmake" ".."))
+                 (zero? (system* "cmake" "--build" "." "--config" "Release"))
+                 (zero? (system* "./dtest" "--runall")))))))))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (inputs
+     `(("lapack" ,lapack)
+       ("libjpeg" ,libjpeg)
+       ("libpng" ,libpng)
+       ("libx11" ,libx11)
+       ("openblas" ,openblas)
+       ("zlib" ,zlib)))
+    (synopsis "Toolkit for making machine learning and data analysis applications in C++")
+    (description
+     "Dlib is a modern C++ toolkit containing machine learning algorithms and tools.  It
+is used in both industry and academia in a wide range of domains including robotics,
+embedded devices, mobile phones, and large high performance computing environments.")
+    (home-page "http://dlib.net")
+    (license license:boost1.0)))
-- 
2.9.2


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-14 19:52   ` Marius Bakke
@ 2016-08-15  7:43     ` Alex Kost
  2016-08-15 11:51       ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Alex Kost @ 2016-08-15  7:43 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

Marius Bakke (2016-08-14 22:52 +0300) wrote:

Hello and welcome!

> From 5e30eff1cf24b236a78cc5abed870992e84f443f Mon Sep 17 00:00:00 2001
> From: Marius Bakke <mbakke@fastmail.com>
> Date: Sat, 13 Aug 2016 11:26:10 +0100
> Subject: [PATCH] gnu: Add dlib.
[...]
> +(define-public dlib
> +  (package
> +    (name "dlib")
> +    (version "19.1")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                    "http://dlib.net/files/dlib-" version ".tar.bz2"))
> +              (sha256
> +               (base32
> +                "0p2pvcdalc6jhb6r99ybvjd9x74sclr0ngswdg9j2xl5pj7knbr4"))
> +              (modules '((guix build utils)))
> +              (snippet
> +               '(begin
> +                  ;; Delete ~13MB of bundled dependencies.
> +                  (delete-file-recursively "dlib/external")
> +                  (delete-file-recursively "docs/dlib/external")))))
> +    (build-system cmake-build-system)
> +    (arguments
> +     `(#:phases
> +       (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))

I think it's better to move this 'let' inside the phase: ...

> +         (modify-phases %standard-phases
> +           (replace 'check
> +             ;; No test target, so we build and run the unit tests here.
> +             (lambda _

... here.
> +               (mkdir-p test-dir)
> +               (with-directory-excursion test-dir
> +                 (zero? (system* "cmake" ".."))
> +                 (zero? (system* "cmake" "--build" "." "--config" "Release"))
> +                 (zero? (system* "./dtest" "--runall")))))))))

There is no point in the first 2 'zero?' calls as their returning
values will be lost, rather:

  (and (zero? (system* "cmake" ".."))
       (zero? (system* "cmake" "--build" "." "--config" "Release"))
       (zero? (system* "./dtest" "--runall")))

> +    (native-inputs
> +     `(("pkg-config" ,pkg-config)))
> +    (inputs
> +     `(("lapack" ,lapack)
> +       ("libjpeg" ,libjpeg)
> +       ("libpng" ,libpng)
> +       ("libx11" ,libx11)
> +       ("openblas" ,openblas)
> +       ("zlib" ,zlib)))
> +    (synopsis "Toolkit for making machine learning and data analysis applications in C++")
> +    (description
> +     "Dlib is a modern C++ toolkit containing machine learning algorithms and tools.  It
> +is used in both industry and academia in a wide range of domains including robotics,
> +embedded devices, mobile phones, and large high performance computing environments.")

As for me, the above lines (in synopsis and description) are too long, I
would stay inside 72-78 columns.

> +    (home-page "http://dlib.net")
> +    (license license:boost1.0)))

Otherwise, the patch looks good to me, so if Leo or other people don't
have comments, I think it can be committed.

-- 
Alex

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-15  7:43     ` Alex Kost
@ 2016-08-15 11:51       ` Marius Bakke
  2016-08-15 20:15         ` Leo Famulari
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-15 11:51 UTC (permalink / raw)
  To: Alex Kost; +Cc: guix-devel

[-- Attachment #1: Type: text/plain, Size: 2047 bytes --]


>> +    (build-system cmake-build-system)
>> +    (arguments
>> +     `(#:phases
>> +       (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
>
> I think it's better to move this 'let' inside the phase: ...
>
>> +         (modify-phases %standard-phases
>> +           (replace 'check
>> +             ;; No test target, so we build and run the unit tests here.
>> +             (lambda _
>
> ... here.

You're right. I tried that in an early iteration and it failed; I
assumed "version" was not available in that scope. Must have noobed
something else since it works fine now.. :)

>> +               (mkdir-p test-dir)
>> +               (with-directory-excursion test-dir
>> +                 (zero? (system* "cmake" ".."))
>> +                 (zero? (system* "cmake" "--build" "." "--config" "Release"))
>> +                 (zero? (system* "./dtest" "--runall")))))))))
>
> There is no point in the first 2 'zero?' calls as their returning
> values will be lost, rather:
>
>   (and (zero? (system* "cmake" ".."))
>        (zero? (system* "cmake" "--build" "." "--config" "Release"))
>        (zero? (system* "./dtest" "--runall")))

Thanks for pointing that out; fixed.

>> +    (native-inputs
>> +     `(("pkg-config" ,pkg-config)))
>> +    (inputs
>> +     `(("lapack" ,lapack)
>> +       ("libjpeg" ,libjpeg)
>> +       ("libpng" ,libpng)
>> +       ("libx11" ,libx11)
>> +       ("openblas" ,openblas)
>> +       ("zlib" ,zlib)))
>> +    (synopsis "Toolkit for making machine learning and data analysis applications in C++")
>> +    (description
>> +     "Dlib is a modern C++ toolkit containing machine learning algorithms and tools.  It
>> +is used in both industry and academia in a wide range of domains including robotics,
>> +embedded devices, mobile phones, and large high performance computing environments.")
>
> As for me, the above lines (in synopsis and description) are too long, I
> would stay inside 72-78 columns.

Disagreeing with the linter, are we? ;) I've trimmed the lengths a bit.

Thanks!


[-- Attachment #2: 0001-gnu-Add-dlib.patch --]
[-- Type: text/x-patch, Size: 3708 bytes --]

From 8ffcd39f9c8e3cea7ae925341cc19dc3b45941a3 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Sat, 13 Aug 2016 11:26:10 +0100
Subject: [PATCH] gnu: Add dlib.

* gnu/packages/machine-learning.scm (dlib): New variable.
---
 gnu/packages/machine-learning.scm | 54 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 53 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index f96672c..674af10 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -1,6 +1,7 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015, 2016 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2016 Efraim Flashner <efraim@flashner.co.il>
+;;; Copyright © 2016 Marius Bakke <mbakke@fastmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -32,13 +33,15 @@
   #:use-module (gnu packages compression)
   #:use-module (gnu packages dejagnu)
   #:use-module (gnu packages gcc)
+  #:use-module (gnu packages image)
   #:use-module (gnu packages maths)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages python)
   #:use-module (gnu packages statistics)
   #:use-module (gnu packages swig)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages xorg))
 
 (define-public libsvm
   (package
@@ -467,3 +470,52 @@ geometric models.")
      "This package provides functions for feed-forward neural networks with a
 single hidden layer, and for multinomial log-linear models.")
     (license (list license:gpl2+ license:gpl3+))))
+
+(define-public dlib
+  (package
+    (name "dlib")
+    (version "19.1")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "http://dlib.net/files/dlib-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "0p2pvcdalc6jhb6r99ybvjd9x74sclr0ngswdg9j2xl5pj7knbr4"))
+              (modules '((guix build utils)))
+              (snippet
+               '(begin
+                  ;; Delete ~13MB of bundled dependencies.
+                  (delete-file-recursively "dlib/external")
+                  (delete-file-recursively "docs/dlib/external")))))
+    (build-system cmake-build-system)
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (replace 'check
+           (lambda _
+             ;; No test target, so we build and run the unit tests here.
+             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
+               (mkdir-p test-dir)
+               (with-directory-excursion test-dir
+                 (and (zero? (system* "cmake" ".."))
+                      (zero? (system* "cmake" "--build" "." "--config" "Release"))
+                      (zero? (system* "./dtest" "--runall"))))))))))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (inputs
+     `(("lapack" ,lapack)
+       ("libjpeg" ,libjpeg)
+       ("libpng" ,libpng)
+       ("libx11" ,libx11)
+       ("openblas" ,openblas)
+       ("zlib" ,zlib)))
+    (synopsis
+     "Toolkit for making machine learning and data analysis applications in C++")
+    (description
+     "Dlib is a modern C++ toolkit containing machine learning algorithms and
+tools.  It is used in both industry and academia in a wide range of domains
+including robotics, embedded devices, mobile phones, and large high performance
+computing environments.")
+    (home-page "http://dlib.net")
+    (license license:boost1.0)))
-- 
2.9.2


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-15 11:51       ` Marius Bakke
@ 2016-08-15 20:15         ` Leo Famulari
  2016-08-15 20:29           ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-15 20:15 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Mon, Aug 15, 2016 at 12:51:15PM +0100, Marius Bakke wrote:
> Date: Sat, 13 Aug 2016 11:26:10 +0100
> Subject: [PATCH] gnu: Add dlib.
> 
> * gnu/packages/machine-learning.scm (dlib): New variable.

Thanks for the updated patch.

Does it build for you? On my x86_64 machine, it fails consistently. This
is all it prints:

Running test_empirical_kernel_map / phase `check' failed after 2043.7 seconds 

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-15 20:15         ` Leo Famulari
@ 2016-08-15 20:29           ` Marius Bakke
  2016-08-15 22:28             ` Leo Famulari
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-15 20:29 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, Alex Kost

Leo Famulari <leo@famulari.name> writes:

> On Mon, Aug 15, 2016 at 12:51:15PM +0100, Marius Bakke wrote:
>> Date: Sat, 13 Aug 2016 11:26:10 +0100
>> Subject: [PATCH] gnu: Add dlib.
>> 
>> * gnu/packages/machine-learning.scm (dlib): New variable.
>
> Thanks for the updated patch.
>
> Does it build for you? On my x86_64 machine, it fails consistently. This
> is all it prints:
>
> Running test_empirical_kernel_map / phase `check' failed after 2043.7 seconds 

Strange, I've built this dozens of times now with no test failures
(GuixSD on amd64). Are you memory constrained by any chance? Any OOM or
segfaults? I don't think the tests have taken more than ~15 minutes to
run either, once they are built.

Will try it out on some other machines.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-15 20:29           ` Marius Bakke
@ 2016-08-15 22:28             ` Leo Famulari
  2016-08-16  0:15               ` Ben Woodcroft
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-15 22:28 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Mon, Aug 15, 2016 at 09:29:11PM +0100, Marius Bakke wrote:
> Leo Famulari <leo@famulari.name> writes:
> > Running test_empirical_kernel_map / phase `check' failed after 2043.7 seconds 
> 
> Strange, I've built this dozens of times now with no test failures
> (GuixSD on amd64). Are you memory constrained by any chance? Any OOM or
> segfaults? I don't think the tests have taken more than ~15 minutes to
> run either, once they are built.

I don't think I'm particularly resource constrained. I have ~7 GiB RAM
available to build with, plenty of disk space, and the processor is this
thing:

http://ark.intel.com/products/64896/Intel-Core-i5-3320M-Processor-3M-Cache-up-to-3_30-GHz

It appears to exit cleanly (no segfault or OOM).

It fails the same test on this machine as well as a slower x86_64
(Thinkpad x200s). One machine is Guix / Debian, the other is GuixSD.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-15 22:28             ` Leo Famulari
@ 2016-08-16  0:15               ` Ben Woodcroft
  2016-08-16 10:45                 ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Ben Woodcroft @ 2016-08-16  0:15 UTC (permalink / raw)
  To: Leo Famulari, Marius Bakke; +Cc: guix-devel, Alex Kost

On 16/08/16 08:28, Leo Famulari wrote:
> On Mon, Aug 15, 2016 at 09:29:11PM +0100, Marius Bakke wrote:
>> Leo Famulari <leo@famulari.name> writes:
>>> Running test_empirical_kernel_map / phase `check' failed after 2043.7 seconds
>> Strange, I've built this dozens of times now with no test failures
>> (GuixSD on amd64). Are you memory constrained by any chance? Any OOM or
>> segfaults? I don't think the tests have taken more than ~15 minutes to
>> run either, once they are built.
> I don't think I'm particularly resource constrained. I have ~7 GiB RAM
> available to build with, plenty of disk space, and the processor is this
> thing:
>
> http://ark.intel.com/products/64896/Intel-Core-i5-3320M-Processor-3M-Cache-up-to-3_30-GHz
>
> It appears to exit cleanly (no segfault or OOM).
>
> It fails the same test on this machine as well as a slower x86_64
> (Thinkpad x200s). One machine is Guix / Debian, the other is GuixSD.
I built this patch on a x86_64 server with 40 cores and 256G of memory 
twice, here's what I got:

Running test_one_vs_all_trainer \

!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!! TEST FAILED: test_one_vs_all_trainer 
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Failure message from test:

Error occurred at line 141.
Error occurred in file 
/tmp/nix-build-dlib-19.1.drv-0/dlib-19.1/dlib/test/one_vs_all_trainer.cpp.
Failing expression was ans == res.
res:
60  0  0
  0 69  1
  0  0 80




Testing Finished
Total number of individual testing statements executed: 471078543
Number of failed tests: 1
Number of passed tests: 133

phase `check' failed after 2206.2 seconds
builder for `/gnu/store/gqbzlp1g5h038rxswaz2b6y78d57q8sx-dlib-19.1.drv' 
failed with exit code 1
@ build-failed /gnu/store/gqbzlp1g5h038rxswaz2b6y78d57q8sx-dlib-19.1.drv 
- 1 builder for 
`/gnu/store/gqbzlp1g5h038rxswaz2b6y78d57q8sx-dlib-19.1.drv' failed with 
exit code 1
guix build: error: build failed: build of 
`/gnu/store/gqbzlp1g5h038rxswaz2b6y78d57q8sx-dlib-19.1.drv' failed


and in the second run.....


!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!! TEST FAILED: test_one_vs_all_trainer 
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Failure message from test:

Error occurred at line 141.
Error occurred in file 
/tmp/nix-build-dlib-19.1.drv-0/dlib-19.1/dlib/test/one_vs_all_trainer.cpp.
Failing expression was ans == res.
res:
30  0 30
  0 35 35
  0  0 80




Testing Finished
Total number of individual testing statements executed: 470341624
Number of failed tests: 1
Number of passed tests: 133

phase `check' failed after 2183.7 seconds
builder for `/gnu/store/gqbzlp1g5h038rxswaz2b6y78d57q8sx-dlib-19.1.drv' 
failed with exit code 1
@ build-failed /gnu/store/gqbzlp1g5h038rxswaz2b6y78d57q8sx-dlib-19.1.drv 
- 1 builder for 
`/gnu/store/gqbzlp1g5h038rxswaz2b6y78d57q8sx-dlib-19.1.drv' failed with 
exit code 1
guix build: error: build failed: build of 
`/gnu/store/gqbzlp1g5h038rxswaz2b6y78d57q8sx-dlib-19.1.drv' failed

real    37m12.419s
user    0m5.344s
sys    0m0.412s

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-16  0:15               ` Ben Woodcroft
@ 2016-08-16 10:45                 ` Marius Bakke
  2016-08-16 20:47                   ` Leo Famulari
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-16 10:45 UTC (permalink / raw)
  To: Ben Woodcroft, Leo Famulari; +Cc: guix-devel, Alex Kost

[-- Attachment #1: Type: text/plain, Size: 2568 bytes --]

Ben Woodcroft <b.woodcroft@uq.edu.au> writes:

> On 16/08/16 08:28, Leo Famulari wrote:
>> On Mon, Aug 15, 2016 at 09:29:11PM +0100, Marius Bakke wrote:
>>> Leo Famulari <leo@famulari.name> writes:
>>>> Running test_empirical_kernel_map / phase `check' failed after 2043.7 seconds
>>> Strange, I've built this dozens of times now with no test failures
>>> (GuixSD on amd64). Are you memory constrained by any chance? Any OOM or
>>> segfaults? I don't think the tests have taken more than ~15 minutes to
>>> run either, once they are built.
>> I don't think I'm particularly resource constrained. I have ~7 GiB RAM
>> available to build with, plenty of disk space, and the processor is this
>> thing:
>>
>> http://ark.intel.com/products/64896/Intel-Core-i5-3320M-Processor-3M-Cache-up-to-3_30-GHz
>>
>> It appears to exit cleanly (no segfault or OOM).
>>
>> It fails the same test on this machine as well as a slower x86_64
>> (Thinkpad x200s). One machine is Guix / Debian, the other is GuixSD.
> I built this patch on a x86_64 server with 40 cores and 256G of memory 
> twice, here's what I got:
>
> Running test_one_vs_all_trainer \
>
> !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
> !!!!!!!!!!!!!!!!!!!!!!!!!!!!! TEST FAILED: test_one_vs_all_trainer 
> !!!!!!!!!!!!!!!!!!!!!!!!!!!!!
> !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
>
> Failure message from test:
>
> Error occurred at line 141.
> Error occurred in file 
> /tmp/nix-build-dlib-19.1.drv-0/dlib-19.1/dlib/test/one_vs_all_trainer.cpp.
> Failing expression was ans == res.
> res:
> 60  0  0
>   0 69  1
>   0  0 80
>
>
>
>
> Testing Finished
> Total number of individual testing statements executed: 471078543
> Number of failed tests: 1
> Number of passed tests: 133

I can reproduce Leos error on a foreign distro (Ubuntu 16.04, 32GiB RAM)
and see this in dmesg:

[497259.758549] dtest[27102]: segfault at b50 ip 00007ffff68270e0 sp 00007fffffffa978 error 4 in libopenblasp-r0.2.15.so[7ffff5715000+18c6000]

That memory address is 128 TiB. Though it consistently builds fine on a
64GiB GuixSD machine as verified with `guix build --check`.

I initially made this package on a foreign distro without "lapack" in
inputs and have verified that dropping LAPACK makes the tests pass.

I also found some other optional dependencies after digging around the
source, as well as a recommendation to disable/enable asserts:

http://dlib.net/dlib/config.h.html

Ben, Leo: Can you try the following patch and see if that works for you?


[-- Attachment #2: 0001-gnu-Add-dlib.patch --]
[-- Type: text/x-patch, Size: 4408 bytes --]

From 7cc190763e1312536fe3ac9cd91ea3756cf66b80 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Sat, 13 Aug 2016 11:26:10 +0100
Subject: [PATCH] gnu: Add dlib.

* gnu/packages/machine-learning.scm (dlib): New variable.
---
 gnu/packages/machine-learning.scm | 63 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 62 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index f96672c..ebf3307 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -1,6 +1,7 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015, 2016 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2016 Efraim Flashner <efraim@flashner.co.il>
+;;; Copyright © 2016 Marius Bakke <mbakke@fastmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -27,18 +28,21 @@
   #: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)
   #:use-module (gnu packages dejagnu)
   #:use-module (gnu packages gcc)
+  #:use-module (gnu packages image)
   #:use-module (gnu packages maths)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages python)
   #:use-module (gnu packages statistics)
   #:use-module (gnu packages swig)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages xorg))
 
 (define-public libsvm
   (package
@@ -467,3 +471,60 @@ geometric models.")
      "This package provides functions for feed-forward neural networks with a
 single hidden layer, and for multinomial log-linear models.")
     (license (list license:gpl2+ license:gpl3+))))
+
+(define-public dlib
+  (package
+    (name "dlib")
+    (version "19.1")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "http://dlib.net/files/dlib-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "0p2pvcdalc6jhb6r99ybvjd9x74sclr0ngswdg9j2xl5pj7knbr4"))
+              (modules '((guix build utils)))
+              (snippet
+               '(begin
+                  ;; Delete ~13MB of bundled dependencies.
+                  (delete-file-recursively "dlib/external")
+                  (delete-file-recursively "docs/dlib/external")))))
+    (build-system cmake-build-system)
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'disable-asserts
+           (lambda _
+             ;; config.h recommends explicitly enabling or disabling asserts
+             ;; when building as a shared library. By default neither is set.
+             (substitute* "dlib/config.h"
+               (("^//#define DLIB_DISABLE_ASSERTS") "#define DLIB_DISABLE_ASSERTS"))))
+         (replace 'check
+           (lambda _
+             ;; No test target, so we build and run the unit tests here.
+             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
+               (mkdir-p test-dir)
+               (with-directory-excursion test-dir
+                 (and (zero? (system* "cmake" ".."))
+                      (zero? (system* "cmake" "--build" "." "--config" "Release"))
+                      (zero? (system* "./dtest" "--runall"))))))))))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (inputs
+     `(("fftw" ,fftw)
+       ("giflib" ,giflib)
+       ;("lapack" ,lapack) XXX lapack here causes test failures in some setups.
+       ("libjpeg" ,libjpeg)
+       ("libpng" ,libpng)
+       ("libx11" ,libx11)
+       ("openblas" ,openblas)
+       ("zlib" ,zlib)))
+    (synopsis
+     "Toolkit for making machine learning and data analysis applications in C++")
+    (description
+     "Dlib is a modern C++ toolkit containing machine learning algorithms and
+tools.  It is used in both industry and academia in a wide range of domains
+including robotics, embedded devices, mobile phones, and large high performance
+computing environments.")
+    (home-page "http://dlib.net")
+    (license license:boost1.0)))
-- 
2.9.2


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-16 10:45                 ` Marius Bakke
@ 2016-08-16 20:47                   ` Leo Famulari
  2016-08-16 23:31                     ` Ben Woodcroft
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-16 20:47 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Tue, Aug 16, 2016 at 11:45:16AM +0100, Marius Bakke wrote:
> I initially made this package on a foreign distro without "lapack" in
> inputs and have verified that dropping LAPACK makes the tests pass.
> 
> I also found some other optional dependencies after digging around the
> source, as well as a recommendation to disable/enable asserts:
> 
> http://dlib.net/dlib/config.h.html
> 
> Ben, Leo: Can you try the following patch and see if that works for you?

Yes, this patch builds for me.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-16 20:47                   ` Leo Famulari
@ 2016-08-16 23:31                     ` Ben Woodcroft
  2016-08-16 23:45                       ` Leo Famulari
  0 siblings, 1 reply; 34+ messages in thread
From: Ben Woodcroft @ 2016-08-16 23:31 UTC (permalink / raw)
  To: Leo Famulari, Marius Bakke; +Cc: guix-devel, Alex Kost



On 17/08/16 06:47, Leo Famulari wrote:
> On Tue, Aug 16, 2016 at 11:45:16AM +0100, Marius Bakke wrote:
>> I initially made this package on a foreign distro without "lapack" in
>> inputs and have verified that dropping LAPACK makes the tests pass.
>>
>> I also found some other optional dependencies after digging around the
>> source, as well as a recommendation to disable/enable asserts:
>>
>> http://dlib.net/dlib/config.h.html
>>
>> Ben, Leo: Can you try the following patch and see if that works for you?
> Yes, this patch builds for me.
Me too, although it appeared non-deterministic. I'm afraid I haven't 
time to see if this patch is suitable to push just now. Leo?

ben

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-16 23:31                     ` Ben Woodcroft
@ 2016-08-16 23:45                       ` Leo Famulari
  2016-08-17  3:24                         ` Ben Woodcroft
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-16 23:45 UTC (permalink / raw)
  To: Ben Woodcroft; +Cc: guix-devel, Alex Kost

On Wed, Aug 17, 2016 at 09:31:11AM +1000, Ben Woodcroft wrote:
> 
> 
> On 17/08/16 06:47, Leo Famulari wrote:
> > On Tue, Aug 16, 2016 at 11:45:16AM +0100, Marius Bakke wrote:
> > > I initially made this package on a foreign distro without "lapack" in
> > > inputs and have verified that dropping LAPACK makes the tests pass.
> > > 
> > > I also found some other optional dependencies after digging around the
> > > source, as well as a recommendation to disable/enable asserts:
> > > 
> > > http://dlib.net/dlib/config.h.html
> > > 
> > > Ben, Leo: Can you try the following patch and see if that works for you?
> > Yes, this patch builds for me.
> Me too, although it appeared non-deterministic. I'm afraid I haven't time to
> see if this patch is suitable to push just now. Leo?

How did it appear non-deterministic to you?

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-16 23:45                       ` Leo Famulari
@ 2016-08-17  3:24                         ` Ben Woodcroft
  2016-08-17  5:01                           ` Ben Woodcroft
                                             ` (2 more replies)
  0 siblings, 3 replies; 34+ messages in thread
From: Ben Woodcroft @ 2016-08-17  3:24 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, Alex Kost



On 17/08/16 09:45, Leo Famulari wrote:
> On Wed, Aug 17, 2016 at 09:31:11AM +1000, Ben Woodcroft wrote:
>>
>> On 17/08/16 06:47, Leo Famulari wrote:
>>> On Tue, Aug 16, 2016 at 11:45:16AM +0100, Marius Bakke wrote:
>>>> I initially made this package on a foreign distro without "lapack" in
>>>> inputs and have verified that dropping LAPACK makes the tests pass.
>>>>
>>>> I also found some other optional dependencies after digging around the
>>>> source, as well as a recommendation to disable/enable asserts:
>>>>
>>>> http://dlib.net/dlib/config.h.html
>>>>
>>>> Ben, Leo: Can you try the following patch and see if that works for you?
>>> Yes, this patch builds for me.
>> Me too, although it appeared non-deterministic. I'm afraid I haven't time to
>> see if this patch is suitable to push just now. Leo?
> How did it appear non-deterministic to you?
Just based on guix build --check:

guix build: error: build failed: derivation 
`/gnu/store/sxybcxw64q1ajzq6dysal75ffgq6238i-dlib-19.1.drv' may not be 
deterministic: output 
`/gnu/store/il57dcii4pzii11zlixjjxxxw699bg5x-dlib-19.1' differs

I'm actually not sure, why does it say "may not be deterministic"? If it 
builds twice and the second version is different, doesn't that mean it 
is definitely not deterministic by counter-example, unless there has 
been some leakage into the build container?

I also tried building it with #parallel-build? #f and #parallel-tests? 
#f. It worked in the first round but failed the second.

Note also that the 'disable-asserts' phase should end with '#t'.

ben

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-17  3:24                         ` Ben Woodcroft
@ 2016-08-17  5:01                           ` Ben Woodcroft
  2016-08-17 14:48                           ` Marius Bakke
  2016-08-18 20:18                           ` Leo Famulari
  2 siblings, 0 replies; 34+ messages in thread
From: Ben Woodcroft @ 2016-08-17  5:01 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, Alex Kost



On 17/08/16 13:24, Ben Woodcroft wrote:
>
>
> On 17/08/16 09:45, Leo Famulari wrote:
>> On Wed, Aug 17, 2016 at 09:31:11AM +1000, Ben Woodcroft wrote:
>>>
>>> On 17/08/16 06:47, Leo Famulari wrote:
>>>> On Tue, Aug 16, 2016 at 11:45:16AM +0100, Marius Bakke wrote:
>>>>> I initially made this package on a foreign distro without "lapack" in
>>>>> inputs and have verified that dropping LAPACK makes the tests pass.
>>>>>
>>>>> I also found some other optional dependencies after digging around 
>>>>> the
>>>>> source, as well as a recommendation to disable/enable asserts:
>>>>>
>>>>> http://dlib.net/dlib/config.h.html
>>>>>
>>>>> Ben, Leo: Can you try the following patch and see if that works 
>>>>> for you?
>>>> Yes, this patch builds for me.
>>> Me too, although it appeared non-deterministic. I'm afraid I haven't 
>>> time to
>>> see if this patch is suitable to push just now. Leo?
>> How did it appear non-deterministic to you?
> Just based on guix build --check:
>
> guix build: error: build failed: derivation 
> `/gnu/store/sxybcxw64q1ajzq6dysal75ffgq6238i-dlib-19.1.drv' may not be 
> deterministic: output 
> `/gnu/store/il57dcii4pzii11zlixjjxxxw699bg5x-dlib-19.1' differs
>
> I'm actually not sure, why does it say "may not be deterministic"? If 
> it builds twice and the second version is different, doesn't that mean 
> it is definitely not deterministic by counter-example, unless there 
> has been some leakage into the build container?
>
> I also tried building it with #parallel-build? #f and #parallel-tests? 
> #f. It worked in the first round but failed the second.
I wasn't clear here. This was with lapack uncommented out. Subsequently 
I tested with both these flags #f and lapack commented out, then it was 
deterministic with --rounds=2. Let me know if you want any further 
testing done.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-17  3:24                         ` Ben Woodcroft
  2016-08-17  5:01                           ` Ben Woodcroft
@ 2016-08-17 14:48                           ` Marius Bakke
  2016-08-18 20:23                             ` Leo Famulari
  2016-08-18 20:18                           ` Leo Famulari
  2 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-17 14:48 UTC (permalink / raw)
  To: Ben Woodcroft, Leo Famulari; +Cc: guix-devel, Alex Kost

[-- Attachment #1: Type: text/plain, Size: 2050 bytes --]

Ben Woodcroft <b.woodcroft@uq.edu.au> writes:

> On 17/08/16 09:45, Leo Famulari wrote:
>> On Wed, Aug 17, 2016 at 09:31:11AM +1000, Ben Woodcroft wrote:
>>>
>>> On 17/08/16 06:47, Leo Famulari wrote:
>>>> On Tue, Aug 16, 2016 at 11:45:16AM +0100, Marius Bakke wrote:
>>>>> I initially made this package on a foreign distro without "lapack" in
>>>>> inputs and have verified that dropping LAPACK makes the tests pass.
>>>>>
>>>>> I also found some other optional dependencies after digging around the
>>>>> source, as well as a recommendation to disable/enable asserts:
>>>>>
>>>>> http://dlib.net/dlib/config.h.html
>>>>>
>>>>> Ben, Leo: Can you try the following patch and see if that works for you?
>>>> Yes, this patch builds for me.
>>> Me too, although it appeared non-deterministic. I'm afraid I haven't time to
>>> see if this patch is suitable to push just now. Leo?
>> How did it appear non-deterministic to you?
> Just based on guix build --check:
>
> guix build: error: build failed: derivation 
> `/gnu/store/sxybcxw64q1ajzq6dysal75ffgq6238i-dlib-19.1.drv' may not be 
> deterministic: output 
> `/gnu/store/il57dcii4pzii11zlixjjxxxw699bg5x-dlib-19.1' differs
>
> I'm actually not sure, why does it say "may not be deterministic"? If it 
> builds twice and the second version is different, doesn't that mean it 
> is definitely not deterministic by counter-example, unless there has 
> been some leakage into the build container?
>
> I also tried building it with #parallel-build? #f and #parallel-tests? 
> #f. It worked in the first round but failed the second.
>
> Note also that the 'disable-asserts' phase should end with '#t'.

I've attached a patch with a #t in the disable-asserts phase, and also
deleting the 6MB static library.

Since `guix build --rounds=2` passes, is there any reason to delay this
patch? I've built this on GuixSD and a foreign distro and naively
verified that they are the same (the .so and all headers have the same
checksum; some of the recorded cmake input paths are different though).

Thanks,
Marius


[-- Attachment #2: 0001-gnu-Add-dlib.patch --]
[-- Type: text/x-patch, Size: 4627 bytes --]

From 77f74972d095aeb08367e00c9d683137bd7a35f3 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Sat, 13 Aug 2016 11:26:10 +0100
Subject: [PATCH] gnu: Add dlib.

* gnu/packages/machine-learning.scm (dlib): New variable.
---
 gnu/packages/machine-learning.scm | 67 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 66 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index f96672c..4332045 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -1,6 +1,7 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2015, 2016 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2016 Efraim Flashner <efraim@flashner.co.il>
+;;; Copyright © 2016 Marius Bakke <mbakke@fastmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -27,18 +28,21 @@
   #: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)
   #:use-module (gnu packages dejagnu)
   #:use-module (gnu packages gcc)
+  #:use-module (gnu packages image)
   #:use-module (gnu packages maths)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages python)
   #:use-module (gnu packages statistics)
   #:use-module (gnu packages swig)
-  #:use-module (gnu packages xml))
+  #:use-module (gnu packages xml)
+  #:use-module (gnu packages xorg))
 
 (define-public libsvm
   (package
@@ -467,3 +471,64 @@ geometric models.")
      "This package provides functions for feed-forward neural networks with a
 single hidden layer, and for multinomial log-linear models.")
     (license (list license:gpl2+ license:gpl3+))))
+
+(define-public dlib
+  (package
+    (name "dlib")
+    (version "19.1")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "http://dlib.net/files/dlib-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "0p2pvcdalc6jhb6r99ybvjd9x74sclr0ngswdg9j2xl5pj7knbr4"))
+              (modules '((guix build utils)))
+              (snippet
+               '(begin
+                  ;; Delete ~13MB of bundled dependencies.
+                  (delete-file-recursively "dlib/external")
+                  (delete-file-recursively "docs/dlib/external")))))
+    (build-system cmake-build-system)
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'disable-asserts
+           (lambda _
+             ;; config.h recommends explicitly enabling or disabling asserts
+             ;; when building as a shared library. By default neither is set.
+             (substitute* "dlib/config.h"
+               (("^//#define DLIB_DISABLE_ASSERTS") "#define DLIB_DISABLE_ASSERTS"))
+             #t))
+         (replace 'check
+           (lambda _
+             ;; No test target, so we build and run the unit tests here.
+             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
+               (mkdir-p test-dir)
+               (with-directory-excursion test-dir
+                 (and (zero? (system* "cmake" ".."))
+                      (zero? (system* "cmake" "--build" "." "--config" "Release"))
+                      (zero? (system* "./dtest" "--runall")))))))
+         (add-after 'install 'delete-static-library
+           (lambda* (#:key outputs #:allow-other-keys)
+             (delete-file (string-append (assoc-ref outputs "out") "/lib/libdlib.a")))))))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (inputs
+     `(("fftw" ,fftw)
+       ("giflib" ,giflib)
+       ;("lapack" ,lapack) XXX lapack here causes test failures in some setups.
+       ("libjpeg" ,libjpeg)
+       ("libpng" ,libpng)
+       ("libx11" ,libx11)
+       ("openblas" ,openblas)
+       ("zlib" ,zlib)))
+    (synopsis
+     "Toolkit for making machine learning and data analysis applications in C++")
+    (description
+     "Dlib is a modern C++ toolkit containing machine learning algorithms and
+tools.  It is used in both industry and academia in a wide range of domains
+including robotics, embedded devices, mobile phones, and large high performance
+computing environments.")
+    (home-page "http://dlib.net")
+    (license license:boost1.0)))
-- 
2.9.2


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-17  3:24                         ` Ben Woodcroft
  2016-08-17  5:01                           ` Ben Woodcroft
  2016-08-17 14:48                           ` Marius Bakke
@ 2016-08-18 20:18                           ` Leo Famulari
  2 siblings, 0 replies; 34+ messages in thread
From: Leo Famulari @ 2016-08-18 20:18 UTC (permalink / raw)
  To: Ben Woodcroft; +Cc: guix-devel, Alex Kost

On Wed, Aug 17, 2016 at 01:24:46PM +1000, Ben Woodcroft wrote:
> On 17/08/16 09:45, Leo Famulari wrote:
> > How did it appear non-deterministic to you?
> Just based on guix build --check:
> 
> guix build: error: build failed: derivation
> `/gnu/store/sxybcxw64q1ajzq6dysal75ffgq6238i-dlib-19.1.drv' may not be
> deterministic: output
> `/gnu/store/il57dcii4pzii11zlixjjxxxw699bg5x-dlib-19.1' differs
> 
> I'm actually not sure, why does it say "may not be deterministic"? If it
> builds twice and the second version is different, doesn't that mean it is
> definitely not deterministic by counter-example, unless there has been some
> leakage into the build container?

Ah yes, that checks if the built output is bit-for-bit identical. I
thought that you meant there was some non-determinism in the build
system, like a race condition caused by parallel building. Which could
also be true.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-17 14:48                           ` Marius Bakke
@ 2016-08-18 20:23                             ` Leo Famulari
  2016-08-19 10:52                               ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-18 20:23 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Wed, Aug 17, 2016 at 03:48:24PM +0100, Marius Bakke wrote:
> I've attached a patch with a #t in the disable-asserts phase, and also
> deleting the 6MB static library.
> 
> Since `guix build --rounds=2` passes, is there any reason to delay this
> patch? I've built this on GuixSD and a foreign distro and naively
> verified that they are the same (the .so and all headers have the same
> checksum; some of the recorded cmake input paths are different though).

I pushed the patch as 5f0ff6a9e. Hopefully dlib is still useful without
lapack. We should really figure out what the issue is and fix it :)

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-18 20:23                             ` Leo Famulari
@ 2016-08-19 10:52                               ` Marius Bakke
  2016-08-21 20:17                                 ` Leo Famulari
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-19 10:52 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, Alex Kost

Leo Famulari <leo@famulari.name> writes:

> I pushed the patch as 5f0ff6a9e. Hopefully dlib is still useful without
> lapack. We should really figure out what the issue is and fix it :)

I noticed this fails to build on Hydra. What's worse is that the i686,
x86_64 and armhf targets fails at completely different things. armhf and
i686 exits cleanly after failing 2 and 5 tests respectively, while the
x86_64 target seems to get the segfault we saw with lapack in inputs.

What should we do? I'd prefer to keep the package so it can easily be
tested on various architectures, but can understand if it is reverted.
Perhaps we can disable substitutes or tests, to stop bothering Hydra?

I'll try reproducing it this weekend on various qemu configurations.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-19 10:52                               ` Marius Bakke
@ 2016-08-21 20:17                                 ` Leo Famulari
  2016-08-22  2:30                                   ` Ben Woodcroft
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-21 20:17 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Fri, Aug 19, 2016 at 11:52:37AM +0100, Marius Bakke wrote:
> Leo Famulari <leo@famulari.name> writes:
> 
> > I pushed the patch as 5f0ff6a9e. Hopefully dlib is still useful without
> > lapack. We should really figure out what the issue is and fix it :)
> 
> I noticed this fails to build on Hydra. What's worse is that the i686,
> x86_64 and armhf targets fails at completely different things. armhf and
> i686 exits cleanly after failing 2 and 5 tests respectively, while the
> x86_64 target seems to get the segfault we saw with lapack in inputs.
> 
> What should we do? I'd prefer to keep the package so it can easily be
> tested on various architectures, but can understand if it is reverted.
> Perhaps we can disable substitutes or tests, to stop bothering Hydra?
> 
> I'll try reproducing it this weekend on various qemu configurations.

Let us know about the results of your tests. Then we can decide what to
do.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-21 20:17                                 ` Leo Famulari
@ 2016-08-22  2:30                                   ` Ben Woodcroft
  2016-08-22 12:01                                     ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Ben Woodcroft @ 2016-08-22  2:30 UTC (permalink / raw)
  To: Leo Famulari, Marius Bakke; +Cc: guix-devel, Alex Kost



On 21/08/16 16:17, Leo Famulari wrote:
> On Fri, Aug 19, 2016 at 11:52:37AM +0100, Marius Bakke wrote:
>> Leo Famulari <leo@famulari.name> writes:
>>
>>> I pushed the patch as 5f0ff6a9e. Hopefully dlib is still useful without
>>> lapack. We should really figure out what the issue is and fix it :)
>> I noticed this fails to build on Hydra. What's worse is that the i686,
>> x86_64 and armhf targets fails at completely different things. armhf and
>> i686 exits cleanly after failing 2 and 5 tests respectively, while the
>> x86_64 target seems to get the segfault we saw with lapack in inputs.
>>
>> What should we do? I'd prefer to keep the package so it can easily be
>> tested on various architectures, but can understand if it is reverted.
>> Perhaps we can disable substitutes or tests, to stop bothering Hydra?
>>
>> I'll try reproducing it this weekend on various qemu configurations.
> Let us know about the results of your tests. Then we can decide what to
> do.
Can this be fixed simply by using #parallel-build #f ? Doing so makes it 
reproducible for me so arguably we should be doing that anyway, even if 
it takes longer to build.

ben

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-22  2:30                                   ` Ben Woodcroft
@ 2016-08-22 12:01                                     ` Marius Bakke
  2016-08-23 18:33                                       ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-22 12:01 UTC (permalink / raw)
  To: Ben Woodcroft, Leo Famulari; +Cc: guix-devel, Alex Kost

Ben Woodcroft <b.woodcroft@uq.edu.au> writes:

> On 21/08/16 16:17, Leo Famulari wrote:
>> On Fri, Aug 19, 2016 at 11:52:37AM +0100, Marius Bakke wrote:
>>> Leo Famulari <leo@famulari.name> writes:
>>>
>>>> I pushed the patch as 5f0ff6a9e. Hopefully dlib is still useful without
>>>> lapack. We should really figure out what the issue is and fix it :)
>>> I noticed this fails to build on Hydra. What's worse is that the i686,
>>> x86_64 and armhf targets fails at completely different things. armhf and
>>> i686 exits cleanly after failing 2 and 5 tests respectively, while the
>>> x86_64 target seems to get the segfault we saw with lapack in inputs.
>>>
>>> What should we do? I'd prefer to keep the package so it can easily be
>>> tested on various architectures, but can understand if it is reverted.
>>> Perhaps we can disable substitutes or tests, to stop bothering Hydra?
>>>
>>> I'll try reproducing it this weekend on various qemu configurations.
>> Let us know about the results of your tests. Then we can decide what to
>> do.
> Can this be fixed simply by using #parallel-build #f ? Doing so makes it 
> reproducible for me so arguably we should be doing that anyway, even if 
> it takes longer to build.

Any chance you can diff the outputs? If there is a race condition or
similar it may indeed fix some problems, but it would be nice to notify
upstream about it.

I opened an issue on their bug tracker, and the suggestion was to
disable BLAS: https://github.com/davisking/dlib/issues/197

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

-marius

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-22 12:01                                     ` Marius Bakke
@ 2016-08-23 18:33                                       ` Marius Bakke
  2016-08-24 10:26                                         ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-23 18:33 UTC (permalink / raw)
  To: Ben Woodcroft, Leo Famulari; +Cc: guix-devel, Alex Kost


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

Marius

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-23 18:33                                       ` Marius Bakke
@ 2016-08-24 10:26                                         ` Marius Bakke
  2016-08-24 17:26                                           ` Leo Famulari
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-24 10:26 UTC (permalink / raw)
  To: Ben Woodcroft, Leo Famulari; +Cc: guix-devel, Alex Kost

[-- Attachment #1: Type: text/plain, Size: 1872 bytes --]

Marius Bakke <mbakke@fastmail.com> 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..).


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-dlib-Remove-unused-fftw-from-inputs.patch --]
[-- Type: text/x-patch, Size: 1276 bytes --]

From 714f38b31996e014ed0cc56391e379e2241ee26e Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
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


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-24 10:26                                         ` Marius Bakke
@ 2016-08-24 17:26                                           ` Leo Famulari
  2016-08-24 19:08                                             ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-24 17:26 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Wed, Aug 24, 2016 at 11:26:28AM +0100, Marius Bakke wrote:
> 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?

I'd say it depends on whether the OpenBLAS users are building
successfully on core-updates, but unfortunately core-updates is
currently failing early in the bootstrap process [0]. Can you take a
look at `guix refresh -l dlib` and pick some important looking
applications to test with the updated OpenBLAS?

[0]
https://hydra.gnu.org/jobset/gnu/core-updates
http://lists.gnu.org/archive/html/guix-devel/2016-08/msg01390.html

> 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).

Maybe dlib is 64-bit only? If that's the case, we can disable it on
those architectures.

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

It's weird that it rebuilds the application again for the tests. Which
build is actually installed in the case of success? Is it worth an
upstream bug report?

> 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..).

Can you hold this patch locally in your "dlib fixes" queue? It would
trigger a rebuild of dlib on Hydra, but I don't see the point when we
expect the build to fail near the end anyways.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-24 17:26                                           ` Leo Famulari
@ 2016-08-24 19:08                                             ` Marius Bakke
  2016-08-24 22:51                                               ` Ben Woodcroft
  2016-08-30 14:43                                               ` Marius Bakke
  0 siblings, 2 replies; 34+ messages in thread
From: Marius Bakke @ 2016-08-24 19:08 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, Alex Kost

Leo Famulari <leo@famulari.name> writes:

> On Wed, Aug 24, 2016 at 11:26:28AM +0100, Marius Bakke wrote:
>> 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?
>
> I'd say it depends on whether the OpenBLAS users are building
> successfully on core-updates, but unfortunately core-updates is
> currently failing early in the bootstrap process [0]. Can you take a
> look at `guix refresh -l dlib` and pick some important looking
> applications to test with the updated OpenBLAS?

I'm currently building the following openblas dependents: `libreoffice
bamm python-biom-format clipper shogun armadillo julia` and will try to
test BLAS functionality in some of them.

>> 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).
>
> Maybe dlib is 64-bit only? If that's the case, we can disable it on
> those architectures.

According to the developer[0], these targets should be supported.

0: https://github.com/davisking/dlib/issues/197

We could disable tests (at least the failing ones) on these platforms
until this issue is resolved. The mips64el target on Hydra times out
after 3600 seconds on one of the tests, but seems fine up to that point.
Some of these tests are fairly CPU heavy, so the timeout may be too low.

>> 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.
>
> It's weird that it rebuilds the application again for the tests. Which
> build is actually installed in the case of success? Is it worth an
> upstream bug report?

We rely on cmake-build-system for everything but the check phase, which
doesn't have a "proper" check target. I'm no cmake expert, but we can
probably prevent re-build by not including the main application in
test/CmakeLists.txt and instead copy/link in our already-built version:

https://github.com/davisking/dlib/blob/master/dlib/test/CMakeLists.txt#L15

>> 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..).
>
> Can you hold this patch locally in your "dlib fixes" queue? It would
> trigger a rebuild of dlib on Hydra, but I don't see the point when we
> expect the build to fail near the end anyways.

Yep, hanging on to it for now.

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-24 19:08                                             ` Marius Bakke
@ 2016-08-24 22:51                                               ` Ben Woodcroft
  2016-08-30 14:43                                               ` Marius Bakke
  1 sibling, 0 replies; 34+ messages in thread
From: Ben Woodcroft @ 2016-08-24 22:51 UTC (permalink / raw)
  To: Marius Bakke, Leo Famulari; +Cc: guix-devel, Alex Kost



On 24 August 2016 3:08:58 PM GMT-04:00, Marius Bakke <mbakke@fastmail.com> wrote:

>
>>> 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.
>>
>> It's weird that it rebuilds the application again for the tests.
>Which
>> build is actually installed in the case of success? Is it worth an
>> upstream bug report?
>
>We rely on cmake-build-system for everything but the check phase, which
>doesn't have a "proper" check target. I'm no cmake expert, but we can
>probably prevent re-build by not including the main application in
>test/CmakeLists.txt and instead copy/link in our already-built version:
>
>https://github.com/davisking/dlib/blob/master/dlib/test/CMakeLists.txt#L15

The cmake build system has an #:out-of-tree? option, maybe set that to #f?

Ben

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-24 19:08                                             ` Marius Bakke
  2016-08-24 22:51                                               ` Ben Woodcroft
@ 2016-08-30 14:43                                               ` Marius Bakke
  2016-08-31 19:09                                                 ` Leo Famulari
  1 sibling, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-08-30 14:43 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, Alex Kost

[-- Attachment #1: Type: text/plain, Size: 2683 bytes --]

Marius Bakke <mbakke@fastmail.com> writes:

> Leo Famulari <leo@famulari.name> writes:
>
>> On Wed, Aug 24, 2016 at 11:26:28AM +0100, Marius Bakke wrote:
>>> 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?
>>
>> I'd say it depends on whether the OpenBLAS users are building
>> successfully on core-updates, but unfortunately core-updates is
>> currently failing early in the bootstrap process [0]. Can you take a
>> look at `guix refresh -l dlib` and pick some important looking
>> applications to test with the updated OpenBLAS?
>
> I'm currently building the following openblas dependents: `libreoffice
> bamm python-biom-format clipper shogun armadillo julia` and will try to
> test BLAS functionality in some of them.

Shogun failed to build in this run. I don't have time to investigate
further, so picking the OpenBLAS update is not very appealing.

Instead I opted to disable the test that fails with lapack (and without,
on Hydra), since it's one specific openblas operation that is not unique
to dlib. I think it's an acceptable tradeoff, to give users the full
dlib functionality, and have the segfault "sort itself" when
core-updates lands in 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).
>>
>> Maybe dlib is 64-bit only? If that's the case, we can disable it on
>> those architectures.
>
> According to the developer[0], these targets should be supported.
>
> 0: https://github.com/davisking/dlib/issues/197
>
> We could disable tests (at least the failing ones) on these platforms
> until this issue is resolved. The mips64el target on Hydra times out
> after 3600 seconds on one of the tests, but seems fine up to that point.
> Some of these tests are fairly CPU heavy, so the timeout may be too low.

Below is a patch which disables these tests (and the above segfault) for
19.1, rather than backporting the patches from dlib master branch.

One note about the patch: I could not figure out how to pass the list of
tests as arguments to `substitute*`, so currently it calls `substitute*`
for each of them. Any tips to prevent this?

It also no longer builds the main application twice for tests.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-dlib-Remove-unused-fftw-from-inputs.patch --]
[-- Type: text/x-patch, Size: 1280 bytes --]

From fe3b7409d32bbfd5c263d1d8f080b160ac676b84 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Tue, 23 Aug 2016 21:17:14 +0100
Subject: [PATCH 1/2] 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


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: 0002-gnu-dlib-Disable-failing-tests.patch --]
[-- Type: text/x-patch, Size: 2778 bytes --]

From 121895b2d7915b1f1ef0430921418783019393d0 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Sat, 27 Aug 2016 17:23:58 +0100
Subject: [PATCH 2/2] gnu: dlib: Disable failing tests.

* gnu/packages/machine-learning.scm (dlib)[arguments]: Add
  'disable-failing-tests phase and prevent building dlib twice.
  [inputs]: Add lapack.
---
 gnu/packages/machine-learning.scm | 21 ++++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 7669702..35f1514 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -499,14 +499,25 @@ single hidden layer, and for multinomial log-linear models.")
              (substitute* "dlib/config.h"
                (("^//#define DLIB_DISABLE_ASSERTS") "#define DLIB_DISABLE_ASSERTS"))
              #t))
+         (add-after 'disable-asserts 'disable-failing-tests
+           ;; A number of tests are known to fail on 32-bit platforms in 19.1.
+           ;; See https://github.com/davisking/dlib/issues/197 for details.
+           (lambda _
+             (for-each
+              (lambda (test)
+                (substitute* "dlib/test/makefile"
+                  (((string-append "SRC \\+= " test "\\.cpp")) "")) #t)
+              (list "learning_to_track" "max_cost_assignment" ; armhf
+                    "optimization" "matrix2" "mpc" ; i686
+                    "empirical_map" ; may segfault with < openblas-0.2.18
+                    "object_detector")))) ; timeout on mips64el
          (replace 'check
            (lambda _
              ;; No test target, so we build and run the unit tests here.
-             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
-               (mkdir-p test-dir)
+             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test")))
                (with-directory-excursion test-dir
-                 (and (zero? (system* "cmake" ".."))
-                      (zero? (system* "cmake" "--build" "." "--config" "Release"))
+                 (setenv "CXXFLAGS" "-std=gnu++11")
+                 (and (zero? (system* "make" "-j" (number->string (parallel-job-count))))
                       (zero? (system* "./dtest" "--runall")))))))
          (add-after 'install 'delete-static-library
            (lambda* (#:key outputs #:allow-other-keys)
@@ -515,7 +526,7 @@ single hidden layer, and for multinomial log-linear models.")
      `(("pkg-config" ,pkg-config)))
     (inputs
      `(("giflib" ,giflib)
-       ;("lapack" ,lapack) XXX lapack here causes test failures in some setups.
+       ("lapack" ,lapack)
        ("libjpeg" ,libjpeg)
        ("libpng" ,libpng)
        ("libx11" ,libx11)
-- 
2.9.3


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-30 14:43                                               ` Marius Bakke
@ 2016-08-31 19:09                                                 ` Leo Famulari
  2016-09-09 12:15                                                   ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-08-31 19:09 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Tue, Aug 30, 2016 at 03:43:05PM +0100, Marius Bakke wrote:
> Shogun failed to build in this run. I don't have time to investigate
> further, so picking the OpenBLAS update is not very appealing.
> 
> Instead I opted to disable the test that fails with lapack (and without,
> on Hydra), since it's one specific openblas operation that is not unique
> to dlib. I think it's an acceptable tradeoff, to give users the full
> dlib functionality, and have the segfault "sort itself" when
> core-updates lands in master.

Okay, this sounds fine to me.

> Below is a patch which disables these tests (and the above segfault) for
> 19.1, rather than backporting the patches from dlib master branch.
> 
> One note about the patch: I could not figure out how to pass the list of
> tests as arguments to `substitute*`, so currently it calls `substitute*`
> for each of them. Any tips to prevent this?

Not from me — Calling all seasoned Schemers to thread :) If nobody
replies I will say this solution is fine.

Changing the subject, you could disable the tests per-architecture. Look
for uses of current-target-system and current-system for usage examples.
But this is not absolutely required, IMO.

> It also no longer builds the main application twice for tests.

Thank you :)

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-08-31 19:09                                                 ` Leo Famulari
@ 2016-09-09 12:15                                                   ` Marius Bakke
  2016-09-10 12:32                                                     ` Marius Bakke
  0 siblings, 1 reply; 34+ messages in thread
From: Marius Bakke @ 2016-09-09 12:15 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, Alex Kost

[-- Attachment #1: Type: text/plain, Size: 428 bytes --]

Leo Famulari <leo@famulari.name> writes:

> Changing the subject, you could disable the tests per-architecture. Look
> for uses of current-target-system and current-system for usage examples.
> But this is not absolutely required, IMO.

This latest patch series now disables tests on a per-arch basis.

There may be others on MIPS failing after the timeout, but extending the
list should be obvious.

Thanks for your patience!


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: dlib-fixes.patch --]
[-- Type: text/x-patch, Size: 5625 bytes --]

From 331d291cabcb5d17d0284414cae1f2ac5c32736a Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Tue, 23 Aug 2016 21:17:14 +0100
Subject: [PATCH 1/3] 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


From 5c78ac7002cd92a2bfd8efa0f9902288dffe6a89 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Sat, 27 Aug 2016 17:23:58 +0100
Subject: [PATCH 2/3] gnu: dlib: Do not build dlib twice for tests.

* gnu/packages/machine-learning.scm (dlib)[arguments]: Use makefile
  instead of cmake in check phase to prevent full rebuild.
---
 gnu/packages/machine-learning.scm | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 7669702..e94ffdd 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -502,11 +502,10 @@ single hidden layer, and for multinomial log-linear models.")
          (replace 'check
            (lambda _
              ;; No test target, so we build and run the unit tests here.
-             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
-               (mkdir-p test-dir)
+             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test")))
                (with-directory-excursion test-dir
-                 (and (zero? (system* "cmake" ".."))
-                      (zero? (system* "cmake" "--build" "." "--config" "Release"))
+                 (setenv "CXXFLAGS" "-std=gnu++11")
+                 (and (zero? (system* "make" "-j" (number->string (parallel-job-count))))
                       (zero? (system* "./dtest" "--runall")))))))
          (add-after 'install 'delete-static-library
            (lambda* (#:key outputs #:allow-other-keys)
-- 
2.9.3


From dd8e87ecfb1f475bd0f88338f233883d02e649af Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Fri, 9 Sep 2016 01:51:23 +0100
Subject: [PATCH 3/3] gnu: dlib: Disable failing tests.

* gnu/packages/machine-learning.scm (dlib)[arguments]: Add phase to
  disable tests that are known not to work in the current release.
  [inputs]: Add lapack.
---
 gnu/packages/machine-learning.scm | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index e94ffdd..dd260c9 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -499,6 +499,29 @@ single hidden layer, and for multinomial log-linear models.")
              (substitute* "dlib/config.h"
                (("^//#define DLIB_DISABLE_ASSERTS") "#define DLIB_DISABLE_ASSERTS"))
              #t))
+         (add-after 'disable-asserts 'disable-failing-tests
+           (lambda _
+             ;; One test times out on MIPS, so we need to disable it.
+             ;; The rest is known to fail on non-x86_64 platforms in the current release.
+             ;; Some have been fixed in git; this list should be readjusted next update.
+             (let* ((system ,(or (%current-target-system)
+                                 (%current-system)))
+                    (failing-tests (cond
+                                    ((string-prefix? "mips64" system)
+                                     '("object_detector" ; timeout
+                                       "data_io"))
+                                    ((string-prefix? "armhf" system)
+                                     '("learning_to_track" "max_cost_assignment"))
+                                    ((string-prefix? "i686" system)
+                                     '("optimization" "matrix2" "mpc"))
+                                    (else '()))))
+               ;; The following test fails due a bug in openblas < 0.2.18.
+               (append! failing-tests '("empirical_map"))
+               (for-each
+                (lambda (test)
+                  (substitute* "dlib/test/makefile"
+                    (((string-append "SRC \\+= " test "\\.cpp")) "")) #t)
+                failing-tests))))
          (replace 'check
            (lambda _
              ;; No test target, so we build and run the unit tests here.
@@ -514,7 +537,7 @@ single hidden layer, and for multinomial log-linear models.")
      `(("pkg-config" ,pkg-config)))
     (inputs
      `(("giflib" ,giflib)
-       ;("lapack" ,lapack) XXX lapack here causes test failures in some setups.
+       ("lapack" ,lapack)
        ("libjpeg" ,libjpeg)
        ("libpng" ,libpng)
        ("libx11" ,libx11)
-- 
2.9.3


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-09-09 12:15                                                   ` Marius Bakke
@ 2016-09-10 12:32                                                     ` Marius Bakke
  2016-09-10 18:16                                                       ` Leo Famulari
  2016-09-13  5:14                                                       ` Leo Famulari
  0 siblings, 2 replies; 34+ messages in thread
From: Marius Bakke @ 2016-09-10 12:32 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel, Alex Kost

[-- Attachment #1: Type: text/plain, Size: 603 bytes --]

Marius Bakke <mbakke@fastmail.com> writes:

> Leo Famulari <leo@famulari.name> writes:
>
>> Changing the subject, you could disable the tests per-architecture. Look
>> for uses of current-target-system and current-system for usage examples.
>> But this is not absolutely required, IMO.
>
> This latest patch series now disables tests on a per-arch basis.
>
> There may be others on MIPS failing after the timeout, but extending the
> list should be obvious.

Here are the patches again, with the "failing-tests" variable renamed to
"disabled-tests". Let me know if I can provide anything else. Thanks!


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-dlib-Remove-unused-fftw-from-inputs.patch --]
[-- Type: text/x-patch, Size: 1280 bytes --]

From 331d291cabcb5d17d0284414cae1f2ac5c32736a Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Tue, 23 Aug 2016 21:17:14 +0100
Subject: [PATCH 1/3] 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


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: 0002-gnu-dlib-Do-not-build-dlib-twice-for-tests.patch --]
[-- Type: text/x-patch, Size: 1568 bytes --]

From 5c78ac7002cd92a2bfd8efa0f9902288dffe6a89 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Sat, 27 Aug 2016 17:23:58 +0100
Subject: [PATCH 2/3] gnu: dlib: Do not build dlib twice for tests.

* gnu/packages/machine-learning.scm (dlib)[arguments]: Use makefile
  instead of cmake in check phase to prevent full rebuild.
---
 gnu/packages/machine-learning.scm | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 7669702..e94ffdd 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -502,11 +502,10 @@ single hidden layer, and for multinomial log-linear models.")
          (replace 'check
            (lambda _
              ;; No test target, so we build and run the unit tests here.
-             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test/build")))
-               (mkdir-p test-dir)
+             (let ((test-dir (string-append "../dlib-" ,version "/dlib/test")))
                (with-directory-excursion test-dir
-                 (and (zero? (system* "cmake" ".."))
-                      (zero? (system* "cmake" "--build" "." "--config" "Release"))
+                 (setenv "CXXFLAGS" "-std=gnu++11")
+                 (and (zero? (system* "make" "-j" (number->string (parallel-job-count))))
                       (zero? (system* "./dtest" "--runall")))))))
          (add-after 'install 'delete-static-library
            (lambda* (#:key outputs #:allow-other-keys)
-- 
2.9.3


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #4: 0003-gnu-dlib-Disable-failing-tests.patch --]
[-- Type: text/x-patch, Size: 2786 bytes --]

From 268d1085beb3195f38e4d96e0bfee55e456d3093 Mon Sep 17 00:00:00 2001
From: Marius Bakke <mbakke@fastmail.com>
Date: Fri, 9 Sep 2016 01:51:23 +0100
Subject: [PATCH 3/3] gnu: dlib: Disable failing tests.

* gnu/packages/machine-learning.scm (dlib)[arguments]: Add phase to
  disable tests that are known not to work in the current release.
  [inputs]: Add lapack.
---
 gnu/packages/machine-learning.scm | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index e94ffdd..b663b4f 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -499,6 +499,29 @@ single hidden layer, and for multinomial log-linear models.")
              (substitute* "dlib/config.h"
                (("^//#define DLIB_DISABLE_ASSERTS") "#define DLIB_DISABLE_ASSERTS"))
              #t))
+         (add-after 'disable-asserts 'disable-failing-tests
+           (lambda _
+             ;; One test times out on MIPS, so we need to disable it.
+             ;; The rest is known to fail on non-x86_64 platforms in the current release.
+             ;; Some have been fixed in git; this list should be readjusted next update.
+             (let* ((system ,(or (%current-target-system)
+                                 (%current-system)))
+                    (disabled-tests (cond
+                                     ((string-prefix? "mips64" system)
+                                      '("object_detector" ; timeout
+                                        "data_io"))
+                                     ((string-prefix? "armhf" system)
+                                      '("learning_to_track" "max_cost_assignment"))
+                                     ((string-prefix? "i686" system)
+                                      '("optimization" "matrix2" "mpc"))
+                                     (else '()))))
+               ;; The following test fails due a bug in openblas < 0.2.18.
+               (append! disabled-tests '("empirical_map"))
+               (for-each
+                (lambda (test)
+                  (substitute* "dlib/test/makefile"
+                    (((string-append "SRC \\+= " test "\\.cpp")) "")) #t)
+                disabled-tests))))
          (replace 'check
            (lambda _
              ;; No test target, so we build and run the unit tests here.
@@ -514,7 +537,7 @@ single hidden layer, and for multinomial log-linear models.")
      `(("pkg-config" ,pkg-config)))
     (inputs
      `(("giflib" ,giflib)
-       ;("lapack" ,lapack) XXX lapack here causes test failures in some setups.
+       ("lapack" ,lapack)
        ("libjpeg" ,libjpeg)
        ("libpng" ,libpng)
        ("libx11" ,libx11)
-- 
2.9.3


^ permalink raw reply related	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-09-10 12:32                                                     ` Marius Bakke
@ 2016-09-10 18:16                                                       ` Leo Famulari
  2016-09-13  5:14                                                         ` Leo Famulari
  2016-09-13  5:14                                                       ` Leo Famulari
  1 sibling, 1 reply; 34+ messages in thread
From: Leo Famulari @ 2016-09-10 18:16 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Sat, Sep 10, 2016 at 01:32:22PM +0100, Marius Bakke wrote:
> Marius Bakke <mbakke@fastmail.com> writes:
> 
> > Leo Famulari <leo@famulari.name> writes:
> >
> >> Changing the subject, you could disable the tests per-architecture. Look
> >> for uses of current-target-system and current-system for usage examples.
> >> But this is not absolutely required, IMO.
> >
> > This latest patch series now disables tests on a per-arch basis.
> >
> > There may be others on MIPS failing after the timeout, but extending the
> > list should be obvious.
> 
> Here are the patches again, with the "failing-tests" variable renamed to
> "disabled-tests". Let me know if I can provide anything else. Thanks!

Looks reasonable to me. Does anyone else have any comments?

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-09-10 18:16                                                       ` Leo Famulari
@ 2016-09-13  5:14                                                         ` Leo Famulari
  0 siblings, 0 replies; 34+ messages in thread
From: Leo Famulari @ 2016-09-13  5:14 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Sat, Sep 10, 2016 at 02:16:09PM -0400, Leo Famulari wrote:
> On Sat, Sep 10, 2016 at 01:32:22PM +0100, Marius Bakke wrote:
> > Marius Bakke <mbakke@fastmail.com> writes:
> > 
> > > Leo Famulari <leo@famulari.name> writes:
> > >
> > >> Changing the subject, you could disable the tests per-architecture. Look
> > >> for uses of current-target-system and current-system for usage examples.
> > >> But this is not absolutely required, IMO.
> > >
> > > This latest patch series now disables tests on a per-arch basis.
> > >
> > > There may be others on MIPS failing after the timeout, but extending the
> > > list should be obvious.
> > 
> > Here are the patches again, with the "failing-tests" variable renamed to
> > "disabled-tests". Let me know if I can provide anything else. Thanks!
> 
> Looks reasonable to me. Does anyone else have any comments?

Okay, pushed! Thanks Marius!

^ permalink raw reply	[flat|nested] 34+ messages in thread

* Re: [PATCH] gnu: Add dlib.
  2016-09-10 12:32                                                     ` Marius Bakke
  2016-09-10 18:16                                                       ` Leo Famulari
@ 2016-09-13  5:14                                                       ` Leo Famulari
  1 sibling, 0 replies; 34+ messages in thread
From: Leo Famulari @ 2016-09-13  5:14 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel, Alex Kost

On Sat, Sep 10, 2016 at 01:32:22PM +0100, Marius Bakke wrote:
> +         (add-after 'disable-asserts 'disable-failing-tests
> +           (lambda _
> +             ;; One test times out on MIPS, so we need to disable it.
> +             ;; The rest is known to fail on non-x86_64 platforms in the current release.
> +             ;; Some have been fixed in git; this list should be readjusted next update.
> +             (let* ((system ,(or (%current-target-system)
> +                                 (%current-system)))
> +                    (disabled-tests (cond
> +                                     ((string-prefix? "mips64" system)
> +                                      '("object_detector" ; timeout
> +                                        "data_io"))
> +                                     ((string-prefix? "armhf" system)
> +                                      '("learning_to_track" "max_cost_assignment"))
> +                                     ((string-prefix? "i686" system)
> +                                      '("optimization" "matrix2" "mpc"))
> +                                     (else '()))))
> +               ;; The following test fails due a bug in openblas < 0.2.18.

I updated OpenBLAS to 0.2.19 on core-updates.

^ permalink raw reply	[flat|nested] 34+ messages in thread

end of thread, other threads:[~2016-09-13  5:15 UTC | newest]

Thread overview: 34+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-13 17:15 [PATCH] gnu: Add dlib Marius Bakke
2016-08-14 17:25 ` Leo Famulari
2016-08-14 19:52   ` Marius Bakke
2016-08-15  7:43     ` Alex Kost
2016-08-15 11:51       ` Marius Bakke
2016-08-15 20:15         ` Leo Famulari
2016-08-15 20:29           ` Marius Bakke
2016-08-15 22:28             ` Leo Famulari
2016-08-16  0:15               ` Ben Woodcroft
2016-08-16 10:45                 ` Marius Bakke
2016-08-16 20:47                   ` Leo Famulari
2016-08-16 23:31                     ` Ben Woodcroft
2016-08-16 23:45                       ` Leo Famulari
2016-08-17  3:24                         ` Ben Woodcroft
2016-08-17  5:01                           ` Ben Woodcroft
2016-08-17 14:48                           ` Marius Bakke
2016-08-18 20:23                             ` Leo Famulari
2016-08-19 10:52                               ` Marius Bakke
2016-08-21 20:17                                 ` Leo Famulari
2016-08-22  2:30                                   ` Ben Woodcroft
2016-08-22 12:01                                     ` Marius Bakke
2016-08-23 18:33                                       ` Marius Bakke
2016-08-24 10:26                                         ` Marius Bakke
2016-08-24 17:26                                           ` Leo Famulari
2016-08-24 19:08                                             ` Marius Bakke
2016-08-24 22:51                                               ` Ben Woodcroft
2016-08-30 14:43                                               ` Marius Bakke
2016-08-31 19:09                                                 ` Leo Famulari
2016-09-09 12:15                                                   ` Marius Bakke
2016-09-10 12:32                                                     ` Marius Bakke
2016-09-10 18:16                                                       ` Leo Famulari
2016-09-13  5:14                                                         ` Leo Famulari
2016-09-13  5:14                                                       ` Leo Famulari
2016-08-18 20:18                           ` Leo Famulari

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.