microsoft / webnn-developer-preview

MIT License
38 stars 7 forks source link

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

Closed ibelem closed 1 month ago

ibelem commented 1 month ago

We updated Safety Checker model for reduceSum with int32 input for SD Turbo in https://github.com/microsoft/webnn-developer-preview/pull/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 commented 1 month ago

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.

fdwr commented 1 month ago

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 commented 1 month ago

Restored again: https://github.com/microsoft/webnn-developer-preview/pull/29