-
Notifications
You must be signed in to change notification settings - Fork 989
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
Use device instead of gateset validation in ZerosSampler #3142
Conversation
all gates in the circuit are from the given gate set. | ||
""" | ||
self.gate_set = gate_set | ||
|
||
def run_sweep( |
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.
I am fine with getting rid of validation altogether, but one alternative I just thought of was to add an optional device parameter. That could also validate the circuit in a more natural way.
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.
OK added device validation. Could also just do this by adding it to a circuit.
…3142) This introduces a dependency of cirq on cirq.google, which we want to avoid. If we want a gate set validating version of this we should put it in cirq.google.
This introduces a dependency of cirq on cirq.google, which we want to avoid. If we want a gate set validating version of this we should put it in cirq.google.