feat(python): Build & publish wheels for Windows (#351)

* restore initial stab at building windows wheels

* test1

* use fixed pip command

* try cmake setup action

* try --no-index flag

* fix path

* oops, wrong path

* remove extraneous build step

* update pyo3 to 0.19.2, rigetti-pyo3 to 0.2.0

* remove test code from workflow

* rename argument to inner

* update maturin to ^1.0

Author: MarquessV

.github/workflows/release-python.yml

@@ -6,7 +6,6 @@ on:
   workflow_dispatch:
     description: "Manually publish release"
 
-
 jobs:
   is-python-release:
     if: ${{ github.event_name == 'workflow_dispatch' || startsWith(github.event.release.tag_name, 'python/v') }}
@@ -43,7 +42,7 @@ jobs:
         # universal2 supports both x86_64 and aarch64
         uses: messense/maturin-action@v1
         with:
-          args: -i python --release --universal2 --manifest-path crates/python/Cargo.toml --out dist
+          args: -i python --release --target universal2-apple-darwin --manifest-path crates/python/Cargo.toml --out dist
       - name: Install wheel
         run: |
           pip install qcs-sdk-python --find-links dist --force-reinstall
@@ -90,10 +89,45 @@ jobs:
         name: wheels
         path: dist
 
+  windows: 
+    runs-on: windows-latest 
+    strategy: 
+      matrix: 
+        python-version: ['3.8', '3.9', '3.10', '3.11'] 
+        target: [x64, x86] 
+    steps: 
+      - uses: actions/checkout@v2 
+      - uses: actions/setup-python@v2 
+        with: 
+          python-version: ${{ matrix.python-version }} 
+          architecture: ${{ matrix.target }} 
+      - name: Update rustup 
+        run: rustup self update 
+      - name: Install Rust toolchain 
+        uses: actions-rs/toolchain@v1 
+        with: 
+          toolchain: stable 
+          target: x86_64-pc-windows-msvc 
+          profile: minimal 
+          default: true 
+      - name: Build wheels 
+        uses: messense/maturin-action@v1 
+        with: 
+          target: ${{ matrix.target }} 
+          args: -i python --release --manifest-path crates/python/Cargo.toml --out dist 
+      - name: Install built wheel 
+        run: | 
+          pip install qcs-sdk-python --find-links dist --force-reinstall --no-deps --no-index
+      - name: Upload wheels 
+        uses: actions/upload-artifact@v2 
+        with: 
+          name: wheels 
+          path: dist 
+
   release:
     name: Release
     runs-on: ubuntu-latest
-    needs: [ macos, linux ]
+    needs: [ macos, linux, windows ]
     steps:
       - uses: actions/download-artifact@v3
       - name: Publish to PyPi

Cargo.lock

@@ -15,17 +15,17 @@ tokio = "1.24.2"
 # be a `quil-py` tag and should match the version used in `crates/python/pyproject.toml`
 # The version must also be specified in order to publish to crates.io. Cargo enforces
 # that the specified version is the same as the version in the git repository.
-quil-rs = { version = "0.21.0", git = "https://github.com/rigetti/quil-rs", tag = "quil-py/v0.5.0" }
+quil-rs = { version = "0.21.0", git = "https://github.com/rigetti/quil-rs", tag = "quil-py/v0.5.1" }
 
 # ndarray is used by the `qcs` crate, but it is also used in the `python` crate via a
 # re-export through the numpy crate. They should be updated as a pair to keep both
 # crates' versions of ndarray in sync.
 # Similarly, pyo3 packages (`numpy`, `rigetti-pyo3`, `pyo3*`) track versions together
 # and need to be updated together.
 ndarray = { version = "0.15.6", features = ["serde"] }
-numpy = "0.17"
-pyo3 = { version = "0.17" }
-pyo3-asyncio = { version = "0.17", features = ["tokio-runtime"] }
-pyo3-build-config = { version = "0.17" }
-rigetti-pyo3 = { version = "0.1.0", features = ["complex"] }
+numpy = "0.19"
+pyo3 = "0.19.2"
+pyo3-asyncio = { version = "0.19", features = ["tokio-runtime"] }
+pyo3-build-config = "0.19.2"
+rigetti-pyo3 = { version = "0.2.0", features = ["complex"] }
 

Cargo.toml

@@ -20,7 +20,7 @@ classifiers = [
   "Programming Language :: Python :: 3.11",
   "Operating System :: OS Independent",
 ]
-dependencies = ["quil==0.5.0"]
+dependencies = ["quil==0.5.1"]
 
 # PEP 621 specifies the [project] table as the source for project metadata. However, Poetry only supports [tool.poetry]
 # We can remove this table once this issue is resolved: https://github.com/python-poetry/poetry/issues/3332
@@ -45,17 +45,17 @@ sdist-include = ["README.md"]
 [tool.poetry.dependencies]
 python = "^3.8"
 
-[tool.poetry.dev-dependencies]
+[tool.poetry.group.dev.dependencies]
 numpy = "^1.24.1"
-maturin = "^0.14.17"
+maturin = "^1.2.3"
 pytest = "^7.1.3"
 pytest-asyncio = "^0.19.0"
 black = "^22.8.0"
 syrupy = "^3.0.6"
 mypy = "^1.4.1"
 
 [build-system]
-requires = ["maturin>=0.14,<0.15"]
+requires = ["maturin>=1.0.0,<2.0.0"]
 build-backend = "maturin"
 
 [tool.black]