Skip to content

Fix for #10643 (Inpainting mask sometimes not working)#10644

Merged
AUTOMATIC1111 merged 1 commit into
AUTOMATIC1111:devfrom
ArthurHeitmann:fix-inpainting-canvas-noise
May 22, 2023
Merged

Fix for #10643 (Inpainting mask sometimes not working)#10644
AUTOMATIC1111 merged 1 commit into
AUTOMATIC1111:devfrom
ArthurHeitmann:fix-inpainting-canvas-noise

Conversation

@ArthurHeitmann

Copy link
Copy Markdown
Contributor

Describe what this pull request is trying to achieve.

A fix for #10643. Short summary: In the chrome webui the HTML canvas for painting the inpainting mask has some noise (on my machine :) ). The backend treats any non 0 pixel as white. As a result the entire mask map turns white and the entire image is changed. This PR fixes that.

Additional notes and description of your changes

The fix converts the initial painted mask to a pure black and white image, with no gradients. If there should be a need for a grayscale mask, this won't work. This doesn't affect the gaussian blur that is applied later on in the process.

Environment this was tested in

  • OS: Windows
  • Browser: chrome
  • Graphics card: NVIDIA RTX 4090 24GB

Screenshots or videos of your changes

N/A

…reaking inpainting, so that it behaves like plain img2img)
@AUTOMATIC1111

Copy link
Copy Markdown
Owner

This doesn't happen on my machine neither in Firefox nor in Chrome. But the change looks reasonable and harmless so I would merge it. The question is, does this happen to a lot of people? If yes, apart from putting it into dev I'd also include it as a bugfix to RC.

@ArthurHeitmann

ArthurHeitmann commented May 22, 2023

Copy link
Copy Markdown
Contributor Author

I've searched for other people having the same problem and I've only found one or two people. So it seems to be quite rare.

@AUTOMATIC1111 AUTOMATIC1111 merged commit 80a723c into AUTOMATIC1111:dev May 22, 2023
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.

2 participants