-
Notifications
You must be signed in to change notification settings - Fork 15
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
Remove transpiler-related tutorial #763
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #763 +/- ##
=======================================
Coverage 62.54% 62.54%
=======================================
Files 47 47
Lines 5882 5882
=======================================
Hits 3679 3679
Misses 2203 2203
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thanks @alecandido. I agree with removing, I would just prefer to open a PR copying this in Qibo before merging, so that it does not get lost in the abyss. Unless it is already there. I can have a look and do it. |
I had a look in Qibo and there is already an example about how to modify the transpiler, so no need to move anything there. However, the tutorial here also talks about how to modify the compiler and plug it to the @alecandido I can do it pushing to this branch, if you agree. |
e80bb21
to
f26a070
Compare
Sorry: I actually already had a look yesterday, so I knew it (I could have written in the OP).
If you wish, that's fine. Just change the name of the file and title with the substitution |
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.
@alecandido I did some modifications so that the tutorial focuses on the compiler that is still in qibolab. If you agree, feel free to merge.
eb3c401
to
c1596a3
Compare
The transpiler has been moved to Qibo a while ago, so I'd consider this a leftover. In
case there is something to be moved before dropping here (or right after) let's do it
quickly, or otherwise postpone it until it will be relevant (i.e. let's not block the
update of Qibolab behind the update of Qibo, since having two outdated docs is worse
than a single one).