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

[Feature]: 指定Sampler和Clip Skip #22

Open
3 tasks done
eigeen opened this issue Feb 27, 2023 · 2 comments
Open
3 tasks done

[Feature]: 指定Sampler和Clip Skip #22

eigeen opened this issue Feb 27, 2023 · 2 comments
Assignees
Labels
enhancement 新的功能

Comments

@eigeen
Copy link

eigeen commented Feb 27, 2023

报告清单

  • 我已仔细阅读并了解上述注意事项
  • 我已使用最新版本测试过,确认功能尚未实现
  • 我确定在 GitHub Issues 中没有相同或相似的需求

需求原因

当前版本缺少指定sampler和clip skip值的功能

预期行为

指定sampler:可从列表中选一个。
指定clip skip值:整数,>=1

解决方案

未来有空闲时间,本人会考虑自行实现并提交pr。因此特开此issue,请作者评估可行性。

@eigeen eigeen added the enhancement 新的功能 label Feb 27, 2023
@sena-nana
Copy link
Owner

sena-nana commented Feb 27, 2023

虽然0.6本身就要实现sampler()
clip的话也并不算难,只要加一点东西,但是这个设置几乎没有人去调。为什么会有加这个参数的想法?我想了解下原因再决定有没有必要

@eigeen
Copy link
Author

eigeen commented Feb 27, 2023

虽然0.6本身就要实现sampler() clip的话也并不算难,只要加一点东西,但是这个设置几乎没有人去调。为什么会有加这个参数的想法?我想了解下原因再决定有没有必要

只是看部分模型标注了最佳的Clip Skip值,所以觉得可以加上(虽然确实大部分时候不用动)

0.6似乎开发了挺久,我看最新的版本运行起来有问题,似乎正在开发,所以暂时不改动了。细节内容群里聊吧。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 新的功能
Projects
None yet
Development

No branches or pull requests

2 participants