Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

QuantumPhaseEstimation takes BigEndian argument #244

Open
@crazy4pi314

Description

@crazy4pi314

Describe the bug
I think this is just an inconsistency, most everything uses LittleEndian and the controlRegister argument takes a BigEndian.

Expected behavior
I would expect the signature for QuantumPhaseEstimation to be

    operation QuantumPhaseEstimation (
        oracle : DiscreteOracle, 
        targetState : Qubit[], 
        controlRegister : LittleEndian
    ) 
    : Unit is Adj + Ctl{
...
}

Metadata

Metadata

Assignees

Labels

Kind-EnhancementNew feature or requestStatus-ChildIssueThis issue is part of an in-progress proposal or other parent issueStatus-NeedsApiReviewThis PR requires an API review before merging in.

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions