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

Missing/wrong hparams in cifar.yaml #1

Open
emirkonuk opened this issue Jan 15, 2024 · 1 comment
Open

Missing/wrong hparams in cifar.yaml #1

emirkonuk opened this issue Jan 15, 2024 · 1 comment

Comments

@emirkonuk
Copy link

Hello,

Thanks for the great work. When trying to replicate your experiments I noticed the "update_interval" parameter is missing for CIFAR10's swag optimizer.

There are some other minor typos like e.g. unnecessary "mean_samples" key or "swag_config" should be replaced by "swag" etc. in the same cifar.yaml file. Maybe you could share a more recent version of the cifar.yaml? If not, can you tell me what the "update_interval" parameter should be?

Best,
Emir

@Feuermagier
Copy link
Owner

Hi Emir,

sorry for the confusion around the wrong configuration file. We ran most CIFAR experiments with an older version of the codebase...

I will test and update the file on the weekend, but for now the update_interval parameter should be 390:
update_interval is the interval (in batches) between successive mean samples taken by SWAG. Since we have 50000 training datapoints, a batch size of 128, SWAG runs for 50 epochs, and we want 50 mean samples, we get an update interval of (50 * 50000) / (128 * 50) = 390.625 which should be rounded down to 390.

Best,
Florian

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