-
Notifications
You must be signed in to change notification settings - Fork 20
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
Why numer of parameter is different with paper #13
Comments
Hi, this may be related to the package we use to measure the model/parameter size. We use the same package to measure their size (SwinIR/HAT/DRCT) in our paper. I need time to check which package was used for measurement. For some reason I can't immediately get the file we used. Nonetheless, their relative relationship should the same since all three models was measued by the same package. Thanks! |
Hi, We recalculated the parameters for HAT / HAT-L / DRCT / DRCT-L, and after careful comparison, we found that, as you mentioned, there was an error in the parameter count for HAT/DRCT, while the numbers for the Large versions are correct. We apologize for this mistake and will be updating the correct numbers on arXiv. Thank you once again! Here are the corrected parameter counts: HAT : 20,772,507 DRCT : 14,139,579 We used fvcore to measure the parameter counts. The code is as follows:
|
I checked num of parameter of DRCT in paper, alomost 10M
But, When I run the DRCT model using 180 embed_dim, num of paramter is almost 13M.
What is different?
The text was updated successfully, but these errors were encountered: