python-mip icon indicating copy to clipboard operation
python-mip copied to clipboard

add aarch64 static cbc library for arm64 architecture

Open knutster opened this issue 1 year ago • 8 comments
trafficstars

add aarch64 static cbc library for arm64 architectures. Verified on AWS Graviton 2 / 3 instances.

knutster avatar Oct 02 '24 15:10 knutster

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Oct 02 '24 15:10 CLAassistant

Added build scripts for ARM linux

knutster avatar Oct 03 '24 08:10 knutster

Hey @knutster, which version of the compiled libraries did you use? Were you able to compile this using the master branch of all of them? I'm trying to compile myself for a Linux build, but I'm having some issues.

JoOkuma avatar Dec 18 '24 17:12 JoOkuma

Looks like this was already done in #367 .

tkralphs avatar May 06 '25 15:05 tkralphs

Hi @tkralphs, but this was never released into pypi, https://pypi.org/project/mip/#history It's only available on version 1.16rc0.

JoOkuma avatar May 06 '25 20:05 JoOkuma

Yes, it has not been released yet, but it is on PyPi and easily pip installable. Making the release is a separate issue and that should be easy once the maintainers decide they want to do that. Regardless of the release, this PR is a duplicate of #367 as far as I can see so I just wanted to alert you to that.

tkralphs avatar May 07 '25 01:05 tkralphs

@tkralphs, I completely agree it's is easily installable, but it's an issue when using dependencies because often release candidates are ignored and not considered valid packages so it seems as arm macos version is unavailable.

Nonetheless, I recognize most of you maintain this package on your free time and I don't mean to be pushy, I just wanted to provide some insight on why they might have opened this PR.

JoOkuma avatar May 07 '25 04:05 JoOkuma

@JoOkuma OK, sorry if I misinterpreted, I thought you were saying that this PR should remain open. Indeed, it would be better if there was a release on Pypi with the ARM64 binaries.

tkralphs avatar May 08 '25 02:05 tkralphs

@tkralphs Just to clarify, the PR you are referring to, #367 , and version 1.16rc0, has support for Apple ARM silicon, not Linux aarch64 ARM which is what this PR is about.

knutster avatar Aug 14 '25 07:08 knutster

@knutster, sorry, you are right, I missed that!

tkralphs avatar Aug 14 '25 13:08 tkralphs

I'm a bit hesitant to merge this without any testing. Would it be possible to extend the CI to run tests on the Linux arm64 runners?

tkralphs avatar Aug 14 '25 13:08 tkralphs

Sure, I'll add it.

knutster avatar Aug 23 '25 07:08 knutster