Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test workflow for legacy opmath #5435

Merged
merged 67 commits into from
Apr 17, 2024
Merged
Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
67 commits
Select commit Hold shift + click to select a range
5f2528f
add use_new_opmath variable
lillian542 Mar 22, 2024
3e1905e
[ci skip]
lillian542 Mar 22, 2024
27352df
add initial legacy_opmath workflow
lillian542 Mar 22, 2024
11fea82
Merge branch 'master' into test_legacy_opmath
mudit2812 Mar 22, 2024
da16e5b
Merge branch 'master' into test_legacy_opmath
lillian542 Apr 3, 2024
a85f990
trigger ci
lillian542 Apr 3, 2024
60f61a7
for science
lillian542 Apr 3, 2024
da9a5d8
see if this triggers full ci
lillian542 Apr 3, 2024
f97f295
or does this make ci run?
lillian542 Apr 3, 2024
3699cf7
will ci still run?
lillian542 Apr 3, 2024
639f411
undo previous test change
lillian542 Apr 3, 2024
4d2657d
try to add line explicitly setting __use_new_opmath
lillian542 Apr 3, 2024
8ef1ac8
fix syntax for two line run statement
lillian542 Apr 3, 2024
cbaef13
temporarily run on pull request
lillian542 Apr 3, 2024
742d548
don't run on pull requests
lillian542 Apr 3, 2024
c823f14
also add variable to interface-unit-tests
lillian542 Apr 3, 2024
9a30cd5
try this
lillian542 Apr 3, 2024
52248a9
run on PR
lillian542 Apr 3, 2024
8cbd9e2
uncomment use_new_opmath kwarg
lillian542 Apr 3, 2024
8188231
sanity check that tests are running as expected
lillian542 Apr 3, 2024
b62c727
set use_new_opmath to false for all
lillian542 Apr 3, 2024
97f9a7b
temporary test to sanity check testing behaviour
lillian542 Apr 4, 2024
0ef2a04
update legacy workflow to run on this branch instead of master for now
lillian542 Apr 4, 2024
e19db17
add pytest fixture and input variable to optionally disable op_math
lillian542 Apr 4, 2024
a73f324
update to use the disable-opmath fixture
lillian542 Apr 4, 2024
04b6402
remove old commented out code
lillian542 Apr 4, 2024
f37925a
use and evaluate string instead of using yaml boolean
lillian542 Apr 4, 2024
2dd4620
update value in legacy_op_math workflow
lillian542 Apr 4, 2024
e2f37e1
pass value for disable_new_opmath through to unit-test.yml
lillian542 Apr 4, 2024
2571b1b
use quotations when passing variable
lillian542 Apr 4, 2024
668591c
add option to devices conftest too
lillian542 Apr 4, 2024
ab595f9
remove temporary sanity-check tests
lillian542 Apr 4, 2024
e1716e2
Legacy Hamiltonian converts Prods to Tensors
lillian542 Apr 5, 2024
668938a
change handling of disable-opmath input
lillian542 Apr 5, 2024
d3feda0
make disable-opmath input variable type consistent
lillian542 Apr 5, 2024
649c534
update failing tests
lillian542 Apr 5, 2024
bcc882d
Update tests/transforms/test_qcut.py
lillian542 Apr 5, 2024
f8e2e0e
test_autograd
Qottmann Apr 8, 2024
9eec37f
black formatting
Qottmann Apr 8, 2024
244ee08
update qutrit device test
lillian542 Apr 11, 2024
b103d23
run CI with lightning master instead of stable
lillian542 Apr 11, 2024
3828844
undo lightning thing
lillian542 Apr 12, 2024
cee0ad1
Merge branch 'master' into test_legacy_opmath
lillian542 Apr 12, 2024
1722b46
use lightning master again
lillian542 Apr 12, 2024
ec5d379
add codecov token
lillian542 Apr 12, 2024
b0d8c1e
Merge branch 'test_legacy_opmath' of github.com:PennyLaneAI/pennylane…
lillian542 Apr 12, 2024
17ed5ff
put external libraries back to lightning stable
lillian542 Apr 12, 2024
9301d22
make autograd test compatible with both
lillian542 Apr 12, 2024
837aee5
temporarily comment out test
lillian542 Apr 12, 2024
0ee82e7
update tests
lillian542 Apr 12, 2024
d31585e
update based on code review
lillian542 Apr 12, 2024
0ffda5d
add test for lines added to hamiltonian
lillian542 Apr 12, 2024
b50bf15
mark new opmath version as xfail
lillian542 Apr 12, 2024
1999627
Merge branch 'master' into test_legacy_opmath
lillian542 Apr 12, 2024
da3bfb7
update tests
lillian542 Apr 16, 2024
d9508ac
revert changes to hamiltonian
lillian542 Apr 16, 2024
994f834
revert added hamiltonian test
lillian542 Apr 16, 2024
37ddf42
update qcut tests
lillian542 Apr 16, 2024
42442b6
update fermi tests
lillian542 Apr 16, 2024
6977fff
update vqe tests
lillian542 Apr 16, 2024
0212bc6
Merge branch 'master' into test_legacy_opmath
lillian542 Apr 16, 2024
e6c59f6
update default_qubit test
lillian542 Apr 16, 2024
1f4e1d7
update device tests
lillian542 Apr 16, 2024
8eff78e
Merge branch 'master' into test_legacy_opmath
lillian542 Apr 16, 2024
13defd5
trigger ci
lillian542 Apr 16, 2024
f18265c
Update workflow to test on master and not run on every PR
lillian542 Apr 17, 2024
89e7d15
Merge branch 'master' into test_legacy_opmath
lillian542 Apr 17, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions .github/workflows/legacy_op_math.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
name: Legacy opmath tests

on:
schedule:
- cron: "0 0 2 * *"
workflow_dispatch:

jobs:
tests:
uses: ./.github/workflows/interface-unit-tests.yml
with:
branch: 'master'
run_lightened_ci: false
use_new_opmath: false
lillian542 marked this conversation as resolved.
Show resolved Hide resolved
6 changes: 6 additions & 0 deletions .github/workflows/unit-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,11 @@ on:
required: false
type: string
default: ''
use_new_opmath:
description: Whether to use the new op_math or not when running the tests
required: false
type: bool
default: true
lillian542 marked this conversation as resolved.
Show resolved Hide resolved

jobs:
test:
Expand Down Expand Up @@ -169,6 +174,7 @@ jobs:
COV_CORE_CONFIG: .coveragerc
COV_CORE_DATAFILE: .coverage.eager
# Calling PyTest by invoking Python first as that adds the current directory to sys.path
run: python -m import pennylane as qml && qml.operation.__use_new_opmath = ${{ inputs.use_new_opmath }}
lillian542 marked this conversation as resolved.
Show resolved Hide resolved
run: python -m pytest ${{ inputs.pytest_test_directory }} ${{ steps.pytest_args.outputs.args }} ${{ env.PYTEST_MARKER }}

- name: Adjust coverage file for Codecov
Expand Down
Loading