| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
This PR fixes bug under is_safetensors_compatible function: when variant and safetensor file does not match, this function should return False. For example, in this model: stable-diffusion-v1-5/stable-diffusion-inpainting, there is file diffusion_pytorch_model.fp16.safetensors, but when variant is None, it should return False instead of True |
Sorry, something went wrong.
|
@a-r-r-o-w @DN6 , pls help review, thx. |
Sorry, something went wrong.
|
Style fixes have been applied. View the workflow run here. |
Sorry, something went wrong.
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
Sorry, something went wrong.
|
Thanks @kaixuanliu! I think we would need to add the variant check when there are no component folders as well. if not components:
return any(".safetensors" in filename for filename in filenames)And I think the changes will break the current is_safetensors_compatible tests here |
Sorry, something went wrong.
|
@DN6 , Hi, thanks for your advice. Have updated the code and test cases |
Sorry, something went wrong.
Signed-off-by: Liu, Kaixuan <kaixuan.liu@intel.com>
Signed-off-by: Liu, Kaixuan <kaixuan.liu@intel.com>
|
Style fixes have been applied. View the workflow run here. |
Sorry, something went wrong.
Signed-off-by: Liu, Kaixuan <kaixuan.liu@intel.com>
|
@DN6 Hi, I have checked all the CI tests offline. Can you help trigger the CI workflow again? |
Sorry, something went wrong.
There was a problem hiding this comment.
Thanks you 🙏🏽 @kaixuanliu
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
No description provided.