-
Notifications
You must be signed in to change notification settings - Fork 6
feature(sunjx): add rejective sampling pipeline in t2i demo #3
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
base: main
Are you sure you want to change the base?
feature(sunjx): add rejective sampling pipeline in t2i demo #3
Conversation
| """ | ||
| Convert filtered samples to rejection sampling training format. | ||
| Args: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
参考repo里面已有的注释格式,使用统一的param type格式
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
description中加上英文和中文的对PR的描述
lightrft/models/grm_vl.py
Outdated
|
|
||
| # Check if model is Qwen2.5-VL which doesn't support dtype parameter | ||
| # We can check by looking at the model path or config | ||
| from transformers import AutoConfig |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里改动的原因是?
No description provided.