bug#72747: tensorflow and tensorflow lite is broken

bug#72747: tensorflow and tensorflow lite is broken

[Enrico Schwass via Bug reports for GNU Guix]

Hello

It seems, that the current version of tensorflow-lite 2.13.1 does not build because of the flatbuffers version it depends on. Before guix updated its flatbuffers version it was doing well. 

I don‘t know if package transformations help here. Never got it right

Bye
Enno

bug#72747: [PATCH] gnu: tensorflow-lite: Fix build.

[Andy Tai]

* gnu/packages/machine-learning.scm (tensorflow-lite): Fix build.
[inputs]: Change flatbuffers-next to flatbuffers-23.1.

* gnu/packages/serialization.scm (flatbuffers-23.1): New variable.

Change-Id: I257c3563cb91cbf8e0f0874367aafced248ab4dc
---
 gnu/packages/machine-learning.scm |  2 +-
 gnu/packages/serialization.scm    | 15 +++++++++++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 4c5b713cbf..fd790e4455 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -3487,7 +3487,7 @@ (define-public tensorflow-lite
        ("cpuinfo" ,cpuinfo)
        ("eigen" ,eigen)
        ("fp16" ,fp16)
-       ("flatbuffers" ,flatbuffers-next)
+       ("flatbuffers" ,flatbuffers-23.1)
        ("gemmlowp" ,gemmlowp)
        ("mesa-headers" ,mesa-headers)
        ("neon2sse" ,neon2sse)
diff --git a/gnu/packages/serialization.scm b/gnu/packages/serialization.scm
index 6aaf416467..c33b251b13 100644
--- a/gnu/packages/serialization.scm
+++ b/gnu/packages/serialization.scm
@@ -899,6 +899,21 @@ (define-public flatbuffers
 game development and other performance-critical applications.")
     (license license:asl2.0)))
 
+(define-public flatbuffers-23.1
+    ;; needed explicitly by tensorflow-lite 2.13.1
+  (package
+    (inherit flatbuffers)
+    (version "23.1.21")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/google/flatbuffers")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name "flatbuffers" version))
+              (sha256
+               (base32
+                "1z3a6l8g2y53i5xzraswfs2i0i3kk52zv7nzc2q3fgisbyiri3pz"))))))
+
 (define-public flatbuffers-next
   (package
     (inherit flatbuffers)

base-commit: 53396a22afc04536ddf75d8f82ad2eafa5082725
-- 
2.34.1

bug#72747: [PATCH v2] gnu: tensorflow-lite: Fix build.

[Andy Tai]

* gnu/packages/machine-learning.scm (tensorflow-lite): Fix build.
[inputs]: Change flatbuffers-next to flatbuffers-23.1.

* gnu/packages/serialization.scm (flatbuffers-23.1): New variable.

Change-Id: I257c3563cb91cbf8e0f0874367aafced248ab4dc
---
 gnu/packages/machine-learning.scm |  2 +-
 gnu/packages/serialization.scm    | 19 +++++++++++++++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 4c5b713cbf..fd790e4455 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -3487,7 +3487,7 @@ (define-public tensorflow-lite
        ("cpuinfo" ,cpuinfo)
        ("eigen" ,eigen)
        ("fp16" ,fp16)
-       ("flatbuffers" ,flatbuffers-next)
+       ("flatbuffers" ,flatbuffers-23.1)
        ("gemmlowp" ,gemmlowp)
        ("mesa-headers" ,mesa-headers)
        ("neon2sse" ,neon2sse)
diff --git a/gnu/packages/serialization.scm b/gnu/packages/serialization.scm
index 6aaf416467..31d3ed9feb 100644
--- a/gnu/packages/serialization.scm
+++ b/gnu/packages/serialization.scm
@@ -899,6 +899,25 @@ (define-public flatbuffers
 game development and other performance-critical applications.")
     (license license:asl2.0)))
 
+(define-public flatbuffers-23.1
+    ;; needed explicitly by tensorflow-lite 2.13.1
+  (package
+    (inherit flatbuffers)
+    (version "23.1.21")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/google/flatbuffers")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name "flatbuffers" version))
+              (sha256
+               (base32
+                "1z3a6l8g2y53i5xzraswfs2i0i3kk52zv7nzc2q3fgisbyiri3pz"))))
+    (arguments
+     (substitute-keyword-arguments (package-arguments flatbuffers)
+       ((#:configure-flags flags #~'())
+        #~(append #$flags '("-DCMAKE_POSITION_INDEPENDENT_CODE=ON")))))))
+
 (define-public flatbuffers-next
   (package
     (inherit flatbuffers)

base-commit: 53396a22afc04536ddf75d8f82ad2eafa5082725
-- 
2.34.1

[bug#72747] [PATCH v3] gnu: tensorflow-lite: Fix build.

[Andy Tai]

* gnu/packages/machine-learning.scm (tensorflow-lite): Fix build.
[inputs]: Change flatbuffers-next to flatbuffers-23.1.

* gnu/packages/serialization.scm (flatbuffers-23.1): New variable.

Change-Id: I257c3563cb91cbf8e0f0874367aafced248ab4dc
---
 gnu/packages/machine-learning.scm |  2 +-
 gnu/packages/serialization.scm    | 19 +++++++++++++++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 62b9ac00e9..20c49a6422 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -3487,7 +3487,7 @@ (define-public tensorflow-lite
        ("cpuinfo" ,cpuinfo)
        ("eigen" ,eigen)
        ("fp16" ,fp16)
-       ("flatbuffers" ,flatbuffers-next)
+       ("flatbuffers" ,flatbuffers-23.1)
        ("gemmlowp" ,gemmlowp)
        ("mesa-headers" ,mesa-headers)
        ("neon2sse" ,neon2sse)
diff --git a/gnu/packages/serialization.scm b/gnu/packages/serialization.scm
index 6aaf416467..31d3ed9feb 100644
--- a/gnu/packages/serialization.scm
+++ b/gnu/packages/serialization.scm
@@ -899,6 +899,25 @@ (define-public flatbuffers
 game development and other performance-critical applications.")
     (license license:asl2.0)))
 
+(define-public flatbuffers-23.1
+    ;; needed explicitly by tensorflow-lite 2.13.1
+  (package
+    (inherit flatbuffers)
+    (version "23.1.21")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/google/flatbuffers")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name "flatbuffers" version))
+              (sha256
+               (base32
+                "1z3a6l8g2y53i5xzraswfs2i0i3kk52zv7nzc2q3fgisbyiri3pz"))))
+    (arguments
+     (substitute-keyword-arguments (package-arguments flatbuffers)
+       ((#:configure-flags flags #~'())
+        #~(append #$flags '("-DCMAKE_POSITION_INDEPENDENT_CODE=ON")))))))
+
 (define-public flatbuffers-next
   (package
     (inherit flatbuffers)

base-commit: 61a7930cb03f5eb9e8003bade21d61262c3db8df
-- 
2.34.1

bug#72747: tensorflow and tensorflow lite is broken

[Ludovic Courtès]

Andy Tai <atai@atai.org> skribis:

> * gnu/packages/machine-learning.scm (tensorflow-lite): Fix build.
> [inputs]: Change flatbuffers-next to flatbuffers-23.1.
>
> * gnu/packages/serialization.scm (flatbuffers-23.1): New variable.
>
> Change-Id: I257c3563cb91cbf8e0f0874367aafced248ab4dc

Applied, thanks!

[bug#72747] tensorflow and tensorflow lite is broken

[Enrico Schwass via Guix-patches via]

Hello

Thank you Andy and Ludovic for creating the patch and for applying it. 

The same patch would work for the full tensorflow package, provided by the guix-science channel. *nudge nudge*. 

I am not sure if there is an ML team for Guix, but if not - regarding the hotness of the topic - it would be nice to have one. I can provide simpler packages and test on M1, x86 and if I finally set it up on honeycomb. 

Thank you again
Enno