-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Renamings in circuit/library/data_encoding #4318
Conversation
what about |
I'm fine with |
I'm good with ZFeatureMap and ZZFeatureMap. You are correct for some reason i was thinking i could make it X and XX but since we use the H at the start only Z makes sense. |
qiskit/circuit/library/__init__.py
Outdated
RY | ||
RYRZ | ||
RYRZAnsatz | ||
SwapRZ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only one of these we are going to call Ansatz?
…-terra into data-preparation-renaming
…-terra into data-preparation-renaming
…-terra into data-preparation-renaming
…-terra into data-preparation-renaming
* renamings * remove 'evolution' and ryrz -> ryrzansatz * more renaming, missing: SwapRZ * put ry/ryrz naming on hold * rename n local circuits * fix test * try to fix sphinx * fix spacing?? Co-authored-by: Jay Gambetta <jay.gambetta@us.ibm.com> Co-authored-by: Kevin Krsulich <kevin.krsulich@ibm.com>
Summary
Rename the module
qiskit/circuit/library/data_encoding
todata_preparation
as this is more expressive. Also rename the contents to more descriptive names.Details and comments
Suggestions (can be changed):
Reasons:
pauli_evolution
would be nice because it is short, but these circuits do also the repetitions, entanglements and H-layers on top of the evolutionsZ
ZZ
andZ