KEMBAR78
Move guidanceScale as generation parameter by Wanaldino · Pull Request #46 · apple/ml-stable-diffusion · GitHub
Skip to content

Conversation

@Wanaldino
Copy link
Contributor

Moved guidanceScale as a parameter for generateImages function. Also propagated this change for the performGuidance functions in order to use this parameter.

@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

@martinlexow
Copy link

This would be a great improvement to the framework. Looking up to this!

@Wanaldino
Copy link
Contributor Author

Wanaldino commented Dec 19, 2022

  • I agree to the terms outlined in CONTRIBUTING.md

Copy link
Contributor

@atiorh atiorh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also add a CLI option for guidanceScale? I also left a comment regarding indentation. Thank you for the PR!

guidanceScale: Float = 7.5,
reduceMemory: Bool = false) {
public init(
textEncoder: TextEncoder,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please maintain the previous indentation and only remove the relevant argument?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done :D

@msiracusa msiracusa merged commit e07c4d0 into apple:main Dec 24, 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 guidanceScale branch January 26, 2023 12:03
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.

5 participants