-
Notifications
You must be signed in to change notification settings - Fork 168
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 Stable Diffusion OpenVINO example #853
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please put back the readme at the root of the stable_diffusion
folder.
"engine": { | ||
"search_strategy": false, | ||
"evaluate_input_model": false, | ||
"host": "local_system", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: both the host and local system are redundant since it's the default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah since this is an example for user, I would like to make it clear.
@@ -74,6 +74,10 @@ def __init__( | |||
self.objective_dict = objective_dict or {} | |||
self.is_marked_pareto_frontier = is_marked_pareto_frontier | |||
|
|||
def get_output_model_path(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see where it is used. could you please help clarify the changes?
Describe your changes
Add Stable Diffusion OpenVINO example.
Next step:
examples/stable_diffusion
andexamples/directml/stable_diffusion
, or rewrite the README.md to link each other.Checklist before requesting a review
lintrunner -a
(Optional) Issue link