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

Question about discriminator input #22

Open
serkansulun opened this issue Mar 6, 2018 · 3 comments
Open

Question about discriminator input #22

serkansulun opened this issue Mar 6, 2018 · 3 comments

Comments

@serkansulun
Copy link

Hi Matt,
Thanks for the great code. According to the paper, the input to the discriminator is a sequence of frames (history of frames, and the next frame). If I understand your code correctly, the input to the discriminator is a single frame, which is only the next frame (either generated or from ground-truth). Is this right? If yes, wouldn't this prevent the discriminator to make use of the continuity in the video?
Thanks in advance.

@dyelax
Copy link
Owner

dyelax commented Mar 8, 2018

@ssulun16 You are totally right. Cant believe I missed that.

From section 2.2 in the paper:

The discriminative model D takes a sequence of frames, and is trained to predict the probability that the last frames of the sequence are generated by G. Note only the last frames are either real of generated by G, the rest of the sequence is always from the dataset. This allows the discriminative model to make use of temporal information, so that G learns to produce sequences that are temporally coherent with its input.

I'm pretty busy with other projects right now, but would be awesome if you could make a PR that fixes this! Seems like it should just be a small tweak in the build_feed_dict function of DiscriminatorModel

@serkansulun
Copy link
Author

I made some other changes before so the code isn't very clean but I'll do that if I can find some time after I have a final version for myself. In the meantime I'll open issues if I find other problems so that it can help other people.

@serkansulun
Copy link
Author

I have fixed it but I also switched to Pytorch in the meantime, it is available in my repositories if anyone needs it.

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