all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#67304: Build python-cdflib.x86_64-linux on master is broken.
       [not found] <1881@guix-ci.nnrss>
@ 2023-11-20 19:02 ` Maxim Cournoyer
  2023-11-20 19:25   ` Eric Bavier
  0 siblings, 1 reply; 5+ messages in thread
From: Maxim Cournoyer @ 2023-11-20 19:02 UTC (permalink / raw)
  To: 67304; +Cc: Sharlatan Hellseher, Eric Bavier

Hello!

cuirass@gnu.org (Cuirass) writes:

> <p>The build <b>python-cdflib.x86_64-linux</b> for specification <b>master</b> is broken. You can find the detailed information about this build <a
> href="https://ci.guix.gnu.org/build/2635702/details">here</a>.</p>
>
> https://ci.guix.gnu.org/build/2635702/details

It seems something in the commits series
https://git.savannah.gnu.org/cgit/guix.git/log/?qt=range&q=2ab5e449246f98b049888dde3c310f5b4a0a64a2..b7abea0fd6a146563830db1dc4ddd0cceb6fcf1c
either broke the test, else it's flaky.

-- 
Thanks,
Maxim




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

* bug#67304: Build python-cdflib.x86_64-linux on master is broken.
  2023-11-20 19:02 ` bug#67304: Build python-cdflib.x86_64-linux on master is broken Maxim Cournoyer
@ 2023-11-20 19:25   ` Eric Bavier
  2023-11-20 22:39     ` Sharlatan Hellseher
  0 siblings, 1 reply; 5+ messages in thread
From: Eric Bavier @ 2023-11-20 19:25 UTC (permalink / raw)
  To: Maxim Cournoyer, 67304; +Cc: Sharlatan Hellseher

Hi Maxim, thanks for the report.

On Mon, 2023-11-20 at 14:02 -0500, Maxim Cournoyer wrote:
> 
> It seems something in the commits series
> https://git.savannah.gnu.org/cgit/guix.git/log/?qt=range&q=2ab5e449246f98b049888dde3c310f5b4a0a64a2..b7abea0fd6a146563830db1dc4ddd0cceb6fcf1c
> either broke the test, else it's flaky.
> 

I have built the package locally many times without fail.

The failed test is one that checks fidelity of a rountrip time format
conversion for a randomly-generated time, so failure may indeed be
intermittent.

I will try to dig a bit further, and maybe Sharlatan can as well.

`~Eric




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

* bug#67304: Build python-cdflib.x86_64-linux on master is broken.
  2023-11-20 19:25   ` Eric Bavier
@ 2023-11-20 22:39     ` Sharlatan Hellseher
  2023-11-20 22:56       ` Sharlatan Hellseher
  0 siblings, 1 reply; 5+ messages in thread
From: Sharlatan Hellseher @ 2023-11-20 22:39 UTC (permalink / raw)
  To: Eric Bavier; +Cc: Maxim Cournoyer, 67304

Hi Guix,

I've checked the cement to the test and it mentioned as a candidate to be
removed as it's not stable

https://github.com/MAVENSDC/cdflib/blob/master/tests/test_astropy_epochs.py#L116
        # Unfortunately, currently there is a pretty big loss of
precision that comes with
        # the compute function.  Need to stop testing early.

We may consider to disable it to stabilize build.

Thanks,
Oleg

On Mon, 20 Nov 2023 at 19:25, Eric Bavier <bavier@posteo.net> wrote:
>
> Hi Maxim, thanks for the report.
>
> On Mon, 2023-11-20 at 14:02 -0500, Maxim Cournoyer wrote:
> >
> > It seems something in the commits series
> > https://git.savannah.gnu.org/cgit/guix.git/log/?qt=range&q=2ab5e449246f98b049888dde3c310f5b4a0a64a2..b7abea0fd6a146563830db1dc4ddd0cceb6fcf1c
> > either broke the test, else it's flaky.
> >
>
> I have built the package locally many times without fail.
>
> The failed test is one that checks fidelity of a rountrip time format
> conversion for a randomly-generated time, so failure may indeed be
> intermittent.
>
> I will try to dig a bit further, and maybe Sharlatan can as well.
>
> `~Eric



-- 

… наш разум - превосходная объяснительная машина которая способна
найти смысл почти в чем угодно, истолковать любой феномен, но
совершенно не в состоянии принять мысль о непредсказуемости.

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

* bug#67304: Build python-cdflib.x86_64-linux on master is broken.
  2023-11-20 22:39     ` Sharlatan Hellseher
@ 2023-11-20 22:56       ` Sharlatan Hellseher
  2023-12-04 20:52         ` Eric Bavier
  0 siblings, 1 reply; 5+ messages in thread
From: Sharlatan Hellseher @ 2023-11-20 22:56 UTC (permalink / raw)
  To: 67304; +Cc: Maxim Cournoyer, Eric Bavier

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

Hi,

I've open an issue in upstram.
Mean while please check the patch disabling the test.

Thanks,
Oleg

On Mon, 20 Nov 2023 at 22:39, Sharlatan Hellseher <sharlatanus@gmail.com> wrote:
>
> Hi Guix,
>
> I've checked the cement to the test and it mentioned as a candidate to be
> removed as it's not stable
>
> https://github.com/MAVENSDC/cdflib/blob/master/tests/test_astropy_epochs.py#L116
>         # Unfortunately, currently there is a pretty big loss of
> precision that comes with
>         # the compute function.  Need to stop testing early.
>
> We may consider to disable it to stabilize build.
>
> Thanks,
> Oleg
>
> On Mon, 20 Nov 2023 at 19:25, Eric Bavier <bavier@posteo.net> wrote:
> >
> > Hi Maxim, thanks for the report.
> >
> > On Mon, 2023-11-20 at 14:02 -0500, Maxim Cournoyer wrote:
> > >
> > > It seems something in the commits series
> > > https://git.savannah.gnu.org/cgit/guix.git/log/?qt=range&q=2ab5e449246f98b049888dde3c310f5b4a0a64a2..b7abea0fd6a146563830db1dc4ddd0cceb6fcf1c
> > > either broke the test, else it's flaky.
> > >
> >
> > I have built the package locally many times without fail.
> >
> > The failed test is one that checks fidelity of a rountrip time format
> > conversion for a randomly-generated time, so failure may indeed be
> > intermittent.
> >
> > I will try to dig a bit further, and maybe Sharlatan can as well.
> >
> > `~Eric
>
>
>
> --
>
> … наш разум - превосходная объяснительная машина которая способна
> найти смысл почти в чем угодно, истолковать любой феномен, но
> совершенно не в состоянии принять мысль о непредсказуемости.



-- 

… наш разум - превосходная объяснительная машина которая способна
найти смысл почти в чем угодно, истолковать любой феномен, но
совершенно не в состоянии принять мысль о непредсказуемости.

[-- Attachment #2: 0001-gnu-python-cdflib-Disable-shaky-test.patch --]
[-- Type: text/x-patch, Size: 1336 bytes --]

From 2f55a5ac236952ea24d81998da488cff4553c272 Mon Sep 17 00:00:00 2001
Message-ID: <2f55a5ac236952ea24d81998da488cff4553c272.1700520904.git.sharlatanus@gmail.com>
From: Sharlatan Hellseher <sharlatanus@gmail.com>
Date: Mon, 20 Nov 2023 22:52:52 +0000
Subject: [PATCH] gnu: python-cdflib: Disable shaky test.

* gnu/packages/astronomy.scm (python-cdflib): Disable one test which
causing random build failure.
[arguments]{test-flags}: Add it.

Change-Id: I05ee2feca3bc0f0139fa1a5f00b4fe260b42ec80
---
 gnu/packages/astronomy.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/astronomy.scm b/gnu/packages/astronomy.scm
index c063285b52..da15283ef3 100644
--- a/gnu/packages/astronomy.scm
+++ b/gnu/packages/astronomy.scm
@@ -1807,7 +1807,10 @@ (define-public python-cdflib
         (base32 "0vpgcbc9pmx0qqfia1frnwq3jkgfp8y3ikqdnzs5bs1sr13p9p3w"))))
     (build-system pyproject-build-system)
     (arguments
-     (list #:phases
+     ;; Disable shaky test.
+     ;; See https://github.com/MAVENSDC/cdflib/issues/234
+     (list #:test-flags #~(list "-k" "not test_compute_cdfepoch16")
+           #:phases
            #~(modify-phases %standard-phases
                (add-before 'build 'set-env-version
                  (lambda _

base-commit: d20ece07dbb09382f361c8bbf0bcab9e83d8b73e
-- 
2.41.0


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

* bug#67304: Build python-cdflib.x86_64-linux on master is broken.
  2023-11-20 22:56       ` Sharlatan Hellseher
@ 2023-12-04 20:52         ` Eric Bavier
  0 siblings, 0 replies; 5+ messages in thread
From: Eric Bavier @ 2023-12-04 20:52 UTC (permalink / raw)
  To: Sharlatan Hellseher, 67304-done; +Cc: Maxim Cournoyer

On Mon, 2023-11-20 at 22:56 +0000, Sharlatan Hellseher wrote:
> Hi,
> 
> I've open an issue in upstram.
> Mean while please check the patch disabling the test.

I've pushed this patch as 33f9d3cc36fbfbd2849d626492721667abfe8306.

We can revisit if we ever get a response from upstream.

`~Eric





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

end of thread, other threads:[~2023-12-04 20:53 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1881@guix-ci.nnrss>
2023-11-20 19:02 ` bug#67304: Build python-cdflib.x86_64-linux on master is broken Maxim Cournoyer
2023-11-20 19:25   ` Eric Bavier
2023-11-20 22:39     ` Sharlatan Hellseher
2023-11-20 22:56       ` Sharlatan Hellseher
2023-12-04 20:52         ` Eric Bavier

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.