Skip to content

Conversation

akashchi
Copy link
Contributor

@akashchi akashchi commented Aug 5, 2025

JIRA: 171611

@akashchi akashchi added the WIP label Aug 5, 2025
@github-actions github-actions bot added the category: GHA CI based on Github actions label Aug 5, 2025
@Copilot Copilot AI review requested due to automatic review settings August 15, 2025 09:37
@github-actions github-actions bot added the category: GGUF GGUF file reader label Aug 15, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR migrates the macOS CI workflow from Intel-based mac-13 runners to ARM64-based mac-14 runners to support Apple Silicon architecture.

  • Updates all job runners from macos-13 to macos-14
  • Changes target architecture from x86_64 to arm64 throughout the workflow
  • Disables certain test suites that are not supported on ARM64 without SVE support

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
.github/workflows/mac.yml Complete migration from mac-13 x86_64 to mac-14 arm64 runners with disabled unsupported tests
tests/python_tests/test_gguf_reader.py Adds xfail marker for ARM-based platforms to skip failing test

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@@ -274,6 +275,7 @@ jobs:

- name: Build Tokenizers Wheel
run: |
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(nproc) - 1 ))
Copy link
Preview

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The nproc command is not available on macOS. Use sysctl -n hw.ncpu instead to get the number of CPU cores on macOS.

Suggested change
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(nproc) - 1 ))
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(sysctl -n hw.ncpu) - 1 ))

Copilot uses AI. Check for mistakes.

@@ -282,13 +284,16 @@ jobs:

- name: Build GenAI Wheel
run: |
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(nproc) - 1 ))
Copy link
Preview

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The nproc command is not available on macOS. Use sysctl -n hw.ncpu instead to get the number of CPU cores on macOS.

Suggested change
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(nproc) - 1 ))
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(sysctl -n hw.ncpu) - 1 ))

Copilot uses AI. Check for mistakes.

python -m pip wheel -v --no-deps --wheel-dir ${{ env.WHEELS_DIR }} \
${{ needs.openvino_download.outputs.ov_wheel_source }} \
${{ env.SRC_DIR }}
working-directory: ${{ env.OV_INSTALL_DIR }}

- name: Build WWB Wheel
run: python -m pip wheel -v --no-deps --wheel-dir ${{ env.WHEELS_DIR }} ${{ env.SRC_DIR }}/tools/who_what_benchmark
run: |
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(nproc) - 1 ))
Copy link
Preview

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The nproc command is not available on macOS. Use sysctl -n hw.ncpu instead to get the number of CPU cores on macOS.

Suggested change
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(nproc) - 1 ))
export CMAKE_BUILD_PARALLEL_LEVEL=$(( $(sysctl -n hw.ncpu) - 1 ))

Copilot uses AI. Check for mistakes.

@@ -405,7 +410,7 @@ jobs:
-DENABLE_JS=ON -DCPACK_GENERATOR=NPM \
-DENABLE_PYTHON=OFF -DENABLE_WHEEL=OFF \
-S ${{ env.SRC_DIR }} -B ${{ env.BUILD_DIR }}
cmake --build ${{ env.BUILD_DIR }} --config ${{ matrix.build-type }} --parallel --verbose
cmake --build ${{ env.BUILD_DIR }} --config ${{ matrix.build-type }} --parallel $(nproc) --verbose
Copy link
Preview

Copilot AI Aug 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The nproc command is not available on macOS. Use sysctl -n hw.ncpu instead to get the number of CPU cores on macOS.

Suggested change
cmake --build ${{ env.BUILD_DIR }} --config ${{ matrix.build-type }} --parallel $(nproc) --verbose
cmake --build ${{ env.BUILD_DIR }} --config ${{ matrix.build-type }} --parallel $(sysctl -n hw.ncpu) --verbose

Copilot uses AI. Check for mistakes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants