-
Notifications
You must be signed in to change notification settings - Fork 17
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
Selfhosted tests #641
Selfhosted tests #641
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
Co-authored-by: Stavros Efthymiou <[email protected]>
Co-authored-by: Stavros Efthymiou <[email protected]>
Co-authored-by: Stavros Efthymiou <[email protected]>
Co-authored-by: Stavros Efthymiou <[email protected]>
Co-authored-by: Stavros Efthymiou <[email protected]>
Co-authored-by: Stavros Efthymiou <[email protected]>
Co-authored-by: Stavros Efthymiou <[email protected]>
7fd5067
to
bc2eb60
Compare
Run on QPU
|
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 @GabrielePalazzo. I updated the workflow to also upload the coverage as an artifact, similar to what we were doing before. It seems to work on dummy, so I would suggest to merge this now and check each instrument's tests later as we are refactoring the drivers.
Note that if you want to run on dummy_couplers, you need to modify the queues.json in qibolab_platforms_qrc and assign a queue to dummy_couplers. This is not really required as the only reason to run these tests on dummy is to check if the workflow itself works (for which the dummy is sufficient).
Run on QPU
|
Checklist: