Migrate PyRIT to open source repository#1
Merged
Conversation
Test Results69 tests 69 ✅ 11s ⏱️ Results for commit 683c561. ♻️ This comment has been updated with latest results. |
romanlutz
force-pushed
the
romanlutz/migration
branch
from
January 10, 2024 09:05
6292a61 to
13baff6
Compare
mshirsekar1
pushed a commit
to mshirsekar1/PyRIT
that referenced
this pull request
Sep 18, 2024
FEAT add ability to fetch PKU-SafeRLHF dataset
Merged
ValbuenaVC
pushed a commit
to ValbuenaVC/PyRIT
that referenced
this pull request
May 21, 2026
…ure modes) Closes the R5.1 rubber-duck follow-ups on top of R5 (ScenarioPipeline): - Add Scenario._finalize_scenario_result_async base hook (no-op default) called once between the last successful step and the COMPLETED state transition, giving composition subclasses a place to write run-summary state into ScenarioResult.metadata. - Override the hook on ScenarioPipeline to persist per-phase outcomes as metadata['phase_executions'] (a list of name/outcome/inner_scenario_result_id dicts), so cross-process readers can reload the pipeline result and walk phases without holding a live pipeline instance. Class docstring updated to reflect the new persistence contract. - Invert metadata merge order in _build_phase_action: pipeline-stamped diagnostic keys (step_name, phase_index) now win over inner-step result metadata. Regression test pins the inversion against a NoisyStep that emits colliding keys. - Document PipelineContext immutability nuance: structurally frozen at the dataclass level, but inner ScenarioResult payloads are not deep- immutable and should be treated as read-only by convention. - Sharpen input_schema docstring on the kept-but-broken 'phases' role: explicit guidance that the OPAQUE tag is an authoring-refusal signal for the wizard until pipelines can round-trip. - Add TestPipelineFailureModes covering Duck microsoft#1's M1 gaps: inner initialize_async / run_async exceptions, predicate exceptions, and partial-progress phase_executions on mid-flight failure. 50/50 composite tests pass (was 41 at R5 ship); 1066/1066 scenario tests pass overall. Pre-commit clean (ruff format/check, ty). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
1 participant
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.
This is the initial commit to move PyRIT to this repository.