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 Availability Annotations #18

Merged
merged 1 commit into from
Dec 14, 2022
Merged

Add Availability Annotations #18

merged 1 commit into from
Dec 14, 2022

Conversation

nathantannar4
Copy link
Contributor

  • Add availability annotations rather than limiting package availability to iOS 16 / macOS 12. This is necessary for anyone who wants to incorporate this package in an existing app that needs to support older platforms, since weak linking a Swift package is not possible.

@msiracusa
Copy link
Collaborator

Thank you for your interest in contributing to Core ML Stable Diffusion! Please review CONTRIBUTING.md. If you would like to proceed with this pull request, please indicate your agreement to the terms outlined in CONTRIBUTING.md by checking the box below.

We appreciate your interest in the project!

  • I agree to the terms outlined in CONTRIBUTING.md

@nathantannar4
Copy link
Contributor Author

  • I agree to the terms outlined in CONTRIBUTING.md

@msiracusa
Copy link
Collaborator

Hi @nathantannar4! Thanks again for putting up this PR. We having a few other PRs in flight at the moment so please bear with us as we figure out the best ordering.

The latest change introduced some conflicts and new symbols that will also need availability annotations. It would be great if you can update this PR to resolve those. Thanks!

@nathantannar4
Copy link
Contributor Author

No problem, revised

@msiracusa msiracusa merged commit 152dd65 into apple:main Dec 14, 2022
@msiracusa msiracusa mentioned this pull request Dec 15, 2022
takeyat pushed a commit to takeyat/ml-stable-diffusion that referenced this pull request Dec 25, 2022
* commit 'e07c4d00c387840f70fa3701fb3a51c2a32f37b8':
  Move guidanceScale as generation parameter (apple#46)
  Add brief instructions to download weights from the Hub (apple#10)
  Adds Negative Prompts (apple#61)
  Changed seed type into UInt32 (apple#47)
  fixes apple#77
  Update README.md (apple#66)
  Add Filename Character Limit (apple#19)
  Implement DPM-Solver++ scheduler (apple#59)
  Fix typos: Successfully facilitate getting pipeline overridden (apple#30)
  Undefined name: from typing import List (apple#31)
  Add Availability Annotations (apple#18)
  README improvements and reduceMemory option in Swift
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.

3 participants