Skip to content

enrich executor_expectation_values input options#170

Merged
dekelmeirom merged 6 commits into
mainfrom
obs_and_bases_input
Apr 29, 2026
Merged

enrich executor_expectation_values input options#170
dekelmeirom merged 6 commits into
mainfrom
obs_and_bases_input

Conversation

@dekelmeirom
Copy link
Copy Markdown
Collaborator

@dekelmeirom dekelmeirom commented Apr 20, 2026

add option to give (obs, bases) as input for executor_expectation_values

in addition, add tests for measurement_bases.py

addresses issue #163

add option to give (obs, bases) as input for executor_expectation_values
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 20, 2026

Coverage Report for CI Build 25098794055

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage increased (+3.4%) to 93.685%

Details

  • Coverage increased (+3.4%) from the base build.
  • Patch coverage: 1 uncovered change across 1 file (73 of 74 lines covered, 98.65%).
  • 3 coverage regressions across 1 file.

Uncovered Changes

File Changed Covered %
qiskit_addon_utils/exp_vals/expectation_values.py 71 70 98.59%

Coverage Regressions

3 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
qiskit_addon_utils/noise_management/trex_factors.py 3 25.0%

Coverage Stats

Coverage Status
Relevant Lines: 871
Covered Lines: 816
Line Coverage: 93.69%
Coverage Strength: 0.94 hits per line

💛 - Coveralls

Comment thread qiskit_addon_utils/exp_vals/expectation_values.py Outdated
Comment thread qiskit_addon_utils/exp_vals/expectation_values.py Outdated
Comment thread qiskit_addon_utils/exp_vals/measurement_bases.py Outdated
Comment thread qiskit_addon_utils/exp_vals/measurement_bases.py Outdated
Comment thread qiskit_addon_utils/exp_vals/expectation_values.py
Comment thread qiskit_addon_utils/exp_vals/expectation_values.py Outdated
caleb-johnson and others added 3 commits April 23, 2026 10:59
* Docstring suggestions

* cleanup docstring

* typo
in addition, move the function to `expectation_values.py` file and fix documentation and formating
Comment thread qiskit_addon_utils/exp_vals/measurement_bases.py
Copy link
Copy Markdown
Collaborator

@caleb-johnson caleb-johnson left a comment

Choose a reason for hiding this comment

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

This LGTM, just one small suggestion on docstring

Copy link
Copy Markdown
Collaborator

@caleb-johnson caleb-johnson left a comment

Choose a reason for hiding this comment

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

Could we include a release note with this as well?

Just reno new basis-format or whatever ... then just fill out the Features section

@dekelmeirom dekelmeirom merged commit b72c373 into main Apr 29, 2026
20 of 21 checks passed
@dekelmeirom dekelmeirom deleted the obs_and_bases_input branch April 29, 2026 08:47
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.

3 participants