✨ Add missing TKET RL actions - #796
flowerthrower wants to merge 2 commits into
Conversation
da0271e to
629877d
Compare
8e325e1 to
bb3ce81
Compare
1ce6b4f to
65c9d98
Compare
|
@coderabbitai full review |
|
|
Warning Review limit reachedNext included review available in 50 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
bb3ce81 to
384893d
Compare
65c9d98 to
7c4a965
Compare
384893d to
86e9632
Compare
7c4a965 to
3853b60
Compare
86e9632 to
10b43ad
Compare
3853b60 to
2ba8e5a
Compare
10b43ad to
2afc79b
Compare
2ba8e5a to
7126f5c
Compare
Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com> Assisted-by: GPT 5.6 via Codex
Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com> Assisted-by: GPT 5.6 via Codex
7126f5c to
eba24fd
Compare
🤖 AI text below 🤖
Description
Adds the TKET actions used by the paper prototype but missing from the production registry:
KAKDecomposition,GraphPlacement, andNoiseAwarePlacement.The placement actions translate TKET placement maps into complete Qiskit layouts through the existing layout pipeline.
NoiseAwarePlacementreceives mean one- and two-qubit gate errors per node and link, while measurement errors are excluded from those averages and supplied separately as readout errors. Registered TKET layout actions must provide exactly onePlacement.TKET layout and routing actions are masked for circuits containing operations wider than two qubits.
Focused tests cover calibration aggregation, wide-operation masking, real graph and noise-aware placement, KAK semantics, and the public placement boundary. Adjacent layout and registration tests and the lint suite pass.
Existing RL models must be retrained because the action schema changes.
This is position 5 of the stack. It depends on #795 and is followed by #797. No new package dependencies are introduced.
Part of #664
Checklist
If PR contains AI-assisted content:
🤖 *AI text below* 🤖(titles are exempt).