all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#49469] [PATCH] gnu: Add python-pysox.
@ 2021-07-08 18:45 jgart via Guix-patches via
  2021-07-08 19:35 ` jgart via Guix-patches via
                   ` (5 more replies)
  0 siblings, 6 replies; 12+ messages in thread
From: jgart via Guix-patches via @ 2021-07-08 18:45 UTC (permalink / raw)
  To: 49469

Hi Guix!

Here is python-pysox.

Tests for this package depend on https://issues.guix.gnu.org/47181

all best,

jgart




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

* [bug#49469] [PATCH] gnu: Add python-pysox.
  2021-07-08 18:45 [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
@ 2021-07-08 19:35 ` jgart via Guix-patches via
  2021-10-09 15:20   ` Arun Isaac
  2021-10-10  6:15 ` [bug#49469] jgart via Guix-patches via
                   ` (4 subsequent siblings)
  5 siblings, 1 reply; 12+ messages in thread
From: jgart via Guix-patches via @ 2021-07-08 19:35 UTC (permalink / raw)
  To: 49469; +Cc: jgart

* gnu/packages/audio.scm (python-pysox): New variable.
---
 gnu/packages/audio.scm | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 3939d4bc7e..5222fb0343 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -2385,6 +2385,36 @@ compensation, (de)interleaving, and byte-swapping
     ;; original developer.
     (license license:expat)))
 
+(define-public python-pysox
+  (package
+    (name "python-pysox")
+    (version "1.4.1")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (pypi-uri "sox" version))
+        (sha256
+          (base32
+            "0m8hszh51lfzacbb4nxc5bn446c0dzljfqgnsff8h2s5j8vd3wmh"))))
+    (build-system python-build-system)
+    (arguments
+     ;; Tests require python-soundfile.
+     ;; TODO: enable tests once http://issues.guix.gnu.org/47181 is completed.
+     `(#:tests? #f))
+    (propagated-inputs
+      `(("python-numpy" ,python-numpy)))
+    (native-inputs
+      `(("python-pytest" ,python-pytest)
+        ("python-pytest-cov" ,python-pytest-cov)))
+    (home-page "https://github.com/rabitt/pysox")
+    (synopsis "Python wrapper around sox")
+    (description "@code{pysox} is a wrapper around the @command{sox} command line tool.
+ pysox's API offers @code{Transformer} and @code{Combiner} classes that allow the user to
+incrementally build up effects and audio manipulations.  @code{pysox} also
+provides methods for querying audio information such as sample rate,
+determining whether an audio file is silent and much more.")
+    (license license:bsd-3)))
+
 (define-public python-pyaudio
   (package
     (name "python-pyaudio")
-- 
2.31.1





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

* [bug#49469] [PATCH] gnu: Add python-pysox.
  2021-07-08 19:35 ` jgart via Guix-patches via
@ 2021-10-09 15:20   ` Arun Isaac
  0 siblings, 0 replies; 12+ messages in thread
From: Arun Isaac @ 2021-10-09 15:20 UTC (permalink / raw)
  To: jgart; +Cc: 49469

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


Hi jgart,

> +     ;; Tests require python-soundfile.
> +     ;; TODO: enable tests once http://issues.guix.gnu.org/47181 is completed.
> +     `(#:tests? #f))

python-soundfile is now in Guix. Could you try getting the tests
working?

Thank you!
Arun

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

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

* [bug#49469]
  2021-07-08 18:45 [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
  2021-07-08 19:35 ` jgart via Guix-patches via
@ 2021-10-10  6:15 ` jgart via Guix-patches via
  2021-10-10  6:15   ` [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
  2021-10-16 10:02 ` Arun Isaac
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 12+ messages in thread
From: jgart via Guix-patches via @ 2021-10-10  6:15 UTC (permalink / raw)
  To: 49469

Here is an updated patch. I got the tests to pass. I also realized that I had 
to wrap some binaries.

all best,

jgart








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

* [bug#49469] [PATCH] gnu: Add python-pysox.
  2021-10-10  6:15 ` [bug#49469] jgart via Guix-patches via
@ 2021-10-10  6:15   ` jgart via Guix-patches via
  0 siblings, 0 replies; 12+ messages in thread
From: jgart via Guix-patches via @ 2021-10-10  6:15 UTC (permalink / raw)
  To: 49469; +Cc: jgart

* gnu/packages/audio.scm (python-pysox): New variable.
---
 gnu/packages/audio.scm | 57 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 57 insertions(+)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index cc51ac51f3..8b412a63d1 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -2429,6 +2429,63 @@ compensation, (de)interleaving, and byte-swapping
     ;; original developer.
     (license license:expat)))
 
+(define-public python-pysox
+  (let ((commit "3d0053381c24ae3490f759d4de87194b85789d36")
+        (revision "0"))
+    (package
+      (name "python-pysox")
+      (version (git-version "1.4.2" revision commit))
+      (source
+        (origin
+          (method git-fetch)
+          (uri (git-reference
+                 (url "https://github.com/rabitt/pysox")
+                 (commit commit)))
+          (file-name (git-file-name name version))
+          (sha256
+            (base32
+              "0i62jx92vfpcr2z7lp69yzqdi9idfs3pifl3rzm2akc2c4cr1mac"))))
+      (build-system python-build-system)
+      (arguments
+        `(#:phases
+          (modify-phases %standard-phases
+            (add-after 'unpack 'patch-sox
+              (lambda* (#:key inputs #:allow-other-keys)
+                (let* ((sox-store-path (assoc-ref inputs "sox"))
+                       (sox-bin (string-append sox-store-path "/bin/sox")))
+                  (substitute* "sox/__init__.py"
+                    (("sox -h")
+                     (string-append sox-bin " -h")))
+                  (substitute* "sox/core.py"
+                    (("\\['sox")
+                     (string-append "['" sox-bin)))
+                  (substitute* "sox/log.py"
+                    (("Logger\\('sox")
+                     (string-append "Logger\('" sox-bin))))))
+            (replace 'check
+              (lambda* (#:key inputs outputs tests? #:allow-other-keys)
+                (when tests?
+                  (add-installed-pythonpath inputs outputs)
+                  (setenv "PYTHONPATH" (string-append ".:" (getenv "PYTHONPATH")))
+                  (invoke "pytest")))))))
+      (propagated-inputs
+        `(("python-numpy" ,python-numpy)
+          ("python-typing-extensions" ,python-typing-extensions)))
+      (native-inputs
+        `(("sox" ,sox)
+          ("python-pytest" ,python-pytest)
+          ("python-pytest-cov" ,python-pytest-cov)
+          ("python-soundfile" ,python-soundfile)))
+      (home-page "https://github.com/rabitt/pysox")
+      (synopsis "Python wrapper around sox")
+      (description
+"@code{pysox} is a wrapper around the @command{sox} command line tool.
+ pysox's API offers @code{Transformer} and @code{Combiner} classes that allow the user to
+incrementally build up effects and audio manipulations.  @code{pysox} also
+provides methods for querying audio information such as sample rate,
+determining whether an audio file is silent and much more.")
+      (license license:bsd-3))))
+
 (define-public python-pyaudio
   (package
     (name "python-pyaudio")
-- 
2.33.0





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

* [bug#49469] [PATCH] gnu: Add python-pysox.
  2021-07-08 18:45 [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
  2021-07-08 19:35 ` jgart via Guix-patches via
  2021-10-10  6:15 ` [bug#49469] jgart via Guix-patches via
@ 2021-10-16 10:02 ` Arun Isaac
  2021-10-16 10:05 ` Arun Isaac
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 12+ messages in thread
From: Arun Isaac @ 2021-10-16 10:02 UTC (permalink / raw)
  To: jgart; +Cc: 49469

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


Hi jgart,

Thanks for the updated patch! Some feedback follows.

> +(define-public python-pysox
> +  (let ((commit "3d0053381c24ae3490f759d4de87194b85789d36")
> +        (revision "0"))
> +    (package
> +      (name "python-pysox")
> +      (version (git-version "1.4.2" revision commit))

According to PyPI, it seems that the latest version is 1.4.1.

And, I see that upstream is not tagging releases correctly in their git
repo. Could you raise this issue with upstream?

> +      (source
> +        (origin
> +          (method git-fetch)
> +          (uri (git-reference
> +                 (url "https://github.com/rabitt/pysox")

We should prefer the PyPI source tarball. If there is something missing
in the PyPI source tarball, and that's why you are using the GitHub
source, could you add a comment explaining the reason?

> +                  (substitute* "sox/log.py"
> +                    (("Logger\\('sox")
> +                     (string-append "Logger\('" sox-bin))))))

I think the above substitution is not required, because in this instance
'sox' is the name of the logger, rather than the full path to the sox
executable.

> +                  (setenv "PYTHONPATH" (string-append ".:" (getenv
> "PYTHONPATH")))

This setting of PYTHONPATH does not seem to be required. The tests
succeed without it.

> +      (description
> +"@code{pysox} is a wrapper around the @command{sox} command line tool.
> + pysox's API offers @code{Transformer} and @code{Combiner} classes that allow the user to
> +incrementally build up effects and audio manipulations.  @code{pysox} also
> +provides methods for querying audio information such as sample rate,
> +determining whether an audio file is silent and much more.")

I think @code{pysox} should be @code{python-pysox} since we are
referring to the python-pysox scheme variable.

Could you send an updated patch? Also, please do use `git format-patch
-v2', `git format-patch -v3', etc. when sending updated patches.

Thanks!
Arun

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

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

* [bug#49469] [PATCH] gnu: Add python-pysox.
  2021-07-08 18:45 [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
                   ` (2 preceding siblings ...)
  2021-10-16 10:02 ` Arun Isaac
@ 2021-10-16 10:05 ` Arun Isaac
  2021-11-24 16:25 ` Arun Isaac
  2021-11-25  4:42 ` [bug#49469] [PATCH v3] " jgart via Guix-patches via
  5 siblings, 0 replies; 12+ messages in thread
From: Arun Isaac @ 2021-10-16 10:05 UTC (permalink / raw)
  To: jgart; +Cc: 49469

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


Also, in the next patch, please add a copyright header for yourself.

Thanks!

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

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

* [bug#49469] [PATCH] gnu: Add python-pysox.
  2021-07-08 18:45 [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
                   ` (3 preceding siblings ...)
  2021-10-16 10:05 ` Arun Isaac
@ 2021-11-24 16:25 ` Arun Isaac
  2021-11-25  4:33   ` jgart via Guix-patches via
  2021-11-25  4:42 ` [bug#49469] [PATCH v3] " jgart via Guix-patches via
  5 siblings, 1 reply; 12+ messages in thread
From: Arun Isaac @ 2021-11-24 16:25 UTC (permalink / raw)
  To: jgart; +Cc: 49469

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


Hi jgart,

Just a gentle reminder. I was wondering if you missed my last mail on
this patch.

Thanks,
Arun

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

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

* [bug#49469] [PATCH] gnu: Add python-pysox.
  2021-11-24 16:25 ` Arun Isaac
@ 2021-11-25  4:33   ` jgart via Guix-patches via
  2021-11-25  4:46     ` jgart via Guix-patches via
  2021-11-29 16:12     ` bug#49469: " Arun Isaac
  0 siblings, 2 replies; 12+ messages in thread
From: jgart via Guix-patches via @ 2021-11-25  4:33 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 49469

On Wed, 24 Nov 2021 21:55:02 +0530 Arun Isaac <arunisaac@systemreboot.net> wrote:
> 
> Hi jgart,
> 
> Just a gentle reminder. I was wondering if you missed my last mail on
> this patch.
> 
> Thanks,
> Arun

Hi Arun,

Thank you for taking the time to review these patches.

Oddly enough, PyPi does not include the data folder that contains the audio files required for testing:

https://github.com/rabitt/pysox/tree/master/tests/data

Here is a snippet of the log showing test failures from not finding the data folder with the audio files:

/gnu/store/grnpyl2q67cyymcx7p27h5l64i23i4bn-python-pysox-1.4.1/lib/python3.8/site-packages/sox/file_info.py:248: OSError
________________ TestTransformerVol.test_limiter_gain_vol_up_db ________________

self = <test_transform.TestTransformerVol testMethod=test_limiter_gain_vol_up_db>

    def test_limiter_gain_vol_up_db(self):
        tfm = new_transformer()
        tfm.vol(2.0, gain_type='db', limiter_gain=0.05)

        actual_args = tfm.effects
        expected_args = ['vol', '2.000000', 'dB', '0.050000']
        self.assertEqual(expected_args, actual_args)

        actual_log = tfm.effects_log
        expected_log = ['vol']
        self.assertEqual(expected_log, actual_log)

>       actual_res = tfm.build(INPUT_FILE, OUTPUT_FILE)

tests/test_transform.py:5322:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
/gnu/store/grnpyl2q67cyymcx7p27h5l64i23i4bn-python-pysox-1.4.1/lib/python3.8/site-packages/sox/transform.py:593: in build
    input_format, input_filepath = self._parse_inputs(
/gnu/store/grnpyl2q67cyymcx7p27h5l64i23i4bn-python-pysox-1.4.1/lib/python3.8/site-packages/sox/transform.py:493: in _parse_inputs
    file_info.validate_input_file(input_filepath)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

input_filepath = '/tmp/guix-build-python-pysox-1.4.1.drv-0/sox-1.4.1/tests/data/input.wav'

    def validate_input_file(input_filepath):
        '''Input file validation function. Checks that file exists and can be
        processed by SoX.

        Parameters
        ----------
        input_filepath : str
            The input filepath.

        '''
        if not os.path.exists(input_filepath):
>           raise IOError(
                "input_filepath {} does not exist.".format(input_filepath)
            )
E           OSError: input_filepath /tmp/guix-build-python-pysox-1.4.1.drv-0/sox-1.4.1/tests/data/input.wav does not exist.

/gnu/store/grnpyl2q67cyymcx7p27h5l64i23i4bn-python-pysox-1.4.1/lib/python3.8/site-packages/sox/file_info.py:248: OSError
================= 282 failed, 373 passed, 10 skipped in 51.02s =================


Next email will contain the new patch set with your suggestions. I've included a copyright statement also.

Thank you!

all best,

jgart




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

* [bug#49469] [PATCH v3] gnu: Add python-pysox.
  2021-07-08 18:45 [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
                   ` (4 preceding siblings ...)
  2021-11-24 16:25 ` Arun Isaac
@ 2021-11-25  4:42 ` jgart via Guix-patches via
  5 siblings, 0 replies; 12+ messages in thread
From: jgart via Guix-patches via @ 2021-11-25  4:42 UTC (permalink / raw)
  To: 49469; +Cc: jgart

* gnu/packages/audio.scm (python-pysox): New variable.
---
 gnu/packages/audio.scm | 56 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 56 insertions(+)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 4b1a602804..694ea1a21b 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -34,6 +34,7 @@
 ;;; Copyright © 2020 Giacomo Leidi <goodoldpaul@autistici.org>
 ;;; Copyright © 2020, 2021 Vinicius Monego <monego@posteo.net>
 ;;; Copyright © 2020 Michael Rohleder <mike@rohleder.de>
+;;; Copyright © 2021 jgart <jgart@dismail.de>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -5532,6 +5533,61 @@ (define-public siggen
 device.  There is support for mono and/or stereo and 8 or 16 bit samples.")
     (license license:gpl2)))
 
+(define-public python-pysox
+  ;; PyPi does not include the data folder containing audio files for testing.
+  (let ((commit "3d0053381c24ae3490f759d4de87194b85789d36")
+        (revision "0"))
+    (package
+      (name "python-pysox")
+      (version (git-version "1.4.2" revision commit))
+      (source
+        (origin
+          (method git-fetch)
+          (uri (git-reference
+                 (url "https://github.com/rabitt/pysox")
+                 (commit commit)))
+          (file-name (git-file-name name version))
+          (sha256
+            (base32
+              "0i62jx92vfpcr2z7lp69yzqdi9idfs3pifl3rzm2akc2c4cr1mac"))))
+      (build-system python-build-system)
+      (arguments
+        `(#:phases
+          (modify-phases %standard-phases
+            (add-after 'unpack 'patch-sox
+              (lambda* (#:key inputs #:allow-other-keys)
+                (let* ((sox-store-path (assoc-ref inputs "sox"))
+                       (sox-bin (string-append sox-store-path "/bin/sox")))
+                  (substitute* "sox/__init__.py"
+                    (("sox -h")
+                     (string-append sox-bin " -h")))
+                  (substitute* "sox/core.py"
+                    (("\\['sox")
+                     (string-append "['" sox-bin))))))
+            (replace 'check
+              (lambda* (#:key inputs outputs tests? #:allow-other-keys)
+                (when tests?
+                  (add-installed-pythonpath inputs outputs)
+                  (invoke "pytest")))))))
+      (propagated-inputs
+        `(("python-numpy" ,python-numpy)
+          ("python-typing-extensions" ,python-typing-extensions)))
+      (native-inputs
+        `(("sox" ,sox)
+          ("python-pytest" ,python-pytest)
+          ("python-pytest-cov" ,python-pytest-cov)
+          ("python-soundfile" ,python-soundfile)))
+      (home-page "https://github.com/rabitt/pysox")
+      (synopsis "Python wrapper around SoX")
+      (description
+"@code{python-pysox} is a wrapper around the @command{sox} command
+line tool.  The API offers @code{Transformer} and @code{Combiner}
+classes that allow the user to incrementally build up effects and audio
+manipulations.  @code{python-pysox} also provides methods for querying
+audio information such as sample rate, determining whether an audio file
+is silent, and much more.")
+      (license license:bsd-3))))
+
 (define-public mda-lv2
   (package
     (name "mda-lv2")
-- 
2.34.0





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

* [bug#49469] [PATCH] gnu: Add python-pysox.
  2021-11-25  4:33   ` jgart via Guix-patches via
@ 2021-11-25  4:46     ` jgart via Guix-patches via
  2021-11-29 16:12     ` bug#49469: " Arun Isaac
  1 sibling, 0 replies; 12+ messages in thread
From: jgart via Guix-patches via @ 2021-11-25  4:46 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 49469

On Wed, 24 Nov 2021 23:33:28 -0500 jgart <jgart@dismail.de> wrote:

I sent the latest patch set (v3) but forgot to Cc you ;)

all best,

jgart




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

* bug#49469: [PATCH] gnu: Add python-pysox.
  2021-11-25  4:33   ` jgart via Guix-patches via
  2021-11-25  4:46     ` jgart via Guix-patches via
@ 2021-11-29 16:12     ` Arun Isaac
  1 sibling, 0 replies; 12+ messages in thread
From: Arun Isaac @ 2021-11-29 16:12 UTC (permalink / raw)
  To: jgart; +Cc: 49469-done

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


Hi jgart,

Thank you for your revised patch! I have pushed it to master.

> Oddly enough, PyPi does not include the data folder that contains the
> audio files required for testing:

Personally, I believe this is bad practice, but sadly it is quite
common.

Regards,
Arun

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

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

end of thread, other threads:[~2021-11-29 16:14 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-08 18:45 [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
2021-07-08 19:35 ` jgart via Guix-patches via
2021-10-09 15:20   ` Arun Isaac
2021-10-10  6:15 ` [bug#49469] jgart via Guix-patches via
2021-10-10  6:15   ` [bug#49469] [PATCH] gnu: Add python-pysox jgart via Guix-patches via
2021-10-16 10:02 ` Arun Isaac
2021-10-16 10:05 ` Arun Isaac
2021-11-24 16:25 ` Arun Isaac
2021-11-25  4:33   ` jgart via Guix-patches via
2021-11-25  4:46     ` jgart via Guix-patches via
2021-11-29 16:12     ` bug#49469: " Arun Isaac
2021-11-25  4:42 ` [bug#49469] [PATCH v3] " jgart via Guix-patches via

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.