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

FVD values of PVDM are strange #6

Open
sihyun-yu opened this issue Jan 12, 2024 · 5 comments
Open

FVD values of PVDM are strange #6

sihyun-yu opened this issue Jan 12, 2024 · 5 comments
Labels
question Further information is requested

Comments

@sihyun-yu
Copy link

sihyun-yu commented Jan 12, 2024

Hi, I am the first author of PVDM, and I just checked the FVD values of PVDM are much worse than the values that I reported in the paper. Could you tell me why such differences exist?

Many people tried (and succeeded) to reproduce the values, so it is weird to me.

@maxin-cn
Copy link
Collaborator

Hi, I am the first author of PVDM, and I just checked the FVD values of PVDM are much worse than the values that I reported in the paper. Could you tell me why such differences exist?

Many people tried (and succeeded) to reproduce the values, so it is weird to me.

Hello, thanks for your interest of our work. For UCF101 and Skytimelapse datasets, we followed the paper and used provided pre-trained checkpoints for evaluation. However, we were unable to reproduce the reported results. It would be very helpful if you could provide any details or a complete evaluation code on how to obtain the results.

@sihyun-yu
Copy link
Author

sihyun-yu commented Jan 12, 2024

Just a quick check:

  • Did you use ema weights?
  • Did you use 2,048 samples for evaluation?
  • Did you use 400 steps (DDPM sampler) for generating samples?

Because with this setup, I got many mails that the results can be reproduced with the checkpoints.

@sihyun-yu
Copy link
Author

Hi,

there even exists a recent arxiv paper that has succeeded to reproduce the values of PVDM: https://arxiv.org/abs/2402.13729v1
If you face difficulties to reproduce the values, you should ask authors before making it public..

@sihyun-yu
Copy link
Author

Can you please reply to this issue?

@maxin-cn
Copy link
Collaborator

Can you please reply to this issue?

Thank you very much for sharing the info and sorry for missing your question. We will thoroughly review the results with the information you've provided. We would greatly appreciate any additional details you could offer regarding the reimplementation of your work. Many thanks again.

@maxin-cn maxin-cn added the question Further information is requested label Jul 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants