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

image_resolution 参数文档是不是写错了? #6670

Closed
1 task done
rover5056 opened this issue Jan 16, 2025 · 1 comment
Closed
1 task done

image_resolution 参数文档是不是写错了? #6670

rover5056 opened this issue Jan 16, 2025 · 1 comment
Labels
solved This problem has been already solved

Comments

@rover5056
Copy link

Reminder

  • I have read the above rules and searched the existing issues.

System Info

  • llamafactory version: 0.9.1.dev0
  • Platform: Linux-5.4.119-1-tlinux4-0010.3-x86_64-with-glibc2.35
  • Python version: 3.10.12
  • PyTorch version: 2.4.1+cu124 (GPU)
  • Transformers version: 4.46.1
  • Datasets version: 3.0.1
  • Accelerate version: 1.0.1
  • PEFT version: 0.11.1
  • TRL version: 0.9.6
  • GPU type: NVIDIA A800-SXM4-80GB
  • DeepSpeed version: 0.15.4
  • vLLM version: 0.6.3.post1

Reproduction

文档里关于图片分辨率的参数 是 : image_resolution ,默认值是 512*512 。看 代码也是

image_resolution: int = field(

所以这里应该是个 int, 512*512 = 262144

但是实际处理的时候,又是

def _preprocess_image(self, image: "ImageObject", **kwargs) -> "ImageObject":

if max(image.width, image.height) > image_resolution:

直接取得值和长短边比较了? 应该是直接和 512 比?
容易造成 confuse

Others

No response

@rover5056 rover5056 added bug Something isn't working pending This problem is yet to be addressed labels Jan 16, 2025
@rover5056
Copy link
Author

看错分支了,最新的代码已经正常。。

@hiyouga hiyouga added solved This problem has been already solved and removed bug Something isn't working pending This problem is yet to be addressed labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solved This problem has been already solved
Projects
None yet
Development

No branches or pull requests

2 participants