-
Notifications
You must be signed in to change notification settings - Fork 8
Move build system to meson-python
#166
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
Open
ndgrigorian
wants to merge
65
commits into
main
Choose a base branch
from
build-with-meson
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
8f9de4d
move mkl_umath build system to meson
ndgrigorian e1ce0ee
add tests to module
ndgrigorian c1b5c43
exclude process_src_template from linting
ndgrigorian 01af689
add python-build to meta.yamls
ndgrigorian c90f88f
fix syntax in build shell scripts
ndgrigorian e734712
update build_pip workflow
ndgrigorian 075b296
update clang workflow
ndgrigorian 2ec1201
fix import error in builds
ndgrigorian 0ae4685
update Windows build scripts
ndgrigorian c4a4038
fix linting
ndgrigorian ab07fd1
fix flags on windows builds
ndgrigorian 903cf97
add wheel to conda-forge meta.yaml
ndgrigorian 8b58419
fix typo in bld.bat
ndgrigorian d745054
add pip to meta.yamls
ndgrigorian 64c1881
ignore git blame of previous commit fixing linting
ndgrigorian d10f0d0
fix race condition in meson build
ndgrigorian 2bd6b7a
require MKL in meson build
ndgrigorian f4d0b02
drop 'cpp' from meson.build
ndgrigorian cb7f656
add cmake to build system dependencies
ndgrigorian 8e3cd1d
remove jq from pip build
ndgrigorian 9a03cfe
add meson option for setting MKL_THREADING
ndgrigorian 6dc291f
simplify mkl_umath conda-forge build scripts
ndgrigorian f7e5989
Merge branch 'main' into build-with-meson
ndgrigorian ff65831
add CLI sources to meson.build
ndgrigorian 1166ac3
Use link_args for RPATH and improve portability
vlad-perevezentsev c66efbb
Remove redundant meson dependency
vlad-perevezentsev 2c06350
Apply remarks from mkl-service PR
vlad-perevezentsev 7e4cbb6
Update AGENT.md and copilot files
vlad-perevezentsev c8f87b4
Update README
vlad-perevezentsev d56a482
fix RPATH and remove duplicate vectorization flags
vlad-perevezentsev 97273ce
Merge remote-tracking branch 'origin/main' into build-with-meson
vlad-perevezentsev 2142b86
Merge main into build-with-meson
vlad-perevezentsev 3144a1e
Remove macOS RPATH support
vlad-perevezentsev 2792ce6
Add numpy to ignore_run_exports
vlad-perevezentsev 36fb2fd
Add wheels upload to conda-package workflow
vlad-perevezentsev d3058d7
Use -ex flags in build.sh
vlad-perevezentsev da30b1a
Fix --platform-tag flag in conda-recipe build.sh
vlad-perevezentsev 0a5872d
Remove explicit CC/CXX from conda-recipe-cf build.sh
vlad-perevezentsev b0128c9
Remove explicit CC/CXX in conda-recipe build.sh
vlad-perevezentsev 3200edd
Switch to editable install in workflows
vlad-perevezentsev 6dbc649
Add intel-openmp dependency and numpy to ignore_run_exports
vlad-perevezentsev af4e9c1
Add gen_umath_c to _ufuncs sources in meson.build
vlad-perevezentsev 8e50ba1
Use main instead of master in GH workflows
vlad-perevezentsev 26b26ec
Rename build_pip.yaml to build_pip.yml
vlad-perevezentsev 3225694
Update the license name in pyproject.toml
vlad-perevezentsev 59e88b8
Remove dead branch from generate_umath.py and make -o required
vlad-perevezentsev f50938c
Update changelog
vlad-perevezentsev 3ffb970
Update README and AGENTS docs for meson build system
vlad-perevezentsev 0dc17a7
Update changelog
vlad-perevezentsev e1c0a63
Merge remote-tracking branch 'origin/main' into build-with-meson
vlad-perevezentsev 7973a93
Mention mkl_threading arg in changelog
vlad-perevezentsev cb9ed81
Fix opt-report flag and Windows export macro
vlad-perevezentsev b61cff8
Require meson-python >=0.16.0 for PEP 639
vlad-perevezentsev 46442a3
Update AI files
vlad-perevezentsev 9cb241e
Add Windows intel-openmp and intel-cmplr-lib-rt deps
vlad-perevezentsev 487fbf8
Restore warning flags on MSVC
vlad-perevezentsev 4e5183a
Remove dangling entry from .git-blame-ignore-revs
vlad-perevezentsev da6a925
Remove redundant pylint disable in test_basic.py
vlad-perevezentsev 1291b05
Enable verbose build in conda-recipe-cf build.sh
vlad-perevezentsev 283d50e
Remove redundant CFLAGS/CXXFLAGS from conda_build_config.yaml
vlad-perevezentsev 27c8f8a
Restore CC=icx in conda build scripts to build with icx
vlad-perevezentsev f065e8a
Add mkl-devel to build-system requires
vlad-perevezentsev 7657322
Update README
vlad-perevezentsev 1a7e2c3
Merge remote-tracking branch 'origin/main' into build-with-meson
vlad-perevezentsev cb77d95
Merge remote-tracking branch 'origin/main' into build-with-meson
vlad-perevezentsev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,25 +1,16 @@ | ||
| # AGENTS.md — .github/ | ||
|
|
||
| CI/CD workflows, automation, security scanning, and package distribution. | ||
| CI/CD workflows and repo automation. | ||
|
|
||
| ## Workflows | ||
| - **conda-package.yml** — main build/test pipeline (Linux/Windows, Python 3.10-3.14) | ||
| - **build_pip.yaml** — PyPI wheel builds for Intel channel | ||
| - **build-with-clang.yml** — Clang compatibility validation | ||
| - **openssf-scorecard.yml** — security posture scanning | ||
| ## Workflows (source of truth) | ||
| - `conda-package.yml` — Intel channel conda build/test pipeline | ||
| - `conda-package-cf.yml` — conda-forge-oriented build/test pipeline | ||
| - `build-with-clang.yml` — Intel clang compatibility checks | ||
| - `build-with-standard-clang.yml` — standard clang compatibility checks | ||
| - `build_pip.yml` — pip build pipeline with pre-release NumPy | ||
| - `pre-commit.yml` — lint/format checks | ||
| - `openssf-scorecard.yml` — security scanning | ||
|
|
||
| ## CI/CD policy | ||
| - Keep build matrix (Python versions, platforms) in workflow files only | ||
| - Required checks: conda build + test on all supported Python versions | ||
| - Artifact naming: `$PACKAGE_NAME $OS Python $VERSION` | ||
| - Conda channel: `https://software.repos.intel.com/python/conda` | ||
|
|
||
| ## Security | ||
| - OpenSSF Scorecard runs automatically | ||
| - CODEOWNERS enforces review policy | ||
| - Dependabot monitors dependencies (`.github/dependabot.yml`) | ||
|
|
||
| ## Notes | ||
| - Workflow/job renames are breaking for downstream tooling | ||
| - Cache key includes `meta.yaml` hash for conda packages | ||
| - Artifacts uploaded per-Python-version for parallel testing | ||
| ## Policy | ||
| - Treat workflow YAML as canonical for platform/Python matrices. | ||
| - Avoid doc claims about CI coverage unless present in workflow config. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,60 @@ | ||
| name: Build project with standard clang compiler | ||
|
|
||
| on: | ||
| pull_request: | ||
| push: | ||
| branches: [main] | ||
|
|
||
| permissions: read-all | ||
|
|
||
| jobs: | ||
| build-with-standard-clang: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| strategy: | ||
| matrix: | ||
| python: ["3.10", "3.11", "3.12", "3.13", "3.14"] | ||
| numpy_version: ["'numpy>=2'"] | ||
|
|
||
| env: | ||
| COMPILER_ROOT: /usr/bin | ||
|
|
||
| defaults: | ||
| run: | ||
| shell: bash -el {0} | ||
|
|
||
| steps: | ||
| - name: Cancel Previous Runs | ||
| uses: styfle/cancel-workflow-action@3155a141048f8f89c06b4cdae32e7853e97536bc # 0.13.0 | ||
| with: | ||
| access_token: ${{ github.token }} | ||
|
|
||
| - name: Install Dependencies | ||
| run: | | ||
| sudo apt-get update && sudo apt-get install -y clang | ||
| - name: Setup Python | ||
| uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0 | ||
| with: | ||
| python-version: ${{ matrix.python }} | ||
| architecture: x64 | ||
|
|
||
| - name: Checkout repo | ||
| uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Install mkl_umath dependencies | ||
| run: | | ||
| pip install meson-python ninja cmake cython mkl-devel | ||
| pip install ${{ matrix.numpy_version }} | ||
| - name: Build mkl_umath | ||
| run: | | ||
| export CC=${{ env.COMPILER_ROOT }}/clang | ||
| pip install -e . --no-build-isolation --no-deps --verbose | ||
| - name: Run mkl_umath tests | ||
| run: | | ||
| pip install pytest | ||
| python -m pytest -sv --pyargs mkl_umath |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.