Closed charchit7 closed 8 months ago
@charchit7 should I review this?
@charchit7 should I review this?
Yes, Merve. Thank you :) Just wanted to know if the flow and length of content was fine here.
Thanks a lot for this!!!
welcome 🤗
Hey @merveenoyan please check, fixed the required changes.
LGTM if you could accept my suggestions
Thanks @merveenoyan. Added the changes. Please have a look. Had forgotten the installation part earlier.
Just two small things, but in general LGTM 👍
I like the high level description that covers the most important parts of SD
Thanks @johko fixed the required changes. Please have a look.
@charchit7 since we merged your other PR, is it ok if you can solve the merge conflict caused by that?
@charchit7 since we merged your other PR, is it ok if you can solve the merge conflict caused by that?
Hey @merveenoyan done. Please have a look.
@johko can I merge this pull request then? I can see the button. Please let me know.
@charchit7 I'm merging it now
Hey Everyone!
This PR adds basic structure of Stable Diffusion and it's use-cases. Part of https://github.com/johko/computer-vision-course/issues/28
Need advice here, since the content of SD can go super large if I go into depth is the basic structure fine here. @MKhalusova @merveenoyan @mattmdjaga @hwaseem04