Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: Switch SIMD tests to meson #24625

Merged
merged 2 commits into from
Sep 7, 2023
Merged

Conversation

seiko2plus
Copy link
Member

All SIMD tests have been transitioned to utilize the Meson build system.
The Intel SDE tests are now consolidated into a single action.
Additionally, the Arm7 test has been removed as it is already
covered by Linux QEMU tests for armhf.

@seiko2plus seiko2plus added component: CI component: SIMD Issues in SIMD (fast instruction sets) code or machinery labels Sep 3, 2023
@seiko2plus seiko2plus marked this pull request as ready for review September 3, 2023 06:34
@charris
Copy link
Member

charris commented Sep 3, 2023

LGTM, but I'll wait for another review.

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

This looks great to me, thanks @seiko2plus! I only have one comment about GCC 9 installation not being successful.

.github/workflows/linux_simd.yml Outdated Show resolved Hide resolved
run: |
echo "::group::Meson Log"
cat build/meson-logs/meson-log.txt
echo "::endgroup::"
Copy link
Member

Choose a reason for hiding this comment

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

This step being included with all the others results in log browsing being a bit slow, since it's >10,000 lines as a result of meson-log.txt being so large. It's hidden by default though, and I think this is fine as a price to pay for such a concise linux_simd.yml file. So I wanted to note this, but no change needed.

@rgommers rgommers added this to the 2.0.0 release milestone Sep 4, 2023

armv7_simd_test:
Copy link
Member

Choose a reason for hiding this comment

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

For completeness: a superset of this test is now tested in linux_qemu.yml, so this removal is expected.

export NUMPY_SITE=$(realpath build-install/usr/lib/python*/site-packages/)
export PYTHONPATH="$PYTHONPATH:$NUMPY_SITE"
cd build-install &&
sde -spr -- python -c "import numpy; numpy.show_config()" &&
Copy link
Member

Choose a reason for hiding this comment

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

This was meant to show CPU info of the machine the CI job landed on (was useful to figure out the bug in SDE).

Suggested change
sde -spr -- python -c "import numpy; numpy.show_config()" &&
python -c "import numpy; numpy.show_config()" &&

Copy link
Member

Choose a reason for hiding this comment

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

The sde -spr usage doesn't seem needed to me here indeed. But it'll work either way, the output in the CI log seems fine. Does that sound right? If so, I can take this along in a follow-up (need to make more CI tweaks soon).

  The SIMD tests have been transitioned to utilize the Meson build system.
  The Intel SDE tests are now consolidated into a single action.
  Additionally, the Arm7 test has been removed as it is already
  covered by Linux QEMU tests for armhf.
Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

LGTM now, thanks @seiko2plus!

@rgommers rgommers merged commit 388778c into numpy:main Sep 7, 2023
51 checks passed
charris added a commit to charris/numpy that referenced this pull request Nov 5, 2023
charris added a commit to charris/numpy that referenced this pull request Nov 6, 2023
charris added a commit to charris/numpy that referenced this pull request Nov 6, 2023
charris added a commit to charris/numpy that referenced this pull request Nov 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: CI component: SIMD Issues in SIMD (fast instruction sets) code or machinery
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants