-
Notifications
You must be signed in to change notification settings - Fork 4
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
Convert the code to an installable package qpt-pls
#2
Comments
Thank you! It will probably be more convenient! (By the way, yout link to your notebook on state tomography failed with 404 for me). |
Hi,
Sorry for the 404, sometimes the Jupyter notebooks do not load properly.
Here is the implementation I wrote [link](
https://github.com/qutip/qutip-notebooks/blob/9fad7f609942c2813921ba12027cc53f76861d97/examples/tomography-iMLE-photon-counting.ipynb
)
The original code was part of a benchmark for different tomography methods
[link](
https://github.com/quantshah/qst-cgan/blob/main/paper-figures/fig3a-imle.ipynb).
It's from a recently accepted PRL paper on state tomography with neural
networks.
I will soon update you with the refactored code from your repository. It's
pretty nice to be able to run your code easily. However I did face some
small issues with dependencies and so decided to make your code into a
package. If the main algorithm can be written down within a single notebook
as the simplest process tomography example, we can also put it on the QuTiP
website and link to your repository. It will be very helpful for the wider
community. We have many example notebooks in the tutorials section:
https://qutip.org/tutorials.html
…On Fri, 27 Aug 2021 at 23:17, Hannoskaj ***@***.***> wrote:
Thank you! It will probably be more convenient!
(By the way, yout link to your notebook on state tomography failed with
404 for me).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVFIBHLRIFRTKPTEHRBBTLT676H3ANCNFSM5C5TRYKA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Hi, I am making a QuTiP tutorial using your code for the minimum HIP implementation: Please let me know if I need to cite your original code and paper better or if you have any other suggestions. The PR is here: |
If you are active on Twitter then please check out : https://twitter.com/quantshah/status/1448681993277952005 Thanks |
I am not on Twitter :) Thank you for your work on the tutorial! If you need help to understand any point, you can write me (I think my mail address should at least be in the article). A few comments on the state of your notebook:
By the way, do you want a better version of |
Hi @Hannoskaj thank you so much for your comments. I will try to incorporate them in the notebook. Do you mind continuing the discussion about the tutorial on the PR in QuTiP (qutip/qutip-notebooks#133) ? This way you can also be part of the review process which would be very helpful. I want to add your contact and name to the notebook as well if that is alright. After all it is your code.
I definitely would be happy to incorporate a better version of proj_CP_threshold if you want to share. Please do. |
Dear @Hannoskaj I updated the qutip notebook for a demonstration of your HIP switch algorithm here: https://github.com/qutip/qutip-notebooks/blob/241f77edd9f3f821da715959d5ee1c6eaada7f78/examples/tomography-process-tomo-pls.ipynb Would you be willing to review the code and PR: qutip/qutip-notebooks#133 It is a self-contained demonstration of your paper with the code from here. I could not for some reason get the HIP switch to work for a 5 qubit QFT process reconstruction. I am going to use your code for some more benchmarks on a current project of mine and it would be great if I get some feedback on whether I used the full power of your algorithm. |
I accidentally created PR #1 when I meant to open a discussion.
@Hannoskaj Thank you for this nice code and implementation of the algorithm. I am one of the developers of the qutip project (qutip.org) and I am interested in using your implementation of quantum process tomography.
I have been working with state tomography and have recently started exploring quantum process tomography.
I wanted to standardise the code a little bit more into a python package that can be eventually installed as simply as
pip install qpt-pls
orpython setup.py install
and used easily by others as a normal python package. I am going to develop the code a bit more in a fork of your repository under (https://github.com/quantshah/qpt-pls).I have been publishing all my code as installable python packages (e.g. qst-cgan, piqs). I would be happy to help develop this into a python package and benchmark against other process tomography methods such as CPTPprojection.
The text was updated successfully, but these errors were encountered: