Skip to content
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

Do we need batch here? #9

Closed
zivzone opened this issue Oct 31, 2022 · 2 comments
Closed

Do we need batch here? #9

zivzone opened this issue Oct 31, 2022 · 2 comments

Comments

@zivzone
Copy link

zivzone commented Oct 31, 2022

Hi,
Thanks for your repo. I', wondering do we needs batch here?
for batch in range (batch_size):
I guess you just put one src image and one occ image as a pair into
color_transfer_sot

BR,
Ziv

@kennyvoo
Copy link
Owner

kennyvoo commented Nov 1, 2022

The batch side here refers to the solver batch instead of the image batch. This function supposes to take in 1 image pair as input. Although this function is taken from DeepFaceLab as mentioned in the code, It was originally from this source. You can refer to this website for more information about color transfer via sliced optimal transport.

@zivzone
Copy link
Author

zivzone commented Nov 3, 2022

Hi, @kennyvoo
Ok, I see. Thanks for your reply!

@zivzone zivzone closed this as completed Nov 3, 2022
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

No branches or pull requests

2 participants