Skip to content

Commit

Permalink
Fix sinter packaging (#826)
Browse files Browse the repository at this point in the history
- Wasn't including all sub packages in the setup call
- Added a CI test that this is correct
  • Loading branch information
Strilanc authored Sep 13, 2024
1 parent fcb795b commit ef2a092
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 7 deletions.
21 changes: 15 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -183,14 +183,23 @@ jobs:
with:
name: "dist-stim-sdist"
path: dist/*.tar.gz
check_sdist_installs:
check_sdist_installs_stim:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
- run: python -m pip install pybind11~=2.11.1 cibuildwheel~=2.16.2 setuptools wheel
- run: python setup.py sdist
- run: pip install dist/*.tar.gz
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
- run: python -m pip install pybind11~=2.11.1 cibuildwheel~=2.16.2 setuptools wheel
- run: python setup.py sdist
- run: pip install dist/*.tar.gz
check_sdist_installs_sinter:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
- run: python -m pip install setuptools
- run: cd glue/sample && python setup.py sdist
- run: pip install glue/sample/dist/*
- run: python -c "import sinter"
merge_upload_artifacts:
needs: ["build_dist", "build_sdist"]
runs-on: ubuntu-latest
Expand Down
2 changes: 1 addition & 1 deletion glue/sample/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
author='Craig Gidney',
author_email='[email protected]',
license='Apache 2',
packages=['sinter'],
packages=['sinter', 'sinter._data', 'sinter._collection', 'sinter._command', 'sinter._decoding'],
package_dir={'': 'src'},
description='Samples stim circuits and decodes them using pymatching.',
long_description=long_description,
Expand Down

0 comments on commit ef2a092

Please sign in to comment.