Enable slow CI test runs and fix identity-only energy-estimator workflows#466
Open
Copilot wants to merge 21 commits into
Open
Enable slow CI test runs and fix identity-only energy-estimator workflows#466Copilot wants to merge 21 commits into
Copilot wants to merge 21 commits into
Conversation
…ia PR comment - Add runSlowTests parameter to ADO test-pip-wheels.yml template (default '1') to enable QDK_CHEMISTRY_RUN_SLOW_TESTS in nightly builds - Add workflow_dispatch input to GitHub Actions build-and-test.yaml to allow running slow tests on demand - Add slow-tests-on-demand.yaml workflow triggered by '@copilot run slow tests' PR comment that dispatches Build and Test with slow tests enabled Agent-Logs-Url: https://github.com/microsoft/qdk-chemistry/sessions/6d975f68-c9c1-4ef3-9659-3cace814bf78 Co-authored-by: lorisercole <30901257+lorisercole@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Run QDK_CHEMISTRY_RUN_SLOW_TESTS on a cadence
Enable slow tests in nightly ADO pipeline and add on-demand trigger via PR comment
May 4, 2026
lorisercole
requested changes
May 4, 2026
… wording - ADO test-pip-wheels.yml: change runSlowTests to type: boolean (default: true) - GitHub Actions build-and-test.yaml: change run_slow_tests to type: boolean - slow-tests-on-demand.yaml: accept '@copilot run slow tests', '@copilot run slow-tests', and '@copilot run tests' as triggers Agent-Logs-Url: https://github.com/microsoft/qdk-chemistry/sessions/ddb44b52-9955-4412-90c3-ec4df55c4b48 Co-authored-by: lorisercole <30901257+lorisercole@users.noreply.github.com>
lorisercole
reviewed
May 4, 2026
📊 Coverage Summary
Detailed Coverage ReportsC++ Coverage DetailsPython Coverage DetailsPybind11 Coverage Details |
Contributor
|
@copilot address the failing pre-commit test |
Agent-Logs-Url: https://github.com/microsoft/qdk-chemistry/sessions/735e4af4-62d7-45ba-8bf1-e78633122aa9 Co-authored-by: lorisercole <30901257+lorisercole@users.noreply.github.com>
Contributor
Author
Contributor
|
@copilot write and update the description of this PR. |
Contributor
Author
Updated the PR description with a summary of all changes across the three files. |
Contributor
|
@copilot run slow tests |
Contributor
|
Currently blocked by the |
Contributor
|
Triggered slow test https://github.com/microsoft/qdk-chemistry/actions/runs/26537531209 Slow test on local |
lorisercole
changed the title
Collaborator
|
Triggered another slow test run https://github.com/microsoft/qdk-chemistry/actions/runs/26539539328 |
lorisercole
approved these changes
May 28, 2026
Contributor
Contributor
Author
Done in fa8aa08. |
Contributor
|
@copilot rename this PR considering all the changes/commits |
Contributor
Author
I can't change the PR title directly from here, but a title that matches the current scope is: |
Contributor
|
GHA slow tests pass.
EDIT: apparently these were transient problems. At 2nd attempt the pipelines completed. |
Comment on lines
+361
to
+365
| if _is_identity_only(hamiltonian.pauli_strings): | ||
| identity_offset += float(np.real(np.sum(hamiltonian.coefficients))) | ||
| Logger.debug( | ||
| f"Dropping identity-only group from measurement circuits. Energy offset += {identity_offset:.4f}" | ||
| ) |
Comment on lines
+246
to
+250
| ), MeasurementData( | ||
| hamiltonians=qubit_hamiltonians, | ||
| bitstring_counts=[], | ||
| shots_list=[], | ||
| ) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Wires up the
QDK_CHEMISTRY_RUN_SLOW_TESTSenvironment variable so that slow Python tests can be enabled on-demand or in nightly CI runs, and fixes the QDK energy estimator path for identity-only measurement groups.Changes
ADO nightly pipeline (
.pipelines/python-wheels.yaml,.pipelines/templates/test-pip-wheels.yml)runSlowTestsboolean parameter (default:true) to the main ADOpython-wheels.yamlpipeline.runSlowTeststhrough all wheel-test template invocations.runSlowTestsboolean parameter (default:true) to the wheel-test template.QDK_CHEMISTRY_RUN_SLOW_TESTSinto all Docker and native test execution environments, including the linux/amd64 Docker path.GitHub Actions (
build-and-test.yaml)workflow_dispatchboolean inputrun_slow_tests(default:false).QDK_CHEMISTRY_RUN_SLOW_TESTSfor the Python test step ('1'when enabled,'0'otherwise).Energy estimator, examples, and tests (
python/src/qdk_chemistry/algorithms/energy_estimator/qdk.py,docs/source/_static/examples/python/quickstart.py,examples/*.ipynb,python/tests/*.py)