Skip to content

Changed seed type into UInt32 #47

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

Merged
merged 1 commit into from
Dec 20, 2022

Conversation

Wanaldino
Copy link
Contributor

The reason is that depending on platform Int could be 64 bits or 32, also unsigned type. So when creating latents and parsing Int to UInt32, the system could throw an unexpected error.

@Wanaldino Wanaldino force-pushed the Switch_seed_type_Int_2_UInt32 branch from 3efcb41 to 616068d Compare December 7, 2022 12:35
@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

@Wanaldino Wanaldino force-pushed the Switch_seed_type_Int_2_UInt32 branch from 616068d to 73c8b54 Compare December 19, 2022 15:48
@Wanaldino
Copy link
Contributor Author

  • I agree to the terms outlined in CONTRIBUTING.md

@atiorh
Copy link
Contributor

atiorh commented Dec 20, 2022

@pcuenca Could you please confirm that #58 is solving the same issue as this PR and approve these changes to be merged instead? (FIFO) 🙏

@pcuenca
Copy link
Contributor

pcuenca commented Dec 20, 2022

Sure, sorry about the duplicate PR!

@atiorh atiorh merged commit 4c00b32 into apple:main Dec 20, 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
@Wanaldino Wanaldino deleted the Switch_seed_type_Int_2_UInt32 branch January 26, 2023 12:04
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.

4 participants