Fix composite transform reproducibility#813
Conversation
There was a problem hiding this comment.
Verified the fix: Python's set iteration order varies across runs due to hash randomization (PYTHONHASHSEED), making CompositeTransformation nondeterministic. The ordered deduplication preserves insertion order while still removing duplicates, ensuring reproducible results.
Tested with PYTHONHASHSEED=random — set ordering differs on every run, confirming this is a real reproducibility issue.
|
Yes, PR #813 is needed. It fixes a real bug. The current code uses a The PR replaces the |
What does this PR do?
Summary
Fix unwanted non-determinism text order in
CompositeTransformation. Not using set for text makes reproducible runs possible.Additions
Changes
Deletions
Checklist
.rstfile inTextAttack/docs/apidoc.'