-
Notifications
You must be signed in to change notification settings - Fork 45
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
Add ECR gate to mappers #60
Add ECR gate to mappers #60
Conversation
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.
Sweet! Thank you for this addition @dexter2206 !
I think we need to fix linter (put commas at the end of the lists, dicts) and we are good to merge!
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.
I think linter is not happy about missing commas at ends of lists :)
Easy fix will be to run black black qiskit_braket_provider
Ah, sorry, I forgot to run black. Fixed in 9fb5c2a :) |
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.
Sweet! Thank you for contribution 🚀
Summary
This adds ECR gate to mappers in the
adapters.py
module. Without this addition, running circuits containing the ECR gate through any of the AWS backends is impossible.Details and comments
No unit tests were added since there are already tests checking all the mappers. There are also no specific tests for other gates, so I wanted to stay consistent.