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

[PTQ4SR] Unfair comparison in paper? #53

Open
Cheeun opened this issue Jun 9, 2023 · 5 comments
Open

[PTQ4SR] Unfair comparison in paper? #53

Cheeun opened this issue Jun 9, 2023 · 5 comments

Comments

@Cheeun
Copy link

Cheeun commented Jun 9, 2023

In the CVPR2023 paper, "Toward Accurate Post-Training Quantization for Image Super Resolution",
Table 5 (Sec. 4.2) of the paper seems unfair.

  • PAMS and FQSR are implemented on EDSR-baseline model (layers=16, dimension size=64),

  • while PTQ4SR (this works) is implemented on EDSR model (layers=32, dimension size=256).

Since the accuracy of EDSR and EDSR-baseline is largely different, isn't it unfair to compare methods with different backbones?

Are there comparisons made using the same EDSR-baseline backbone?

Looking forward to your code release!

@Jeozhao
Copy link

Jeozhao commented Jun 13, 2023

I found that this code you mentioned is no longer available. Could you please share a code with me? My email is 751350322@qq.com

@Cheeun
Copy link
Author

Cheeun commented Jun 13, 2023

Hi, unfortunately, I couldn't find the code as well, but I just left the issue here as the link from the paper leads to this repository.

@tuzhijun
Copy link
Contributor

Sorry for that the code is not quite ready yet for personal resason. We will release our code within one month. And we will add QAT experiments based on EDSR (layers=16, dimension size=64) and present them in the latest code.

@Angiemaster
Copy link

Did the code release?

@leenas233
Copy link

Sorry for that the code is not quite ready yet for personal resason. We will release our code within one month. And we will add QAT experiments based on EDSR (layers=16, dimension size=64) and present them in the latest code.

Will this code be released?

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

5 participants