Skip to content

Commit

Permalink
Exclude OQD packages, modules and data from wheels (#1403)
Browse files Browse the repository at this point in the history
**Context:** A number of OQD packages, modules and data files had been
added recently (e.g. #1348 and #1355). However, as support for OQD
devices is still under development, these files should not be included
in the wheels.

**Description of the Change:** Excludes _all_ OQD packages, modules and
data files (such as toml config files) from the wheels. Furthermore, the
OQD device entry in the [Supported devices
documentation](https://docs.pennylane.ai/projects/catalyst/en/stable/dev/devices.html)
has also been removed.
  • Loading branch information
joeycarter authored Jan 3, 2025
1 parent 44ad2e7 commit b741618
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 18 deletions.
1 change: 0 additions & 1 deletion MANIFEST.in
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,3 @@ recursive-include frontend/catalyst/lib *
recursive-include frontend/catalyst/enzyme *
recursive-include frontend/mlir_quantum *
recursive-include frontend/catalyst/third_party/cuda/ *.toml
recursive-include frontend/catalyst/third_party/oqd/ *.toml
2 changes: 0 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -194,8 +194,6 @@ wheel:
cp $(OQC_BUILD_DIR)/librtd_oqc* $(MK_DIR)/frontend/catalyst/lib
cp $(OQC_BUILD_DIR)/oqc_python_module.so $(MK_DIR)/frontend/catalyst/lib
cp $(OQC_BUILD_DIR)/backend/*.toml $(MK_DIR)/frontend/catalyst/lib/backend
cp $(OQD_BUILD_DIR)/librtd_oqd* $(MK_DIR)/frontend/catalyst/lib
cp $(OQD_BUILD_DIR)/backend/*.toml $(MK_DIR)/frontend/catalyst/lib/backend
cp $(COPY_FLAGS) $(LLVM_BUILD_DIR)/lib/libmlir_float16_utils.* $(MK_DIR)/frontend/catalyst/lib
cp $(COPY_FLAGS) $(LLVM_BUILD_DIR)/lib/libmlir_c_runner_utils.* $(MK_DIR)/frontend/catalyst/lib
cp $(COPY_FLAGS) $(LLVM_BUILD_DIR)/lib/libmlir_async_runtime.* $(MK_DIR)/frontend/catalyst/lib
Expand Down
13 changes: 0 additions & 13 deletions doc/dev/devices.rst
Original file line number Diff line number Diff line change
Expand Up @@ -95,16 +95,3 @@ Supported backend devices include:
See the `Catalyst configuration file <https://github.com/PennyLaneAI/catalyst/blob/main/frontend/catalyst/third_party/oqc/src/oqc.toml>`__
for natively supported instructions.

* - ``oqd.default``

- Experimental support for execution on `Open Quantum Design (OQD) <https://openquantumdesign.org/>`__
trapped-ion hardware. To use OQD with Catalyst, use the ``backend`` argument to specify the
OQD backend to use when initializing the device:

.. code-block:: python
dev = qml.device("oqd", backend="default", shots=1024, wires=2)
See the `Catalyst configuration file <https://github.com/PennyLaneAI/catalyst/blob/main/frontend/catalyst/third_party/oqd/src/oqd.toml>`__
for natively supported instructions.
4 changes: 4 additions & 0 deletions doc/releases/changelog-dev.md
Original file line number Diff line number Diff line change
Expand Up @@ -168,6 +168,10 @@

* A default backend for OQD trapped-ion quantum devices has been added.
[(#1355)](https://github.com/PennyLaneAI/catalyst/pull/1355)
[(#1403)](https://github.com/PennyLaneAI/catalyst/pull/1355)

Support for OQD devices is still under development, therefore the OQD modules are currently not
included in the distributed wheels.

* `expval` and `var` operations no longer keep the static shots attribute, as a step towards supporting dynamic shots across catalyst.
[(#1360)](https://github.com/PennyLaneAI/catalyst/pull/1360)
Expand Down
7 changes: 5 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@
entry_points = {
"pennylane.plugins": [
"oqc.cloud = catalyst.third_party.oqc:OQCDevice",
"oqd = catalyst.third_party.oqd:OQDDevice",
"softwareq.qpp = catalyst.third_party.cuda:SoftwareQQPP",
"nvidia.custatevec = catalyst.third_party.cuda:NvidiaCuStateVec",
"nvidia.cutensornet = catalyst.third_party.cuda:NvidiaCuTensorNet",
Expand Down Expand Up @@ -331,7 +330,11 @@ def run(self):
packages=find_namespace_packages(
where="frontend",
include=["catalyst", "catalyst.*", "mlir_quantum"],
exclude=["catalyst.third_party.oqc.*"],
exclude=[
"catalyst.third_party.oqc.*",
"catalyst.third_party.oqd.*",
"catalyst.third_party.oqd", # Exclude OQD from wheels as it is still under development
],
),
package_dir={"": "frontend"},
include_package_data=True,
Expand Down

0 comments on commit b741618

Please sign in to comment.