Skip to content

MAINT: Add to_dict/from_dict roundtrip serialization to model classes#1738

Merged
rlundeen2 merged 9 commits into
microsoft:mainfrom
rlundeen2:users/rlundeen/model_dict
May 20, 2026
Merged

MAINT: Add to_dict/from_dict roundtrip serialization to model classes#1738
rlundeen2 merged 9 commits into
microsoft:mainfrom
rlundeen2:users/rlundeen/model_dict

Conversation

@rlundeen2
Copy link
Copy Markdown
Contributor

@rlundeen2 rlundeen2 commented May 15, 2026

The output modules in the thin CLI needs access to REST data from models because it can't access memory. This allows us to deserialize and serialize common modules

rlundeen2 and others added 4 commits May 15, 2026 10:53
Comment thread pyrit/models/attack_result.py
Comment thread tests/unit/models/test_attack_result.py
Comment thread pyrit/models/attack_result.py Outdated
Comment thread pyrit/models/message.py
Comment thread pyrit/models/score.py Outdated
Comment thread pyrit/models/message.py
Comment thread pyrit/models/message_piece.py
rlundeen2 and others added 2 commits May 19, 2026 13:13
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Comment thread pyrit/models/score.py Outdated
Comment thread pyrit/models/scenario_result.py Outdated
Comment thread pyrit/memory/memory_models.py Outdated
rlundeen2 and others added 2 commits May 19, 2026 15:52
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…pletion_time)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@rlundeen2 rlundeen2 force-pushed the users/rlundeen/model_dict branch from 79810bc to 9b36ec1 Compare May 20, 2026 00:10
@rlundeen2 rlundeen2 enabled auto-merge May 20, 2026 00:12
@rlundeen2 rlundeen2 added this pull request to the merge queue May 20, 2026
Merged via the queue into microsoft:main with commit d00b98c May 20, 2026
48 checks passed
@rlundeen2 rlundeen2 deleted the users/rlundeen/model_dict branch May 20, 2026 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants