Skip to content

Commit

Permalink
Skip oqd pytests in build-wheels (#1390)
Browse files Browse the repository at this point in the history
**Context:**
A big redesign of oqd ion dialect and the associated toml file format
took place during #1260 . Therefore, the frontend oqd pytests (which
performs some toml parsing) should be removed, since (a) they are
interpreting against the old toml format, and (b) the toml parsing now
happens in the mlir layer alongside the actual `--quantum-to-ion` pass,
not at the frontend python layer as a standalone.
  • Loading branch information
paul0403 authored Dec 18, 2024
1 parent c88192a commit 99820e8
Show file tree
Hide file tree
Showing 10 changed files with 0 additions and 692 deletions.
1 change: 0 additions & 1 deletion .github/workflows/build-wheel-linux-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -494,7 +494,6 @@ jobs:
python${{ matrix.python_version }} -m pytest frontend/test/async_tests
# python${{ matrix.python_version }} -m pytest frontend/test/pytest --runbraket=LOCAL -n auto
python${{ matrix.python_version }} -m pytest frontend/test/test_oqc/oqc -n auto
python${{ matrix.python_version }} -m pytest frontend/test/test_oqd/oqd -n auto
- name: Run Standalone Plugin Tests
# Run only on Thursday at the given time
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/build-wheel-macos-arm64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -475,7 +475,6 @@ jobs:
python${{ matrix.python_version }} -m pytest frontend/test/async_tests
# python${{ matrix.python_version }} -m pytest frontend/test/pytest --runbraket=LOCAL -n auto
python${{ matrix.python_version }} -m pytest frontend/test/test_oqc/oqc -n auto
python${{ matrix.python_version }} -m pytest frontend/test/test_oqd/oqd -n auto
- name: Run Standalone Plugin Tests
# Run only on Thursday at the given time
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/build-wheel-macos-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -447,7 +447,6 @@ jobs:
python${{ matrix.python_version }} -m pytest frontend/test/async_tests
# python${{ matrix.python_version }} -m pytest frontend/test/pytest --runbraket=LOCAL -n auto
python${{ matrix.python_version }} -m pytest frontend/test/test_oqc/oqc -n auto
python${{ matrix.python_version }} -m pytest frontend/test/test_oqd/oqd -n auto
- name: Run Standalone Plugin Tests
# Run only on Thursday at the given time
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/scripts/linux_arm64/rh8/test_wheels.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,4 +47,3 @@ export PATH=/catalyst/llvm-build/bin:/opt/_internal/cpython-${PYTHON_MAJOR_MINOR
/usr/bin/python3 -m pytest /catalyst/frontend/test/async_tests
# /usr/bin/python3 -m pytest -v /catalyst/frontend/test/pytest --runbraket=LOCAL -n auto
/usr/bin/python3 -m pytest /catalyst/frontend/test/test_oqc/oqc -n auto
/usr/bin/python3 -m pytest /catalyst/frontend/test/test_oqd/oqd -n auto
3 changes: 0 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ COVERAGE_REPORT ?= term-missing
ENABLE_OPENQASM?=ON
TEST_BACKEND ?= "lightning.qubit"
TEST_BRAKET ?= NONE
SKIP_OQD ?= true
ENABLE_ASAN ?= OFF
TOML_SPECS ?= $(shell find ./runtime ./frontend -name '*.toml' -not -name 'pyproject.toml')

Expand Down Expand Up @@ -168,7 +167,6 @@ endif
@echo "check the Catalyst PyTest suite"
$(ASAN_COMMAND) $(PYTHON) -m pytest frontend/test/pytest --tb=native --backend=$(TEST_BACKEND) --runbraket=$(TEST_BRAKET) $(PARALLELIZE)
$(ASAN_COMMAND) $(PYTHON) -m pytest frontend/test/test_oqc/oqc
$(ASAN_COMMAND) $(PYTHON) -m pytest frontend/test/test_oqd/oqd --skip-oqd=$(SKIP_OQD) $(PARALLELIZE)
ifeq ($(TEST_BRAKET), NONE)
$(ASAN_COMMAND) $(PYTHON) -m pytest frontend/test/async_tests --tb=native --backend=$(TEST_BACKEND)
endif
Expand Down Expand Up @@ -276,7 +274,6 @@ coverage-frontend:
@echo "Generating coverage report for the frontend"
$(ASAN_COMMAND) $(PYTHON) -m pytest frontend/test/pytest $(PARALLELIZE) --cov=catalyst --tb=native --cov-report=$(COVERAGE_REPORT)
$(ASAN_COMMAND) $(PYTHON) -m pytest frontend/test/test_oqc/oqc $(PARALLELIZE) --cov=catalyst --cov-append --tb=native --cov-report=$(COVERAGE_REPORT)
$(ASAN_COMMAND) $(PYTHON) -m pytest frontend/test/test_oqd/oqd --skip-oqd=$(SKIP_OQD) $(PARALLELIZE) --cov=catalyst --cov-append --tb=native --cov-report=$(COVERAGE_REPORT)
ifeq ($(TEST_BRAKET), NONE)
$(ASAN_COMMAND) $(PYTHON) -m pytest frontend/test/async_tests --tb=native --backend=$(TEST_BACKEND) --tb=native
endif
Expand Down
38 changes: 0 additions & 38 deletions frontend/test/test_oqd/conftest.py

This file was deleted.

17 changes: 0 additions & 17 deletions frontend/test/test_oqd/oqd/oqd_beam_parameters.toml

This file was deleted.

67 changes: 0 additions & 67 deletions frontend/test/test_oqd/oqd/oqd_qubit_parameters.toml

This file was deleted.

Loading

0 comments on commit 99820e8

Please sign in to comment.