Skip to content

ApplyQFT operation name is misleading #1812

Open
@tcNickolas

Description

@tcNickolas

To follow up #1810 and the internal discussion on this: using ApplyQFT name for an operation that applies only a part of QFT is misleading and can lead to long and unpleasant debugging sessions. I would suggest renaming it to something like ApplyQFTRotations, which is a more accurate name, since it accurately describes that only the rotations part of the QFT is applied, and to introduce a new operation ApplyQFT that actually applies a complete QFT.

For example, I'm working on the second part of QFT kata, and every task in it includes applying either QFT or Adjoint QFT. We don't have dependencies between kata cells, and we don't have a single operation to apply QFT, so I have to include instructions regarding this in every task. I will run into the same issue when working on the QPE kata, programming assignments that include QFT and QPE, and so on.

Metadata

Metadata

Labels

libraryrelated to Q# library code rather than the compiler implementation or language design

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions