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

improve: 优化 base64 image 加载时的文件命名方式 #216

Merged
merged 5 commits into from
Nov 16, 2024

Conversation

KwokKwok
Copy link
Contributor

之前 Load Image 节点扩展支持 base64 图片加载的文件命名考虑欠妥,这里做一些修正:

  1. 使用 md5 hash 代替 timestamp,避免重复写入相同的图片文件
  2. 已存在相同 hash 的文件,则直接加载

对于 base64 图片来说,hash 冲突导致相同 hash 的概率可忽略不计,可以放心使用


写入文件的命名结果如图:

Snipaste_2024-11-13_13-10-21

KwokKwok and others added 2 commits November 13, 2024 13:16
1. 使用 md5 hash 代替 timestamp,避免重复写入相同的图片文件
2. 已存在相同 hash 的文件,则直接加载
3. 对于 base64 图片来说,hash 冲突的概率可忽略不计
@doombeaker
Copy link
Collaborator

感谢提交 PR。
麻烦在自己fork的仓库,设置下这个 secrets.BIZYAIR_KEY

才能启动对应 CI

https://github.com/siliconflow/BizyAir/blob/master/.github/workflows/workflow-ci.yml#L101C30-L101C48

@KwokKwok
Copy link
Contributor Author

感谢提交 PR。 麻烦在自己fork的仓库,设置下这个 secrets.BIZYAIR_KEY

才能启动对应 CI

https://github.com/siliconflow/BizyAir/blob/master/.github/workflows/workflow-ci.yml#L101C30-L101C48

确定是需要在我 fork 的仓库设置吗?我感觉走的主仓库的 CI 吧,刚才我设置了一下,重新触发了一下 CI 好像还是不行
image

@KwokKwok
Copy link
Contributor Author

是否是环境变量使用有误?

BIZYAIR_API_KEY = env("BIZYAIR_API_KEY", str, load_api_key()[1])

BIZYAIR_KEY: ${{ secrets.BIZYAIR_KEY }}

@doombeaker
Copy link
Collaborator

doombeaker commented Nov 15, 2024

是否是环境变量使用有误?

BIZYAIR_API_KEY = env("BIZYAIR_API_KEY", str, load_api_key()[1])

BIZYAIR_KEY: ${{ secrets.BIZYAIR_KEY }}

是否是环境变量使用有误?

BIZYAIR_API_KEY = env("BIZYAIR_API_KEY", str, load_api_key()[1])

BIZYAIR_KEY: ${{ secrets.BIZYAIR_KEY }}

这个环境变量是在这个脚本里用的:

bizyair_key = os.getenv("BIZYAIR_KEY", "")

这个脚本会写生成 api_key.ini 文件。

我先把两种环境变量都设置上看看。

@doombeaker doombeaker merged commit d9165dc into siliconflow:master Nov 16, 2024
2 of 4 checks passed
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