all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#68735] [PATCH] gnu: Add python-gguf.
@ 2024-01-26 12:12 David Pflug
  2024-01-26 14:28 ` Ricardo Wurmus
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: David Pflug @ 2024-01-26 12:12 UTC (permalink / raw)
  To: 68735; +Cc: David Pflug

* gnu/packages/machine-learning.scm (python-gguf): New variable.

Needed for llama-cpp update (In #68455).

Change-Id: Icf6138486cccc5b4c3141424baccf17c4edb3449
---
 gnu/packages/machine-learning.scm | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 3a12eb8c2f..0e88f7265b 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -22,6 +22,7 @@
 ;;; Copyright © 2023 Navid Afkhami <navid.afkhami@mdc-berlin.de>
 ;;; Copyright © 2023 Zheng Junjie <873216071@qq.com>
 ;;; Copyright © 2023 Troy Figiel <troy@troyfigiel.com>
+;;; Copyright © 2024 David Pflug <david@pflug.io>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -5277,3 +5278,25 @@ (define-public oneapi-dnnl
      "OneAPI Deep Neural Network Library (oneDNN) is a cross-platform
 performance library of basic building blocks for deep learning applications.")
     (license license:asl2.0)))
+
+(define-public python-gguf
+  (package
+    (name "python-gguf")
+    (version "0.6.0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "gguf" version))
+       (sha256
+        (base32 "0rbyc2h3kpqnrvbyjvv8a69l577jv55a31l12jnw21m1lamjxqmj"))))
+    (build-system pyproject-build-system)
+    (arguments
+      `(#:phases
+        (modify-phases %standard-phases
+                       (delete 'check))))
+    (inputs (list poetry python-pytest))
+    (propagated-inputs (list python-numpy))
+    (home-page "https://ggml.ai")
+    (synopsis "Read and write ML models in GGUF for GGML")
+    (description "Read and write ML models in GGUF for GGML")
+    (license license:expat)))

base-commit: cdf1d7dded027019f0ebbd5d6f0147b13dfdd28d
-- 
2.41.0





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

* [bug#68735] [PATCH] gnu: Add python-gguf.
  2024-01-26 12:12 [bug#68735] [PATCH] gnu: Add python-gguf David Pflug
@ 2024-01-26 14:28 ` Ricardo Wurmus
  2024-01-26 16:12   ` David Pflug
  2024-01-26 16:27 ` [bug#68735] [PATCH v2] " David Pflug
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 6+ messages in thread
From: Ricardo Wurmus @ 2024-01-26 14:28 UTC (permalink / raw)
  To: 68735; +Cc: david

Hi David,

thanks for the patch.

Why do you delete the check phase?  When tests are to be disabled please
use “#:tests? #false”.  If no tests are included in the Pypi tarball,
please fetch the code from the upstream repository instead.

Please use a plain list for the arguments field.

Please also use complete sentences in the description field.

-- 
Ricardo




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

* [bug#68735] [PATCH] gnu: Add python-gguf.
  2024-01-26 14:28 ` Ricardo Wurmus
@ 2024-01-26 16:12   ` David Pflug
  0 siblings, 0 replies; 6+ messages in thread
From: David Pflug @ 2024-01-26 16:12 UTC (permalink / raw)
  To: Ricardo Wurmus, 68735

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

Hi Ricardo,

Thanks for the feedback. I'd been trying to skip tests because upstream only provides a stub: https://github.com/ggerganov/llama.cpp/blob/master/gguf-py/tests/test_gguf.py

I'll prep a v2 with the requested changes.

Warm Regards,

David

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 854 bytes --]

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

* [bug#68735] [PATCH v2] gnu: Add python-gguf.
  2024-01-26 12:12 [bug#68735] [PATCH] gnu: Add python-gguf David Pflug
  2024-01-26 14:28 ` Ricardo Wurmus
@ 2024-01-26 16:27 ` David Pflug
  2024-01-27  3:50 ` [bug#68735] [PATCH v3] " David Pflug
  2024-02-11  8:27 ` bug#68735: Close Andreas Enge
  3 siblings, 0 replies; 6+ messages in thread
From: David Pflug @ 2024-01-26 16:27 UTC (permalink / raw)
  To: 68735; +Cc: David Pflug

* gnu/packages/machine-learning.scm (python-gguf): New variable.

Change-Id: Icf6138486cccc5b4c3141424baccf17c4edb3449
---
 gnu/packages/machine-learning.scm | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 3a12eb8c2f..0e88f7265b 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -22,6 +22,7 @@
 ;;; Copyright © 2023 Navid Afkhami <navid.afkhami@mdc-berlin.de>
 ;;; Copyright © 2023 Zheng Junjie <873216071@qq.com>
 ;;; Copyright © 2023 Troy Figiel <troy@troyfigiel.com>
+;;; Copyright © 2024 David Pflug <david@pflug.io>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -5277,3 +5278,25 @@ (define-public oneapi-dnnl
      "OneAPI Deep Neural Network Library (oneDNN) is a cross-platform
 performance library of basic building blocks for deep learning applications.")
     (license license:asl2.0)))
+
+(define-public python-gguf
+  (package
+    (name "python-gguf")
+    (version "0.6.0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "gguf" version))
+       (sha256
+        (base32 "0rbyc2h3kpqnrvbyjvv8a69l577jv55a31l12jnw21m1lamjxqmj"))))
+    (build-system pyproject-build-system)
+    (arguments
+      `(#:phases
+        (modify-phases %standard-phases
+                       (delete 'check))))
+    (inputs (list poetry python-pytest))
+    (propagated-inputs (list python-numpy))
+    (home-page "https://ggml.ai")
+    (synopsis "Read and write ML models in GGUF for GGML")
+    (description "Read and write ML models in GGUF for GGML")
+    (license license:expat)))

base-commit: cdf1d7dded027019f0ebbd5d6f0147b13dfdd28d
-- 
2.41.0





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

* [bug#68735] [PATCH v3] gnu: Add python-gguf.
  2024-01-26 12:12 [bug#68735] [PATCH] gnu: Add python-gguf David Pflug
  2024-01-26 14:28 ` Ricardo Wurmus
  2024-01-26 16:27 ` [bug#68735] [PATCH v2] " David Pflug
@ 2024-01-27  3:50 ` David Pflug
  2024-02-11  8:27 ` bug#68735: Close Andreas Enge
  3 siblings, 0 replies; 6+ messages in thread
From: David Pflug @ 2024-01-27  3:50 UTC (permalink / raw)
  To: 68735; +Cc: David Pflug

* gnu/packages/machine-learning.scm (python-gguf): New variable.

Change-Id: Icf6138486cccc5b4c3141424baccf17c4edb3449
---
 gnu/packages/machine-learning.scm | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/machine-learning.scm b/gnu/packages/machine-learning.scm
index 3a12eb8c2f..8beedb24fa 100644
--- a/gnu/packages/machine-learning.scm
+++ b/gnu/packages/machine-learning.scm
@@ -22,6 +22,7 @@
 ;;; Copyright © 2023 Navid Afkhami <navid.afkhami@mdc-berlin.de>
 ;;; Copyright © 2023 Zheng Junjie <873216071@qq.com>
 ;;; Copyright © 2023 Troy Figiel <troy@troyfigiel.com>
+;;; Copyright © 2024 David Pflug <david@pflug.io>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -5277,3 +5278,23 @@ (define-public oneapi-dnnl
      "OneAPI Deep Neural Network Library (oneDNN) is a cross-platform
 performance library of basic building blocks for deep learning applications.")
     (license license:asl2.0)))
+
+(define-public python-gguf
+  (package
+    (name "python-gguf")
+    (version "0.6.0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "gguf" version))
+       (sha256
+        (base32 "0rbyc2h3kpqnrvbyjvv8a69l577jv55a31l12jnw21m1lamjxqmj"))))
+    (build-system pyproject-build-system)
+    (arguments
+      (list #:tests? #false))
+    (inputs (list poetry python-pytest))
+    (propagated-inputs (list python-numpy))
+    (home-page "https://ggml.ai")
+    (synopsis "Read and write ML models in GGUF for GGML")
+    (description "A Python library for reading and writing GGUF & GGML format ML models.")
+    (license license:expat)))

base-commit: cdf1d7dded027019f0ebbd5d6f0147b13dfdd28d
-- 
2.41.0





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

* bug#68735: Close
  2024-01-26 12:12 [bug#68735] [PATCH] gnu: Add python-gguf David Pflug
                   ` (2 preceding siblings ...)
  2024-01-27  3:50 ` [bug#68735] [PATCH v3] " David Pflug
@ 2024-02-11  8:27 ` Andreas Enge
  3 siblings, 0 replies; 6+ messages in thread
From: Andreas Enge @ 2024-02-11  8:27 UTC (permalink / raw)
  To: 68735-done

Pushed as commit b6439fc5f800b5d22d48192bc347be794f050329 after adding the
first line of the commit message and a comment to explain why tests are
disabled, and turning the synopsis into a complete sentence.

QA suggests to put python-pytest into native instead of regular inputs;
but "guix gc --references" shows that the package retains references
to python-pytest and to poetry, so I kept them where they are.

Ricardo, please feel free to correct me.

Thanks for the patch, David!

Andreas





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

end of thread, other threads:[~2024-02-11  8:27 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-26 12:12 [bug#68735] [PATCH] gnu: Add python-gguf David Pflug
2024-01-26 14:28 ` Ricardo Wurmus
2024-01-26 16:12   ` David Pflug
2024-01-26 16:27 ` [bug#68735] [PATCH v2] " David Pflug
2024-01-27  3:50 ` [bug#68735] [PATCH v3] " David Pflug
2024-02-11  8:27 ` bug#68735: Close Andreas Enge

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.