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

Add --min-deployment-target Argument to Set Deployment Target #362

Open
swiftlysingh opened this issue Oct 16, 2024 · 0 comments
Open

Add --min-deployment-target Argument to Set Deployment Target #362

swiftlysingh opened this issue Oct 16, 2024 · 0 comments

Comments

@swiftlysingh
Copy link

Hi everyone,

I’m working on converting the Stable Diffusion XL model for iOS 18 and encountered an issue during post-training quantization. The error message suggested that iOS 16 doesn't support the optimizations I'm applying:

AssertionError: The IOS16 only supports per-tensor LUT, but got more than one LUT on 0th axis. LUT shape: (80, 1, 1, 1, 16, 1)

This seems to be due to the minimum deployment target being set to iOS 16 by default. Updating the target to iOS 18 should resolve the issue.

For more context, see this related Apple Developer forum thread.

Proposed Solution:

To make this more flexible, I propose adding a --min-deployment-target argument to the conversion script, which would allow users to set their desired deployment target. This could be set to the current default macOS15.

Open to Suggestions:

I’m open to feedback and would appreciate any input on the best way to implement this feature. If there’s a preferred approach for handling deployment target settings, I’d be happy to discuss further or explore alternatives. Let me know if you'd like me to submit a PR, or if there are any changes you’d suggest to make this more robust.

Best regards,
Pushpinder Pal Singh

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

No branches or pull requests

1 participant