-
Notifications
You must be signed in to change notification settings - Fork 58
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
added plot_probabilities method in result class #82
added plot_probabilities method in result class #82
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #82 +/- ##
==========================================
- Coverage 92.84% 89.48% -3.36%
==========================================
Files 60 63 +3
Lines 7629 8150 +521
==========================================
+ Hits 7083 7293 +210
- Misses 546 857 +311
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Ok! Besides the potential issue regarding the Pauli op error, I think this PR is ready to be merged! (cc @vishal-ph ) |
I see, there are still some changes requested by Shahidee. Are those already implemented? If yes, lets resolve the comments and we can proceed with merge |
You man regarding the use of plt.show(), right? With my version of Matplotlib it is not necessary. I am unsure what version is Shahidee using |
Description
plot_probabilities
method under the result classChecklist
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Via UnitTest in the test_result_object.py file. Both for 'vectored' and 'qiskit.shot_simulator'.