-
Notifications
You must be signed in to change notification settings - Fork 3
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
[QHC-833] Make optimization of Drag
gates
#863
base: solving-transpilation
Are you sure you want to change the base?
[QHC-833] Make optimization of Drag
gates
#863
Conversation
AI AnalysisReview:The addition of TODO comments in the code to implement bunching of Drag gates and deletion of Drag gates without amplitude is a good practice for future development. It shows that there is a plan to improve the codebase further and maintain its efficiency. Make sure to revisit these TODOs to implement the functionality at a later stage.
|
circuit_optimizer.py
Drag
gates
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## solving-transpilation #863 +/- ##
=========================================================
- Coverage 96.78% 96.77% -0.01%
=========================================================
Files 221 221
Lines 7765 7811 +46
=========================================================
+ Hits 7515 7559 +44
- Misses 250 252 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
TODOs in this PR:
bunch_drag_gates(...)
)delete_gates_with_no_amplitude(...)
)And next PRs:
Solve more buggs about final layout/compute it yourself? 🔥🔥🔥
Improve readability of layout 🔥🔥🔥: