Message ID | 87tugbrwiv.fsf@planete-kraus.eu |
---|---|
State | Accepted |
Headers | show |
Series | [bug#51890] Vigra: disable more failing tests on core-updates-frozen | expand |
Context | Check | Description |
---|---|---|
cbaines/applying patch | fail | View Laminar job |
cbaines/issue | success | View issue |
Hi, Vivien Kraus <vivien@planete-kraus.eu> skribis: > From 4311c36dcd280ba385a3d0f33800428102bef0b8 Mon Sep 17 00:00:00 2001 > From: Vivien Kraus <vivien@planete-kraus.eu> > Date: Tue, 16 Nov 2021 08:18:42 +0000 > Subject: [PATCH] gnu: vigra: Disable more failing tests. > > * gnu/packages/image.scm (vigra) [phase disable-broken-tests]: Also disable > test2, test3, test_arraytypes and test_multidef. Looks like it was worked around by disabling NumPy support in Vigra, in commit 94b76dc85ca3098bf4ceb0f5b884b5708b9e8134. I’m not sure how NumPy is used; is there a preference between these two options (disabling flaky tests and removing the dependency on NumPy)? Thanks, Ludo’.
On November 18, 2021 9:09:50 AM UTC, "Ludovic Courtès" <ludo@gnu.org> wrote: >Hi, > >Vivien Kraus <vivien@planete-kraus.eu> skribis: > >> From 4311c36dcd280ba385a3d0f33800428102bef0b8 Mon Sep 17 00:00:00 2001 >> From: Vivien Kraus <vivien@planete-kraus.eu> >> Date: Tue, 16 Nov 2021 08:18:42 +0000 >> Subject: [PATCH] gnu: vigra: Disable more failing tests. >> >> * gnu/packages/image.scm (vigra) [phase disable-broken-tests]: Also disable >> test2, test3, test_arraytypes and test_multidef. > >Looks like it was worked around by disabling NumPy support in Vigra, in >commit 94b76dc85ca3098bf4ceb0f5b884b5708b9e8134. > >I’m not sure how NumPy is used; is there a preference between these two >options (disabling flaky tests and removing the dependency on NumPy)? > >Thanks, >Ludo’. It looked to me like vigra was limping along and numpy compatibility was being dragged along by patches here and there. When I built the dependencies of vigra nothing failed to build. I'm definitely not against reinstating support for numpy but I'm not sure just disabling the tests is the right way.
Hello, Efraim Flashner <efraim@flashner.co.il> writes: > On November 18, 2021 9:09:50 AM UTC, "Ludovic Courtès" <ludo@gnu.org> wrote: >>Hi, >> >>Vivien Kraus <vivien@planete-kraus.eu> skribis: >> >>> From 4311c36dcd280ba385a3d0f33800428102bef0b8 Mon Sep 17 00:00:00 2001 >>> From: Vivien Kraus <vivien@planete-kraus.eu> >>> Date: Tue, 16 Nov 2021 08:18:42 +0000 >>> Subject: [PATCH] gnu: vigra: Disable more failing tests. >>> >>> * gnu/packages/image.scm (vigra) [phase disable-broken-tests]: Also disable >>> test2, test3, test_arraytypes and test_multidef. >> >>Looks like it was worked around by disabling NumPy support in Vigra, in >>commit 94b76dc85ca3098bf4ceb0f5b884b5708b9e8134. >> >>I’m not sure how NumPy is used; is there a preference between these two >>options (disabling flaky tests and removing the dependency on NumPy)? >> >>Thanks, >>Ludo’. > > It looked to me like vigra was limping along and numpy compatibility > was being dragged along by patches here and there. When I built the > dependencies of vigra nothing failed to build. I'm definitely not > against reinstating support for numpy but I'm not sure just disabling > the tests is the right way. I know nothing about vigra, but judging from its description "[...] It is an image processing and analysis library that puts its main emphasis on customizable algorithms and data structures. [...]" it seems it'd probably make good use of Numpy? Not sure what are the consequences of doing without; perhaps efficiency. Thanks, Maxim
From 4311c36dcd280ba385a3d0f33800428102bef0b8 Mon Sep 17 00:00:00 2001 From: Vivien Kraus <vivien@planete-kraus.eu> Date: Tue, 16 Nov 2021 08:18:42 +0000 Subject: [PATCH] gnu: vigra: Disable more failing tests. * gnu/packages/image.scm (vigra) [phase disable-broken-tests]: Also disable test2, test3, test_arraytypes and test_multidef. --- gnu/packages/image.scm | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/gnu/packages/image.scm b/gnu/packages/image.scm index ed846d7141..6d5408864e 100644 --- a/gnu/packages/image.scm +++ b/gnu/packages/image.scm @@ -1230,6 +1230,12 @@ (define-public vigra ;; <https://github.com/ukoethe/vigra/issues/436>. (substitute* "vigranumpy/test/CMakeLists.txt" (("test1\\.py") "")) + (substitute* "vigranumpy/test/CMakeLists.txt" + (("test[23]\\.py") "")) + (substitute* "vigranumpy/test/CMakeLists.txt" + (("test_arraytypes\\.py") "")) + (substitute* "vigranumpy/test/CMakeLists.txt" + (("test_multidef\\.py") "")) #t))) #:configure-flags (list "-Wno-dev" ; suppress developer mode with lots of warnings -- 2.33.1