-
Notifications
You must be signed in to change notification settings - Fork 159
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
Deprecate circuit_indices and observable_indices and add circuits and observables #331
Deprecate circuit_indices and observable_indices and add circuits and observables #331
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
…l method which is called by the __call__ in base class
5034627
to
571ccf4
Compare
Pull Request Test Coverage Report for Build 2429957947
💛 - Coveralls |
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.
This looks good from a correctness perspective. We should update the tutorials to include examples that pass the objects and not just indices.
👍 Updated in 837368e |
Summary
Deprecate circuit_indices and observable_indices and add circuits and observables.
circuits and observables can take actual circuits and observables as well as indices.
Details and comments
Fixes #315
TODO: