awkward icon indicating copy to clipboard operation
awkward copied to clipboard

chore(ci): use macos-15-intel for x86 wheel building and testing

Open ariostas opened this issue 2 months ago • 11 comments

The wheels that are built in macos-14 and macos-15 are identical (since they are both arm64), so I just switched to using `macos-latest.

This closes #3718 which was failing since there were duplicate wheels.

Also, I guess we quietly dropped support for x86 macs, but no one has complained, so I guess it's fine.

ariostas avatar Nov 11 '25 01:11 ariostas

Also, I guess we quietly dropped support for x86 macs, but no one has complained, so I guess it's fine.

I see x86 macos wheels under https://pypi.org/project/awkward-cpp/50/#files

ikrommyd avatar Nov 11 '25 01:11 ikrommyd

Oh right, there hasn't been an awkward-cpp release since #3666, so I guess we still have time to properly announce it and/or bump minor version when there is a new awkward-cpp release.

ariostas avatar Nov 11 '25 01:11 ariostas

The documentation preview is ready to be viewed at http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com/PR3720

github-actions[bot] avatar Nov 11 '25 01:11 github-actions[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 82.72%. Comparing base (b749e49) to head (12f27a9). :warning: Report is 487 commits behind head on main.

Additional details and impacted files

see 200 files with indirect coverage changes

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Nov 11 '25 01:11 codecov[bot]

Thanks to Henry for pointing out that macos-15-intel was an option!

ariostas avatar Nov 11 '25 02:11 ariostas

@ariostas - it looks like the ci runner does not come with llvm?

ianna avatar Dec 10 '25 15:12 ianna

@ianna I think we'll have to restrict to numba 0.62 for macOS x86 because both numba and llvmlite stopped shipping wheels for it.

ariostas avatar Dec 10 '25 15:12 ariostas

@ianna I think we'll have to restrict to numba 0.62 for macOS x86 because both numba and llvmlite stopped shipping wheels for it.

Yes. Or maybe it is time to drop macOS x86?

ianna avatar Dec 10 '25 15:12 ianna

Hmm yeah, I guess we should do that soon. Maybe for now we can give up on doing the full testing and just test with the minimal requirements. Is there a plan for a minor version bump soon? We probably should schedule the drop of support to go with that

ariostas avatar Dec 10 '25 16:12 ariostas

@ianna I think we'll have to restrict to numba 0.62 for macOS x86 because both numba and llvmlite stopped shipping wheels for it.

~~I'm a bit confused, shouldn't leaving it unpinned just grab the latest possible version (0.62) if no wheels exist for 0.63?~~

Edit: No I'm stupid. It would try to compile from source.

ikrommyd avatar Dec 10 '25 16:12 ikrommyd

Or maybe it is time to drop macOS x86?

I think for that, it's probably good to see what the scientific python community is doing in general.

ikrommyd avatar Dec 10 '25 16:12 ikrommyd

Hi @ariostas - I think, just tensorflow-cpu would be ok for ml on ubuntu. Thanks.

ianna avatar Dec 11 '25 13:12 ianna

@ianna this is ready now. The GPU tests are failing, but that's due to the cudf issue. Also, you'll need to mark some of the tests are not required since it's not longer going to test on all versions of Python.

ariostas avatar Dec 11 '25 16:12 ariostas