Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.maint.python > #14872

Re: Remaining numpy 1.24 issues on 32bit architectures

From Nilesh Patra <nilesh@debian.org>
Newsgroups linux.debian.maint.python
Subject Re: Remaining numpy 1.24 issues on 32bit architectures
Date 2023-01-30 12:40 +0100
Message-ID <FTAFX-2boa-9@gated-at.bofh.it> (permalink)
References <FTdT3-1XbN-5@gated-at.bofh.it> <FTxyp-29BG-1@gated-at.bofh.it>
Organization The Debian Project

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

Hi,

On Mon, Jan 30, 2023 at 09:12:57AM +0100, Andreas Tille wrote:
> I made some tiny steps forward ("only" 84 failures instead of 89 when I
> wrote my first mail) in the numpy 1.24 migration for 32bit architectures
> but I'm facing issues I do not have a real clue for.  In
> 
>    https://salsa.debian.org/med-team/python-skbio/-/blob/master/debian/patches/numpy-1.24.patch#L123-L126

Apologies for pointing the discussion into an orthogonal direction for
once. Ofcourse, we could try fixing these, but if you look closely, skbio
has never built on 32 bit archs ever since around 2016 on i386[3] and
it has never built on the rest of 32 bit ever since it entered debian[4]
and now this new upstream FTBFS that you point to, won't really block
migration in any way.

So my question is this: Why are we trying hard to fix this on 32-bit _now_
given that the upstream support has never been solid for this package on
32-bit archs?

> ...
> which obviosly[2] failed.  I wonder whether someone might give some
> hints how to get dtypes consistently to one integer representation which
> is the background of nearly all these test suite issues.

I can think of two alternatives to fix this:

1. There are a few type conversions to "int" (.astype(int)) in the skbio source code.
This defaults to 32-bit integer type on 32-bit machines. Explicitly
casting them to 64-bit can fix this. I happened to write a similar patch
for another package recently, see[5] if it helps.

2. Just ignore datatypes while comparing pandas dataframes with
`check_dtype` parameter. An example/reference patch here[6]

> > [1] https://buildd.debian.org/status/package.php?p=python-skbio&suite=experimental
> [2] https://salsa.debian.org/med-team/python-skbio/-/jobs/3868951
[3]: https://buildd.debian.org/status/logs.php?pkg=python-skbio&arch=i386
[4]:https://buildd.debian.org/status/logs.php?pkg=python-skbio&arch=armhf
[5]: https://salsa.debian.org/med-team/python-bioframe/-/blob/master/debian/patches/32-bits.patch
[6]: https://salsa.debian.org/python-team/packages/python-upsetplot/-/blob/master/debian/patches/ignore-dtype-while-asserting.patch

-- 
Best,
Nilesh

Back to linux.debian.maint.python | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread


Thread

Remaining numpy 1.24 issues on 32bit architectures Andreas Tille <andreas@an3as.eu> - 2023-01-29 12:20 +0100
  Re: Remaining numpy 1.24 issues on 32bit architectures Andreas Tille <andreas@an3as.eu> - 2023-01-30 09:20 +0100
    Re: Remaining numpy 1.24 issues on 32bit architectures Nilesh Patra <nilesh@debian.org> - 2023-01-30 12:40 +0100
      Re: Remaining numpy 1.24 issues on 32bit architectures Andreas Tille <tille@debian.org> - 2023-01-30 13:50 +0100

csiph-web