Skip to content

feat: Added cost_confidence API on jobs, hugr and qir#312

Merged
qfreewilll merged 1 commit intomainfrom
add-cost-confidence-api
Feb 12, 2026
Merged

feat: Added cost_confidence API on jobs, hugr and qir#312
qfreewilll merged 1 commit intomainfrom
add-cost-confidence-api

Conversation

@qfreewilll
Copy link
Collaborator

@qfreewilll qfreewilll commented Feb 12, 2026

Output of the integration tests. The new warnings for the old cost() endpoints are visible for qir and hugr. There was no test for jobs.cost() and I didn't add one.

uv run pytest -vsn 0 -ra --reruns=0 integration/test_jobs.py integration/test_qir.py integration/test_qsys_jobs.py -k 'test_job_cost_confidence or test_qir_cost or test_hugr_cost or test_hugr_costing or test_costing_qir_on_NG_devices'

integration/test_qsys_jobs.py::test_hugr_costing PASSED
integration/test_qsys_jobs.py::test_hugr_cost_confidence PASSED
integration/test_qir.py::test_costing_qir_on_NG_devices PASSED
integration/test_qir.py::test_qir_cost_confidence PASSED
integration/test_jobs.py::test_job_cost_confidence
PASSED

================ warnings summary ================
integration/test_qsys_jobs.py::test_hugr_costing
  /Users/will.angenent/qnexus/qnexus/client/hugr.py:291: DeprecationWarning: hugr.cost() is deprecated. Please update to use hugr.cost_confidence() instead.
    warnings.warn(

integration/test_qsys_jobs.py: 4 warnings
integration/test_qir.py: 4 warnings
integration/test_jobs.py: 2 warnings
  /Users/will.angenent/qnexus/.venv/lib/python3.12/site-packages/quantinuum_schemas/models/backend_config.py:262: DeprecationWarning: QuantinuumConfig is deprecated for submission to Helios systems and support will be removed in a future release. Please use HeliosConfig instead.
    warnings.warn(

integration/test_qir.py::test_costing_qir_on_NG_devices
  /Users/will.angenent/qnexus/qnexus/client/qir.py:270: DeprecationWarning: qir.cost() is deprecated. Please update to use qir.cost_confidence() instead.
    warnings.warn(

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=========================================================================================== 5 passed, 23 deselected, 12 warnings in 28.01s ===========================================================================================

@qfreewilll qfreewilll force-pushed the add-cost-confidence-api branch from 886f2a9 to 583c7cc Compare February 12, 2026 13:48
Copy link
Collaborator

@vanyae-cqc vanyae-cqc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Shared some thoughts but let me know if anything doesn't make sense

@qfreewilll qfreewilll force-pushed the add-cost-confidence-api branch from 583c7cc to 41c2b89 Compare February 12, 2026 14:19
@qfreewilll qfreewilll force-pushed the add-cost-confidence-api branch from 41c2b89 to 12ff5c6 Compare February 12, 2026 16:09
@qfreewilll qfreewilll merged commit 0ce29a7 into main Feb 12, 2026
7 checks passed
@qfreewilll qfreewilll deleted the add-cost-confidence-api branch February 12, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants