-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Batch circuit submit in cirq_ionq #6652
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6652 +/- ##
========================================
Coverage 97.84% 97.85%
========================================
Files 1081 1081
Lines 93015 93371 +356
========================================
+ Hits 91011 91365 +354
- Misses 2004 2006 +2 ☔ View full report in Codecov by Sentry. |
@pavoljuhas Thanks for merging this. I fixed the merge issues but the CI issues indicated a bug in implementation, so I am still working on fixing it and extending unit tests. |
@pavoljuhas The CI issues were fixed. A bug was fixed, tests have been updated. This ticket is ready for review. |
No change in the effective code.
Let pylint check for `unexpected-keyword-arg`.
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.
Thank you for contributing this!
Enable batch circuit job submit in cirq_ionq