Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugs encountered when using Open in SAM Detector #283

Closed
wangsvip opened this issue Nov 15, 2023 · 2 comments
Closed

Bugs encountered when using Open in SAM Detector #283

wangsvip opened this issue Nov 15, 2023 · 2 comments

Comments

@wangsvip
Copy link

After initially using Open in SAM Detector to remove excess content, and then using Open in MaskEditor to remove other content, I found that the limited content was not completely removed. When I clicked Open in MaskEditor again, I found that it had read the original image. Could it be that Open in MaskEditor can only be used once?

1. Original picture

640

2. First click Open in SAM Detector

image

3.Remove unnecessary background content

image
image

4.Click Open in MaskEditor for the first time to remove other content

image
image

5.When I found that some content had not been deleted, I clicked Open in MaskEditor again.

image
image

6.I found that it read the original image. Can this function only be used once? Does it have to be done all at once?

image

7.In addition, I also found 2 bugs. I cannot make punctuation in the red area. There is also the Undo button. Sometimes you can click once to undo a punctuation. Sometimes you need to click 2 or 3 times to undo the last punctuation. This bug is reproducible

image

8.After this problem occurs, you can only close comfyui completely and restart the application to make annotations again. I hope this bug will be fixed as soon as possible.

@ltdrdata
Copy link
Owner

I recently discovered that the mask editor has been malfunctioning. Nevertheless, I am in the process of planning a PR to make changes to the mask editor.

@ltdrdata
Copy link
Owner

ltdrdata commented Dec 7, 2023

Mask editor issue is fixed on latest ComfyUI. Update ComfyUI.

comfyanonymous/ComfyUI#2171

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

No branches or pull requests

2 participants