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
Attachment:
signature.asc
Description: PGP signature