Feat new gate - #191
Feat new gate#191MathieuG-Colibri wants to merge 133 commits into
Conversation
…aket_observable and submit_job_braket functions
…into feat-New-gate
…into feat-New-gate
…hout verbatim box
…into feat-New-gate
| try: | ||
| value = getattr(self, attr_name) | ||
| except NotImplementedError: | ||
| continue |
There was a problem hiding this comment.
Ummm .... quand est-ce qu'on aurait besoin de ça ?
There was a problem hiding this comment.
Dans certains cas, on a pas implémenté la traduction de la porte, genre pour cirq, et on renvoie explicitement NotImplementedError, c'est pour catch ça (oui c'est moche)
There was a problem hiding this comment.
Et pourquoi on laisse pas l'erreur remonter du coup ? Si la porte est pas traduite, autant laisser la pipeline casser que silencieusement catch une erreur ?
There was a problem hiding this comment.
C'est pas la traduction c'est le to_dict, c'est pour éviter de récupérer des attributs pas implémentées dans le dico, je me suis mal exprimé dans le commentaires au dessus, c'est que les cirq gates c'est le premier exemple qui m'est venu.
Dans cet exemple, ça lèverait une erreur de traduction sur le to_dict qui ne sert pas à traduire, donc pour moi faut catch cette erreur, vu qu'elle ne concerne pas cette fonction
Co-authored-by: hJaffaliColibritd <133856040+hJaffaliColibritd@users.noreply.github.com>
|
|
6 participants
Added native gates PRX and Rzz for IQM computers.
Also added the ComposedGate interface for future more complex gates.
Fixed adjust_measure for non continuous and partial observables.
Changed the overall translation from mpqp to Braket ==> stop padding the braket circuit with Id gates and accepting non continuous circuits. For the sake of more optimized hardware runs.
Also refactoring and future proofing.