Skip to content

LCUBlockEncoding and LinearCombination have misleading names #1773

@petim0

Description

@petim0

This is a small issue, and maybe it’s just me, but I’ve lost quite some time using LCUBlockEncoding thinking that it meant a Linear Combination of Block Encodings, rather than a Linear Combination of Unitaries that is itself a block encoding.

Meanwhile, the block that actually implements a linear combination of block encodings is LinearCombination

The name is totally fair once you understand the meaning, but when you type "LCU" in the documentation, you first find LCUBlockEncoding. As a result, I was totally unaware of the other option and spend some time trying to work around a design that isn’t adapted to block encodings.

I would suggest either changing the names, or at least cross-referencing both options in the docs for LCUBlockEncoding and LinearCombination.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions