lllyasviel / lllyasviel/stable-diffusion-webui-forge
img2img: max resolution cap for Resize by
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 13k
- Forks
- 1.7k
- PR merge metrics
- No merged PRs in 30d
Description
Two related pain points in img2img resize:
**1. Resize by**
Scale only multiplies the source (e.g. 2048² × 0.9 → 1856²). There is no ceiling. In a batch some images are much larger, so the same Scale OOMs on those and is fine on the rest. You have to move resize again for every new image with a different resolution.
An optional **max side / max megapixels** (`0` = off) would clamp uniformly, keep aspect ratio, and leave Scale unchanged across mixed inputs.
**2. Resize to**
Today both width and height are independent. It would help if you could lock aspect ratio to **one** side: set width *or* height, and have the other computed from the source so the image is not stretched or arbitrarily cropped.
Thanks
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No files or tests are named. Start by locating the img2img Resize by and Resize to controls and their image-dimension handling. Done means an optional max side or megapixel cap preserves aspect ratio without changing Scale, and Resize to can derive one dimension from the other without stretching.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100