Skip to content

Update for pip install#21

Merged
ggeorgakoudis merged 1 commit into
mainfrom
update-for-pip-install
Sep 13, 2025
Merged

Update for pip install#21
ggeorgakoudis merged 1 commit into
mainfrom
update-for-pip-install

Conversation

@ggeorgakoudis

@ggeorgakoudis ggeorgakoudis commented Sep 7, 2025 •
edited
Loading

Copy link
Copy Markdown

Closes #11
Closes #22

@ggeorgakoudis ggeorgakoudis mentioned this pull request Sep 10, 2025
Open
DrTodd13
DrTodd13 approved these changes Sep 12, 2025
View reviewed changes

@DrTodd13 DrTodd13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

Refactor project for PyPI distribution and wheel packaging
0d51ffd 
[pyomp]
- Move to a src layout
- Remove dependency on LLVM binaries in pyomp
- Add versioning
- Try-except without error for CUDALegalization (removed in recent numba) for
  CUDA lowering
- Remove deprecated compile_isolated usage and update tests
- Remove static libnrt building to avoid numba version dependency and use
  trampolines to link numba symbols to openmp target cpu modules
- Add hello world examples
- Update README

[ci]
- Create pypi workflow
- Add test pipelines for multiple platforms, numba, and python versions
- Simplify conda pipeline

[openmp libraries]
- Build openmp libraries through setup.py using LLVM static libraries from
  conda env
- Add patches for missing includes and link statically with LLVM libs

[pyomp pass]
- Export entry point function to interface with python for running the pass
- Link pass plugin with LLVM static libraries from conda env

[packaging]
- Extend range of supported numba (0.57-0.60) and python versions (3.9-3.12)
- Make target offloading mandatory in tests
- Remove building the static library in setup.py

[packaging.wheel]
- Add cibuildwheel scripts
- Add MANIFEST.in
- Add license files
- Upload pre-release to testpypi, full release to pyp
- Build and publish sdist

[packaging.conda]
- Remove llvm-openmp-dev recipe
- Add missing deps in meta.yaml
- Upload as a separate command to detect errors
- Use "main" tag for full release for conda packages, "test" for pre-release,
  "dev" for PR

[container]
- Update dockerfile to avoid TOS issues
- Add pyomp's version to the container
@ggeorgakoudis
ggeorgakoudis merged commit 934ba5a into main Sep 13, 2025
184 checks passed
@ggeorgakoudis
ggeorgakoudis deleted the update-for-pip-install branch September 13, 2025 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DrTodd13 DrTodd13 DrTodd13 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CUDALegalization is removed in numba 0.58.x Cannot install PyOMP with pip

2 participants

@ggeorgakoudis @DrTodd13