[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#30266: Add FANN

From: Ben Woodcroft
Subject: bug#30266: Add FANN
Date: Fri, 2 Feb 2018 23:12:07 +1000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0

Thanks Mark. I made a few small changes and pushed the patch as a9b34762391e6f3b066aef6fc1ebc614ec88db86.

On 02/02/18 01:32, Mark Meyer wrote:
Thanks for the good feedback. Please consider the attached patch.

Best, Mark

On Sun, Jan 28, 2018, at 03:13, Ben Woodcroft wrote:

Hi Mark,

Thanks for the patch.

+(define-module (gnu packages ai)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix utils)
+  #:use-module (guix download)
+  #:use-module (guix packages)
+  #:use-module (guix build-system cmake)
+  #:use-module (guix git-download)
+  #:use-module (gnu packages))
I think machine-learning.scm would be a good home for this package, so a new file wouldn't be needed.

+(define-public libfann
+  (package
+    (name "libfann")
+    (version "d71d54788b")

The last release is 129 commits behind and it has been some time, so I think you are right - we should package from a git commit.

Usually we package from git using this style though (i.e. reason for not using the release, use of 'file-name', and a version that includes the last release followed by '-1.<commit>').

  ;; There are no recent releases so we package from git.
  (let ((commit "da121155a977197cab9fbb15953ca1b40b11eb87"))
      (name "newick-utils")
      (version (string-append "1.6-1." (string-take commit 8)))
      (source (origin
                (method git-fetch)
                (uri (git-reference
                      (url "")
                      (commit commit)))
                (file-name (string-append name "-" version "-checkout"))

+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "")
+                    (commit version)))
+              (sha256
+               (base32
+                "0ibwpfrjs6q2lijs8slxjgzb2llcl6rk3v2ski4r6215g5jjhg3x"))))
+    (build-system cmake-build-system)
+    (arguments
+     `(#:tests? #f))
I see from the website:

Once you have installed the library you can test it out by going to the examples directory and type make runtest, which will compile the examples and run a few of them to test that everything is working.
Is that possible? Ideally, since there are tests it would be good to run them.

+    (home-page "")
+    (synopsis "Fast Artificial Neural Network")
+    (description
+     "FANN is a free open source neural network library, which implements
+multilayer artificial neural networks in C with support for both fully
+connected and sparsely connected networks.")
+    (license license:lgpl2.1)))
Otherwise LGTM. Can you send an updated patch please?

Thanks, ben

Mark Meyer

reply via email to

[Prev in Thread] Current Thread [Next in Thread]