-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: expand box types and match order #15
Conversation
johnchildren
commented
Sep 8, 2023
- Expand box types to support more (but not all) of the current possible pytket box types.
- Re-order the OpType variants to match those of the boxes.
- Expand box types to support more (but not all) of the current possible pytket box types. - Re-order the OpType variants to match those of the boxes.
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.
Nice!
/// NYI | ||
CliffBox, | ||
|
||
/// See \ref CustomGate | ||
CustomGate, |
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.
Is this not needed anymore?
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.
As far as I can tell CustomGate is the same as Composite I believe.
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.
50788ab Fixed here
The left member of the tuple should be a Register.
## 🤖 New release * `tket-json-rs`: 0.1.1 -> 0.2.0 (⚠️ API breaking changes) ###⚠️ `tket-json-rs` breaking changes ``` --- failure enum_struct_variant_field_added: pub enum struct variant field added --- Description: An enum's exhaustive struct variant has a new field, which has to be included when constructing or matching on this variant. ref: https://doc.rust-lang.org/reference/attributes/type_system.html#the-non_exhaustive-attribute impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.24.0/src/lints/enum_struct_variant_field_added.ron Failed in: field cx_config of variant OpBox::PauliExpBox in /tmp/.tmpycvRG8/tket-json-rs/src/circuit_json.rs:109 field control_state of variant OpBox::QControlBox in /tmp/.tmpycvRG8/tket-json-rs/src/circuit_json.rs:166 ``` <details><summary><i><b>Changelog</b></i></summary><p> <blockquote> ## 0.2.0 (2023-10-16) ### Features - Expand box types and match order ([#15](#15)) </blockquote> </p></details> --- This PR was generated with [release-plz](https://github.com/MarcoIeni/release-plz/). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>