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

Different results #4

Open
hieuvt29 opened this issue Jan 6, 2021 · 4 comments
Open

Different results #4

hieuvt29 opened this issue Jan 6, 2021 · 4 comments

Comments

@hieuvt29
Copy link

hieuvt29 commented Jan 6, 2021

Hi authors,
It's great that you publish the source code, but the default hyper-parameters seem not to be correctly tuned, so could you please share the hyper-params to produce the graph below (taken from your paper)
I also add the graph showing the results that I ran with hyper-params from your source code with command
python run.py --model vgg --nQuery 100 --data CIFAR10 --alg badge

Reported in the paper:
image
Run default code: (the x-axis is the number of data point, divided by 100 - the query batch size)
The gap is nearly 10% even with more data
image

@JordanAsh
Copy link
Owner

JordanAsh commented Jan 6, 2021 via email

@hieuvt29
Copy link
Author

hieuvt29 commented Jan 7, 2021

Thank you for replying,
I assume you're taking about the learning rate for training the task model, but as I understand based on the code here https://github.com/JordanAsh/badge/blob/master/query_strategies/strategy.py#L63, the model will be trained until getting 99% accuracy on training set, so it'll fit the training data without really care about the learning rate (I also checked the log). Should I change something else?

@JordanAsh
Copy link
Owner

JordanAsh commented Jan 7, 2021 via email

@hieuvt29
Copy link
Author

hieuvt29 commented Jan 7, 2021

"task model" here I mean the classifier (or the learner), I agree that different learning rates will eventually get different accuracies on test set but the variance should not be that big (10%).
Anyway, thank you for your replies!

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