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 BindingsArray as allowed input type to BindingsArray.coerce() #11645

Merged

Conversation

ihincks
Copy link
Contributor

@ihincks ihincks commented Jan 25, 2024

Summary

This PR allows the BindingsArray.coerce() method to pass-through an input of type BindingsArray.

Details and comments

This PR does not check sequence-like inputs in the tests because that conflicts with #11642. At the very least, we need to see if that PR gets approved.

@ihincks ihincks added the mod: primitives Related to the Primitives module label Jan 25, 2024
@ihincks ihincks requested review from a team as code owners January 25, 2024 20:00
@qiskit-bot
Copy link
Collaborator

One or more of the the following people are requested to review this:

  • @Qiskit/terra-core
  • @ajavadia
  • @ikkoham
  • @levbishop
  • @t-imamichi

@coveralls
Copy link

Pull Request Test Coverage Report for Build 7659798891

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 89.561%

Totals Coverage Status
Change from base Build 7659269267: 0.04%
Covered Lines: 59593
Relevant Lines: 66539

💛 - Coveralls

@ikkoham ikkoham added the Changelog: None Do not include in changelog label Jan 26, 2024
Copy link
Contributor

@ikkoham ikkoham left a comment

Choose a reason for hiding this comment

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

Thank you for finding and fixing the bug. This feature is not released yet, so change log is none.

@t-imamichi t-imamichi added this pull request to the merge queue Jan 26, 2024
Merged via the queue into Qiskit:main with commit ce02f06 Jan 26, 2024
12 checks passed
@ihincks ihincks deleted the primitives/fix-bindings_array-coerce branch January 26, 2024 13:32
wshanks pushed a commit to wshanks/qiskit-terra that referenced this pull request Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: None Do not include in changelog mod: primitives Related to the Primitives module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants