-
Notifications
You must be signed in to change notification settings - Fork 81
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
Configs for Image Classification (cifar10) #16
Comments
@MostafaDehghani for clarity on image tasks. Note: we might take awhile to reply due to the upcoming deadlines. Thanks |
@vanzytay Thanks for the quick response! |
Thank you @keroro824 for the question. So if I understood correctly, you are looking for the configs for the vanilla transformer to reproduce its results on CIFAR10 dataset in LRA. For that, you can use the followings as the model hparams:
We are planning to release the code for all models and the best performing configurations, as soon as possible. In the meantime, please let us know if you had any questions :) |
@MostafaDehghani Thank you !!! I can replicate it now! |
No problem at all! Perfect! |
The above comment states 1 layer and leaves learning rate unspecified. This means learning rate will be 0.0005 inherited from base_cifar10_config. The arxiv paper states: 3 layers, learning rate 0.01. The openreview paper states: 3 layers, learning rate 0.01. Notably, the config file still contains nothing. Currently, the code in this repository is inconsistent with the published articles. Do you plan on fixing these inconsistencies? Or did you abandon this project? |
IIRC config files takes precedence over the paper hparams. We will update the readme here to state this. |
The best results in the paper are all reproducible from the code in the repo. Have you tried the configs that are shared here? Many people reproduced the results without any issue after our last update. LRA is a living benchmark. We tried our best to tune hyper-parameters of each model we had in the paper and some of the authors of those models reached out to help us find better hyper parameters. The codebase has the most updated version of those and it can be used for reproducing the results.
If you read the code carefully, you can see that the config file you are referring to is inheriting from the base config! |
This was not clear to me, I apologize for the misunderstanding.
I meant that the file was empty, so learning rate was inherited as 0.0005 from base config file while article reported learning rate 0.01. I was under the impression that the article hyperparams would be used, but as vanzytay clarified this is not the case. |
It is a good reminder to us that an update of the paper is due to ask researchers to defer to the codebase to reproduce the results. In our 2nd update, we ran all the cifar results again to make sure they were reproducible. So the code configs should be good. Do give it a try and let us know if you run into any other issues. Thanks! |
Thanks for the great work!
I have a question regarding the hyperparams for training cifar10. I used the setting in this repo and replaced several hyperparams (eg n_layers n_heads etc) with the ones reported in the paper, but the best testing acc I got was 0.36:
Could you point out which params I could adjust to match the accuracy (this is for full attention).
The text was updated successfully, but these errors were encountered: