-
Notifications
You must be signed in to change notification settings - Fork 55
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
refactor QuantumNetwork
to support general quantum networks
#1244
base: master
Are you sure you want to change the base?
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
The build of the |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1244 +/- ##
========================================
Coverage 99.81% 99.82%
========================================
Files 72 72
Lines 10520 10659 +139
========================================
+ Hits 10501 10640 +139
Misses 19 19
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Thanks @Canoming. All the methods seem to be implemented correctly.
Regarding the documentation, I have noticed that the new classes QuantumComb
and QuantumChannel
need to be added to qibo.srt so that they appear on the website. The file README.md is not included in any toctree. I think the best place to add it is in advancedexamples.rst.
I only have a few minor suggestions at the code level:
@Canoming I'm sorry for my absence. I can't review this PR this week but I can next week. |
Hi, @AlejandroSopena , I've updated the code according to your comments. The |
no cover Co-authored-by: Alejandro Sopena <44305203+AlejandroSopena@users.noreply.github.com>
for more information, see https://pre-commit.ci
Checklist:
I've refactored the
QuantumNetworks
to support general quantum networks and make it more compatible with usual tensor contraction methods.To make things clear, I made a class diagram for the future development of the module