-
Notifications
You must be signed in to change notification settings - Fork 26
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
[Alpha pipeline] Add segmentation component #149
Conversation
83730b1
to
a126c99
Compare
@@ -0,0 +1,24 @@ | |||
export METADATA='{"run_id":"test_niels", "base_path":"/Users/nielsrogge/Documents/fondant_artifacts"}' |
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.
Will remove this script when PR gets approved
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.
Thanks @NielsRogge :) Seems like the development of the components is going fast now which is nice to see.
Left a few comments.
I feel like a lot of the dask inference stuff can be packaged into a common util at some point to make it easier to develop (not part of this PR). Wdyt?
examples/pipelines/controlnet-interior-design/components/segment_images/src/main.py
Show resolved
Hide resolved
examples/pipelines/controlnet-interior-design/components/segment_images/src/main.py
Outdated
Show resolved
Hide resolved
examples/pipelines/controlnet-interior-design/components/segment_images/src/main.py
Outdated
Show resolved
Hide resolved
examples/pipelines/controlnet-interior-design/components/segment_images/src/main.py
Outdated
Show resolved
Hide resolved
examples/pipelines/controlnet-interior-design/components/segment_images/src/palette.py
Show resolved
Hide resolved
examples/pipelines/controlnet-interior-design/components/segment_images/requirements.txt
Show resolved
Hide resolved
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.
Thanks @NielsRogge!
Don't forget to remove the script.
78f7a5a
to
4950956
Compare
This PR adds the final component of the ControlNet pipeline, namely the segmentation one. --------- Co-authored-by: Niels Rogge <nielsrogge@Nielss-MacBook-Pro.local> Co-authored-by: Philippe Moussalli <philippe.moussalli95@gmail.com> Co-authored-by: Robbe Sneyders <robbe.sneyders@ml6.eu>
This PR adds the final component of the ControlNet pipeline, namely the segmentation one.