-
Notifications
You must be signed in to change notification settings - Fork 2
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
Can't reproduce #1
Comments
hi, thanks for your interests. Could you please provide your experimental results? Just FYI, you need to tune the hyperparameters following the experimental setup described in our paper. |
as I mentioned, you need to tune the hyperparameters for FLIT(+) follow our paper. We do not find a set of hyperparameters that fits all datasets. However, it is wired to see that fedavg has such good performance. I will check our experiments, and will get you back soon. |
Thanks for your prompt response, with reference to your suggestion, I have also executed the relevant FedAvg code on other datasets, but also got surprising results on some datasets, as follows.
|
Thanks for your reply. In view of the current problems, I suggest that you carefully check the current open source code for errors and update the correct code. If the current code has no errors in FedAvg, it is obvious that you did not find the optimal baseline of FedAvg. I also recommend that you re-run the relevant experiments of FLIT(+) (especially on the ESOL and ClinTox datasets) to ensure that the conclusions in the paper are correct. |
Sorry, I'm having some problems reproducing your work. I can't get the same results as in your paper(https://doi.org/10.1016/j.patter.2022.100521) by following the readme and code here.
The text was updated successfully, but these errors were encountered: