Source: astropy-healpix Source-Version: 0.2-2 Severity: wishlish Control: forwarded -1 https://github.com/astropy/astropy-healpix/issues/86 Control: tags -1 patch Control: affects -1 src:reproject
Hi Leo, as long as the upstream issue #86 is unfixed, it may be useful to apply this patch, which pragmatically fixes the issue. And it should not harm the other archs too much -- the case that the function returns a value that does not fit in 32 bit seems a bit rare to me in the moment. The problem with the FTBFS is that astropy-healpix is a new dependency for reproject, which now cannot migrate because of the missing mips build. Best regards Ole
From: Ole Streicher <oleb...@debian.org> Date: Tue, 21 Aug 2018 21:06:34 +0200 Subject: Revert return type of healpix_rangesearch_radec_simple to int This partially reverts https://github.com/astropy/astropy-healpix/pull/15 (commit #fd48d256364a33d37950c41e95736db6810e9f33), and pragmatically solves the testing failure on MIPS big endian. See https://github.com/astropy/astropy-healpix/issues/86 for the discussion. The patch should be reverted once a final solution is found there. --- cextern/astrometry.net/healpix-utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cextern/astrometry.net/healpix-utils.c b/cextern/astrometry.net/healpix-utils.c index 809010f..c833a8f 100644 --- a/cextern/astrometry.net/healpix-utils.c +++ b/cextern/astrometry.net/healpix-utils.c @@ -139,7 +139,7 @@ ll* healpix_rangesearch_radec(double ra, double dec, double radius, int Nside, l // The following is a version of the healpix_rangesearch_radec function // that works with standard C types to make interfacing with Python easier. -int64_t healpix_rangesearch_radec_simple(double ra, double dec, double radius, +int healpix_rangesearch_radec_simple(double ra, double dec, double radius, int Nside, int approx, int64_t **indices) { double xyz[3]; ll* hps = ll_new(256); @@ -151,5 +151,5 @@ int64_t healpix_rangesearch_radec_simple(double ra, double dec, double radius, return(-1); } ll_copy(hps, 0, hps->N, *indices); - return (int64_t)ll_size(hps); + return (int)ll_size(hps); }