✨ Evaluate stochastic RL pass candidates - #757
Closed
flowerthrower wants to merge 24 commits into
Closed
flowerthrower wants to merge 24 commits into
flowerthrower wants to merge 24 commits into
Conversation
12 tasks
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
12 tasks
flowerthrower
force-pushed
the
v3/666-hybrid-mdp
branch
from
August 20, 2026 12:27
49f7954 to
86230a9
Compare
flowerthrower
force-pushed
the
v3/667-stochastic-actions
branch
3 times, most recently
from
August 21, 2026 15:55
cb1e427 to
fae3016
Compare
flowerthrower
force-pushed
the
v3/667-stochastic-actions
branch
from
August 27, 2026 10:29
5f9194f to
aa8e530
Compare
flowerthrower
changed the base branch from
v3/666-hybrid-mdp
to
v3/674-normalized-features
August 27, 2026 10:29
Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com>
flowerthrower
force-pushed
the
v3/667-stochastic-actions
branch
from
August 27, 2026 13:09
303bb1e to
b5e29bd
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
Assisted-by: GPT-5.6 via Codex Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com>
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
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
flowerthrower
force-pushed
the
v3/667-stochastic-actions
branch
from
August 27, 2026 13:18
b5e29bd to
3f75c42
Compare
Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com> Assisted-by: GPT 5.6 via Codex
Co-authored-by: Antonio Tudisco <anto.tu98@hotmail.it> Assisted-by: GPT-5.6 via Codex Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com>
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
Assisted-by: GPT-5.6 via Codex Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com> Assisted-by: GPT 5.6 via Codex
Assisted-by: GPT-5.6 via Codex 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>
Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com> Assisted-by: GPT 5.6 via Codex
Assisted-by: GPT-5 via Codex Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com> Assisted-by: GPT 5.6 via Codex
Assisted-by: GPT-5 via Codex Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com> Assisted-by: GPT 5.6 via Codex
Assisted-by: GPT-5.6 via Codex Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com>
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
Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com>
Signed-off-by: flowerthrower <flowerthrower@users.noreply.github.com>
flowerthrower
force-pushed
the
v3/667-stochastic-actions
branch
from
August 27, 2026 13:21
3f75c42 to
b78feea
Compare
Member
Author
|
🤖 AI text below 🤖 Superseded by #791 after reordering the stack; the implementation remains on the same head branch. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 AI text below 🤖
Description
Evaluates repeated candidates for Qiskit RL actions marked as stochastic and retains the candidate with the best configured figure of merit. Candidates are translated to the target basis before scoring; supported scoring failures fall back to minimizing inserted SWAPs.
PredictorEnv.stochastic_action_trialsconfigures the number of attempts and defaults to 20.QiskitSabreMappingand theSabreSwapaction introduced by #785 use this path. A failed candidate does not abort the episode: the remaining attempts continue, and an all-failed run leaves the circuit unchanged. Pass and scoring timeouts are re-raised so the timeout layer can truncate immediately rather than continuing unbounded trials. Set the trial count to 1 to retain single-attempt selection.This is position 9 of the stack. It depends on the timeout PR #789 and is followed by the GNN PR #788.
Validated with repository lint, focused RL checks, and direct pass- and scoring-timeout smokes.
Fixes #667
Checklist
If PR contains AI-assisted content:
🤖 *AI text below* 🤖(titles are exempt).