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

Update Safety Checker model for reduceSum with int32 input for Stable Diffusion 1.5 #24

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

ibelem
Copy link
Contributor

@ibelem ibelem commented Aug 2, 2024

We updated Safety Checker model for reduceSum with int32 input for SD Turbo in #11, but forgot to update the model for Stable Diffusion 1.5. @philloooo reported the issue.

@Adele101 Could you please help to copy safety_checker_int32_reduceSum.onnx of https://huggingface.co/microsoft/sd-turbo-webnn/tree/main/safety_checker and upload it with same file name into https://huggingface.co/microsoft/stable-diffusion-v1.5-webnn/tree/main , thanks!

@fdwr Please help to review and merge this PR when @Adele101 uploaded the int32 SC model for Stable Diffusion 1.5. Thanks!

@fdwr
Copy link
Collaborator

fdwr commented Aug 2, 2024

Change looks good. Adele is OOF today to upload the new safety checker model. So will merge Monday after she uploads it. Thanks for updating.

@Adele101 Adele101 merged commit 4445eac into microsoft:main Aug 5, 2024
1 check passed
@fdwr
Copy link
Collaborator

fdwr commented Aug 8, 2024

I'm reverting this for now until this CR finishes https://huggingface.co/microsoft/stable-diffusion-v1.5-webnn/discussions/1, because otherwise it breaks the sample.

@fdwr
Copy link
Collaborator

fdwr commented Aug 13, 2024

Restored again: #29

@ibelem ibelem deleted the safety-checker-sd-1.5 branch September 25, 2024 07:36
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.

3 participants