-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Closed
Labels
area/docsarea/simulationcomplexity/lowintroduces/modifies 1-2 concepts, should take 1-2 days max for an advanced contributorintroduces/modifies 1-2 concepts, should take 1-2 days max for an advanced contributorskill-level/beginnerNo background required, self-contained issueNo background required, self-contained issuetriage/acceptedA consensus emerged that this bug report, feature request, or other action should be worked onA consensus emerged that this bug report, feature request, or other action should be worked on
Description
The signature of compute_amplitudes_sweep and compute_amplitudes requires bitstrings to be a list of int's. Because of the semantics in python3 of not allowing leading zeros for int's (https://stackoverflow.com/questions/36386346/syntaxerror-invalid-token) this can be confusing.
For eg: if I want to measure a bitstring 0010, I need to provide 10 as input to the bitstring which is confusing to the user.
I propose to change the type of bitstrings to Sequence[str].
Metadata
Metadata
Assignees
Labels
area/docsarea/simulationcomplexity/lowintroduces/modifies 1-2 concepts, should take 1-2 days max for an advanced contributorintroduces/modifies 1-2 concepts, should take 1-2 days max for an advanced contributorskill-level/beginnerNo background required, self-contained issueNo background required, self-contained issuetriage/acceptedA consensus emerged that this bug report, feature request, or other action should be worked onA consensus emerged that this bug report, feature request, or other action should be worked on