diff mbox series

[bug#46842] gnu: gnuradio: Updated to 3.9.0.0.

Message ID 878s6g167l.fsf@gnu.org
State Accepted
Headers show
Series [bug#46842] gnu: gnuradio: Updated to 3.9.0.0. | expand

Checks

Context Check Description
cbaines/submitting builds success
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch fail View Laminar job
cbaines/issue success View issue

Commit Message

Ludovic Courtès March 21, 2021, 10:40 p.m. UTC
Hi,

João Pedro Simas <jpsimas@gmail.com> skribis:

> Hi,
> Sorry, this took a bit more time than I expected, but now I think I've
> figured it out.
>
> About the bundled copy of volk, it was removed in this commit
> https://github.com/gnuradio/gnuradio/commit/80c04479da962d048d41165081b026aafdaa0316,
> that is included in 3.9.0.0. I've also removed the:
>>         '("-DENABLE_INTERNAL_VOLK=OFF")
> configure-flag because the env var is not used anymore in gnuradio.

OK.

> I've also added the urls of the issues concerning the tests, but, in the
> end, I found that one of the tests (qa_dtv) was failing due to sdl not
> being included as an input and there, so I've included it, and that there's
> another test that fails, but intermittently (qa_packet_headerparser_b) so I
> disabled it in addition to the original one (qa_header_payload_demux).
>
> I also realized I had forgotten to add a change to the patch file related
> to the lines that patch the references to codec2 inside a swig file that
> was removed in 3.9, because swig was replaced with pybind11. I've added
> that too.
>
> Finally, I wrote the changelog in the commit log. The updated patch file is
> attached.

The patch LGTM but I got this one ‘qa_dtv’ test failure on x86_64:

--8<---------------cut here---------------start------------->8---
222/241 Test #226: qa_cvsd_vocoder ..............................   Passed    1.18 sec
        Start 230: qa_ulaw_vocoder
223/241 Test #228: qa_g723_24_vocoder ...........................   Passed    0.93 sec
        Start 231: qa_codec2_vocoder
224/241 Test #218: qa_dtv .......................................***Failed    3.73 sec
F
======================================================================
FAIL: test_000 (__main__.test_dtv)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/tmp/guix-build-gnuradio-3.9.0.0.drv-0/gnuradio-3.9.0.0/gr-dtv/python/dtv/qa_dtv.py", line 171, in test_000
    self.assertFloatTuplesAlmostEqual(out_data, expected_data, 5)
  File "/tmp/guix-build-gnuradio-3.9.0.0.drv-0/gnuradio-3.9.0.0/gnuradio-runtime/python/gnuradio/gr_unittest.py", line 96, in assertFloatTuplesAlmostEqual
    return all([
  File "/tmp/guix-build-gnuradio-3.9.0.0.drv-0/gnuradio-3.9.0.0/gnuradio-runtime/python/gnuradio/gr_unittest.py", line 97, in <listcomp>
    self.assertAlmostEqual(x, y, places, msg)
AssertionError: -0.0002739299 != -0.0011434406 within 5 places (0.0008695107 difference)

----------------------------------------------------------------------
Ran 1 test in 2.559s

FAILED (failures=1)
--8<---------------cut here---------------end--------------->8---

Does that ring a bell?

Attached is a variant of the patch where I adjusted indentation to
follow our conventions.

BTW, please send patches as attachments in the future (this one had been
mangled on its way).

Thanks,
Ludo’.

PS: Please keep the bug Cc’d.

Comments

Guillaume Le Vaillant March 22, 2021, 10:46 a.m. UTC | #1
Ludovic Courtès <ludo@gnu.org> skribis:

> The patch LGTM but I got this one ‘qa_dtv’ test failure on x86_64:
>
> --8<---------------cut here---------------start------------->8---
> 222/241 Test #226: qa_cvsd_vocoder ..............................   Passed    1.18 sec
>         Start 230: qa_ulaw_vocoder
> 223/241 Test #228: qa_g723_24_vocoder ...........................   Passed    0.93 sec
>         Start 231: qa_codec2_vocoder
> 224/241 Test #218: qa_dtv .......................................***Failed    3.73 sec
> F
> ======================================================================
> FAIL: test_000 (__main__.test_dtv)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File "/tmp/guix-build-gnuradio-3.9.0.0.drv-0/gnuradio-3.9.0.0/gr-dtv/python/dtv/qa_dtv.py", line 171, in test_000
>     self.assertFloatTuplesAlmostEqual(out_data, expected_data, 5)
>   File "/tmp/guix-build-gnuradio-3.9.0.0.drv-0/gnuradio-3.9.0.0/gnuradio-runtime/python/gnuradio/gr_unittest.py", line 96, in assertFloatTuplesAlmostEqual
>     return all([
>   File "/tmp/guix-build-gnuradio-3.9.0.0.drv-0/gnuradio-3.9.0.0/gnuradio-runtime/python/gnuradio/gr_unittest.py", line 97, in <listcomp>
>     self.assertAlmostEqual(x, y, places, msg)
> AssertionError: -0.0002739299 != -0.0011434406 within 5 places (0.0008695107 difference)
>
> ----------------------------------------------------------------------
> Ran 1 test in 2.559s
>
> FAILED (failures=1)
> --8<---------------cut here---------------end--------------->8---
>
> Does that ring a bell?

Hi,

I just tested this patch, and I don't reproduce the issue with the
qa_dtv test. Does it fail every time on your machine?
Ludovic Courtès March 23, 2021, 10:33 a.m. UTC | #2
Hi,

Guillaume Le Vaillant <glv@posteo.net> skribis:

> I just tested this patch, and I don't reproduce the issue with the
> qa_dtv test. Does it fail every time on your machine?

I just tried once, so perhaps it’s a transient failure?  If it sounds
good to you Guillaume, feel free to push.

Thanks,
Ludo’.
Guillaume Le Vaillant March 24, 2021, 11:10 a.m. UTC | #3
I added a copyright line, tweaked the commit message and pushed the
patch as 083809229bf199399c33b072056be77dbfd75555.
In the following commits, I updated/fixed some packages depending on
gnuradio to make them work with the new version.
Ludovic Courtès March 24, 2021, 4:17 p.m. UTC | #4
Guillaume Le Vaillant <glv@posteo.net> skribis:

> I added a copyright line, tweaked the commit message and pushed the
> patch as 083809229bf199399c33b072056be77dbfd75555.
> In the following commits, I updated/fixed some packages depending on
> gnuradio to make them work with the new version.

Thank you!

Ludo’.
diff mbox series

Patch

From 069a988e65e28764b3cbab137aa7b1a90464d69f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C3=A3o?= <jpsimas@gmail.com>
Date: Fri, 12 Mar 2021 19:42:02 -0300
Subject: [PATCH] gnu: gnuradio: Update to 3.9.0.0.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* gnu/packages/radio.scm (gnuradio): Update to 3.9.0.0.
 - Replace swig with pybind11.
 - Remove sections related to removing/deleting the included copy of volk,
as it is not included anymore.
 - Add sdl as an input to prevent the test "qa_dtv" from failing.
 - Disable broken tests.

Signed-off-by: Ludovic Courtès <ludo@gnu.org>
---
 gnu/packages/radio.scm | 32 +++++++++++++++++++-------------
 1 file changed, 19 insertions(+), 13 deletions(-)

diff --git a/gnu/packages/radio.scm b/gnu/packages/radio.scm
index 3378b36adb..df63ff6868 100644
--- a/gnu/packages/radio.scm
+++ b/gnu/packages/radio.scm
@@ -67,6 +67,7 @@ 
   #:use-module (gnu packages qt)
   #:use-module (gnu packages readline)
   #:use-module (gnu packages ruby)
+  #:use-module (gnu packages sdl)
   #:use-module (gnu packages sphinx)
   #:use-module (gnu packages swig)
   #:use-module (gnu packages tcl)
@@ -286,32 +287,26 @@  used by RDS Spy, and audio files containing @dfn{multiplex} signals (MPX).")
 (define-public gnuradio
   (package
     (name "gnuradio")
-    (version "3.8.0.0")
+    (version "3.9.0.0")
     (source
      (origin
        (method url-fetch)
        (uri (string-append "https://www.gnuradio.org/releases/gnuradio/"
                            "gnuradio-" version ".tar.xz"))
        (sha256
-        (base32 "0aw55gf5549b0fz2qdi7vplcmaf92bj34h40s34b2ycnqasv900r"))
-       (modules '((guix build utils)))
-       (snippet
-        '(begin
-           ;; Delete bundled volk to use the shared one.
-           (delete-file-recursively "volk")
-           #t))))
+        (base32 "1jvm9xd0l2pz1fww4zii6hl7ccnvy256nrf70ljb594n7j9j49ha"))))
     (build-system cmake-build-system)
     (native-inputs
      `(("doxygen" ,doxygen)
        ("ghostscript" ,ghostscript)
        ("orc" ,orc)
        ("pkg-config" ,pkg-config)
+       ("pybind11" ,pybind11)
        ("python-cheetah" ,python-cheetah)
        ("python-mako" ,python-mako)
        ("python-pyzmq" ,python-pyzmq)
        ("python-scipy" ,python-scipy)
        ("python-sphinx" ,python-sphinx)
-       ("swig" ,swig)
        ("texlive" ,(texlive-union (list texlive-amsfonts
                                         texlive-latex-amsmath
                                         ;; TODO: Add newunicodechar.
@@ -343,6 +338,7 @@  used by RDS Spy, and audio files containing @dfn{multiplex} signals (MPX).")
        ("python-pyyaml" ,python-pyyaml)
        ("qtbase" ,qtbase)
        ("qwt" ,qwt)
+       ("sdl" ,sdl)
        ("volk" ,volk)
        ("zeromq" ,zeromq)))
     (arguments
@@ -354,8 +350,6 @@  used by RDS Spy, and audio files containing @dfn{multiplex} signals (MPX).")
        #:imported-modules (,@%cmake-build-system-modules
                            (guix build glib-or-gtk-build-system)
                            (guix build python-build-system))
-       #:configure-flags
-       '("-DENABLE_INTERNAL_VOLK=OFF")
        #:phases
        (modify-phases %standard-phases
          (add-after 'unpack 'fix-paths
@@ -371,8 +365,7 @@  used by RDS Spy, and audio files containing @dfn{multiplex} signals (MPX).")
              (substitute* "cmake/Modules/GrPython.cmake"
                (("dist-packages")
                 "site-packages"))
-             (substitute* '("gr-vocoder/swig/vocoder_swig.i"
-                            "gr-vocoder/include/gnuradio/vocoder/codec2.h"
+             (substitute* '("gr-vocoder/include/gnuradio/vocoder/codec2.h"
                             "gr-vocoder/include/gnuradio/vocoder/freedv_api.h")
                (("<codec2/")
                 "<"))
@@ -384,6 +377,19 @@  used by RDS Spy, and audio files containing @dfn{multiplex} signals (MPX).")
                                     "/bin/Xvfb :1 &"))
              (setenv "DISPLAY" ":1")
              #t))
+         (replace 'check
+           (lambda* (#:key tests? parallel-tests? #:allow-other-keys)
+             (invoke "ctest" "-j" (if parallel-tests?
+                                      (number->string (parallel-job-count))
+                                      "1")
+                     "--output-on-failure"
+                     ;;disable broken tests
+                     "-E" (string-join
+                           '(;; https://github.com/gnuradio/gnuradio/issues/3871
+                             "qa_header_payload_demux"
+                             ;; https://github.com/gnuradio/gnuradio/issues/4348
+                             "qa_packet_headerparser_b")
+                           "|"))))
          (add-after 'install 'wrap-python
            (assoc-ref python:%standard-phases 'wrap))
          (add-after 'wrap-python 'wrap-glib-or-gtk
-- 
2.31.0