search

rasterio / affine

Affine transformation matrices
BSD 3-Clause "New" or "Revised" License
156 stars 28 forks source link

Add support to ppc64le #63

Closed gururajrkatti closed 3 years ago

gururajrkatti commented 3 years ago

This PR adds CI support for the IBM Power Little Endian (ppc64le) architecture. The idea is to ensure that the builds on this architecture are continuously tested along with the Intel builds (amd64) as this is part of the ubuntu distro on that architecture as well and detecting (and fixing) any issues or failures early would help to ensure that we are always up to date. Since ppc64le focal doesn't have pypy3 support, excluded only that.

coveralls commented 3 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 4ea38e80cd17918659ac678595f70479586168c8 on gururajrkatti:master into d9d4050191b16dab3a26307596bedcce469a177f on sgillies:master.

sgillies commented 3 years ago

Thanks @gururajrkatti , but I'm going to decline this PR. The affine package uses only Python standard library modules and so these architecture-specific jobs are redundant. Also, Travis CI is cutting off support for open source projects without a subscription and I need to reduce not increase my Travis build time usage.

gururajrkatti commented 3 years ago

Hi @sgillies, thanks for your response and feedback, I understand your point. I agree that since it is python standard package, it ideally should work on all architectures. However, in our past experience we have run into architecture specific issues even with pure python packages with their tests failing in some cases , mostly because of an indirect dependency on native code or libraries via other python dependencies. by enabling the travis support here, we are attempting to catch those issues early, hope that makes some sense? Regarding Travis CI cutting support for open source, I don't think that is the case. You can refer to the related blog https://blog.travis-ci.com/oss-announcement. For requesting more credit points for your builds on travis you can get it through support.travis-ci.com. Adding @gerrith3 for more details