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

standalone Benchmark running the MNIST and Shakespeare #79

Closed
sjtu-cz opened this issue Dec 16, 2020 · 8 comments
Closed

standalone Benchmark running the MNIST and Shakespeare #79

sjtu-cz opened this issue Dec 16, 2020 · 8 comments
Assignees

Comments

@sjtu-cz
Copy link

sjtu-cz commented Dec 16, 2020

I can not get the same results when I run the command:
sh run_fedavg_standalone_pytorch.sh 0 10 10 10 shakespeare ./../../../data/shakespeare rnn hetero 100 1 0.8 sgd 0
my results is:
https://wandb.ai/pilgrim_cz/fedml/runs/5jqpw1i4?workspace=user-pilgrim_cz
the benchmart res is https://wandb.ai/automl/fedml/runs/144ey9w6
same problem when I run minst dataset
Is there anything wrong? I did not change any code.

@Elliebababa
Copy link
Contributor

Elliebababa commented Dec 16, 2020

Hi, the link you provide is not accessible. And we have updated the code recently. Did you run the most updated code? Or could you please provide more information so that we can help.

@sjtu-cz
Copy link
Author

sjtu-cz commented Dec 16, 2020

image

@sjtu-cz
Copy link
Author

sjtu-cz commented Dec 16, 2020

ok, I will test the most updated code later, updated 7 hours ago ?

@Elliebababa
Copy link
Contributor

Yes, please have a try.

@Elliebababa Elliebababa self-assigned this Dec 16, 2020
@chaoyanghe
Copy link
Member

@sjtu-cz thanks for your feedback. What's your git version?
Please let us know your results based on the latest code.

@sjtu-cz
Copy link
Author

sjtu-cz commented Dec 16, 2020

The results are correct now, the problem is caused by parameter passing error in FedML/fedml_experiments/standalone/fedavg/run_fedavg_standalone_pytorch.sh
EPOCH=$10 => EPOCH=${10} ...
This is a small problem,the former epoch is always 0, so the result is not correct.

@sjtu-cz sjtu-cz closed this as completed Dec 16, 2020
@chaoyanghe
Copy link
Member

@sjtu-cz Great! Happy to know you can reproduce the result.

@am009
Copy link
Contributor

am009 commented Apr 11, 2022

I encountered same problem today. 😭 The error in the script still exists. I really hope it can be corrected so that other people will not meet the same problem again.
I opened a pull request at #180 . Hope it's helpful.

Related link:
https://stackoverflow.com/questions/4912733/how-to-handle-more-than-10-parameters-in-shell

chaoyanghe added a commit that referenced this issue Apr 15, 2022
[#79] Fix argument passing in scripts under fedml_experiments.
fedml-alex pushed a commit that referenced this issue Jun 1, 2022
[#79] Fix argument passing in scripts under fedml_experiments.
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

4 participants