Skip to content

Fix web UI compatibility of generation seeds - #739

Merged
ShiftHackZ merged 1 commit into
developfrom
fix/webui-compatible-random-seeds
Nov 14, 2026
Merged

ShiftHackZ merged 1 commit into
developfrom
fix/webui-compatible-random-seeds

Conversation

@ShiftHackZ

Copy link
Copy Markdown
Owner

Description

Addresses #569: random seed and variation-seed dice previously generated values too large to copy accurately into AUTOMATIC1111 web UI numeric fields.

  • Match AUTOMATIC1111 random generation: 0–4,294,967,293 inclusive.
  • Validate manual A1111/SwarmUI seeds against JavaScript’s safe integer range, with inline feedback and generation submission checks. Empty input and -1 still request randomness; other seed controls retain their Long range.
  • Add a SeedValidator interface with a separate internal implementation, constructor-configured limits and Koin bindings for both generation screens.
  • Synchronize validation messages across all six locales.

Validation

  • 56 tests pass: 38 seed regression tests and 18 existing sampler tests.
  • Android full debug assembly and iOS Simulator compilation pass.
  • Pixel 3a XL, Android 12: checked both seed fields in txt2img/img2img, manual boundary and oversized input, malformed input, and clear/dice recovery.
  • Actual A1111 image reproduction and iOS runtime behavior remain untested.

Checklist

  • I have ensured this PR does not contain any breaking changes for existing functionality
  • I have added automated tests that cover the new behavior and removed obsolete tests and code
  • I have added documentation for any new public APIs and update existing documentation for any changes to public APIs

Tested On

  • Android: Emulator
  • Android: Real device
  • iOS: Simulator
  • iOS: Real iPhone device

@ShiftHackZ ShiftHackZ linked an issue Nov 14, 2026 that may be closed by this pull request
@ShiftHackZ
ShiftHackZ merged commit 8d144f0 into develop Nov 14, 2026
1 check passed
@ShiftHackZ
ShiftHackZ deleted the fix/webui-compatible-random-seeds branch November 14, 2026 20:04
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.

Random seeds are too large to replicate in A1111 webui

1 participant