Allow building of tfx-bsl on mac arm64 machines

[tangm]

Building on Apple silicon causes an error due to headers for avx-specific instructions being included for arrow. This commit adds an exclusion on avx* files from arrow when building on mac arm64.

Fixes #48

Not very familiar with Bazel or the project itself, but keen to get tfx working on apple silicon. Happy to work on this to get it through.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[iindyk]

Thanks for the contribution!
We can merge this, but unfortunately we don't have testing infra setup for arm64, so there's no guarantee that this will not be (accidentally) broken, though we'll try to avoid this.

How did you test this? did you build from source & run the tests?

Could you also please drop a line in release notes saying that there's partial support for building from source on mac arm64 machines.

[tangm]

Hi @iindyk , thank you for looking at this.

Have now added to RELEASE.md. I tested this by:

• building from source in a venv with python 3.10
• Installing the built wheel into the venv
• ran what I think are the tests from run_all_tests.py

Specifically:

export USE_BAZEL_VERSION=5.3.2
python3.10 -m venv .venv
. .venv/bin/activate
pip install -U pip wheel numpy
python setup.py bdist_wheel
pip install dist/tfx_bsl-1.14.0-cp310-cp310-macosx_11_0_universal2.whl
python -m tfx_bsl.test_util.run_all_tests --start_dirs=tfx_bsl

Happy to run additional tests locally, or if I can help with getting the testing infra setup on arm64 going.

[iindyk]

Thanks @tangm! I'd be happy to get any help in the infra setup but unfortunately it's going to be all about internal VMs and resource allocations.

[iindyk]

please move this under Current Version (~line 9)

[tangm]

Oops, my bad. Done.

[Dentrax]

Hey, thanks for this. Any plans for new version cut on PyPI with new GitHub tag? @iindyk

[iindyk]

Unfortunately, we don't have releasing and test infra set up for this build. So the only way to get a mac arm64-compatible wheel at the moment is to build from source.

[Dentrax]

for this build

Sorry, I didn't mean just for this one. I was wondering if there will be any tfx-bsl 1.14.1 version (patch or minor) cut soon.

[iindyk]

there's a minor release planned in a ~ week