-
Notifications
You must be signed in to change notification settings - Fork 494
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
Mistakes of Workload calculation #441
Comments
i think they do make the mistake :( |
@shadow150519 Yes, this is a typo in the paper. Despite the typo, the final conclusion in equation (6) is right and BytePS correctly implements the traffic allocation strategy. Thank you for pointing out this. |
@ymjiang and the topt should be the following result eq7 is wrong and should be the following i think these should be the correct result |
i think you do the calculation totally wrong in section 4.1 |
The calculation of$Mss_{GPU}$ and $Mss_{CPU}$ may have some mistakes. The following is my calculation procedure:
while in the paper:
The text was updated successfully, but these errors were encountered: