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

Contributing EDEN compression pipeline #527

Merged
merged 4 commits into from
Oct 12, 2022

Conversation

yanivbi
Copy link
Contributor

@yanivbi yanivbi commented Oct 7, 2022

This PR contains a contribution by VMware Research of EDEN -- an unbiased lossy compression method that uses a random rotation followed by deterministic quantization and scaling, as described in the following ICML 2022 paper:

"EDEN: Communication-Efficient and Robust Distributed Mean Estimation for Federated Learning"
Shay Vargaftik, Ran Ben Basat, Amit Portnoy, Gal Mendelson, Yaniv Ben Itzhak, Michael Mitzenmacher,
Proceedings of the 39th International Conference on Machine Learning, PMLR 162:21984-22014, 2022.

https://proceedings.mlr.press/v162/vargaftik22a.html

yanivbi and others added 3 commits September 18, 2022 09:34
Co-Authored-By: Shay Vargaftik <16802840+shayvar@users.noreply.github.com>
Co-Authored-By: Shay Vargaftik <16802840+shayvar@users.noreply.github.com>
Co-Authored-By: Shay Vargaftik <16802840+shayvar@users.noreply.github.com>
@psfoley psfoley self-requested a review October 7, 2022 20:26
@shayvar
Copy link
Contributor

shayvar commented Oct 9, 2022

This PR contains a contribution by VMware Research of EDEN -- an unbiased lossy compression method that uses a random rotation followed by deterministic quantization and scaling, as described in the following ICML 2022 paper:

"EDEN: Communication-Efficient and Robust Distributed Mean Estimation for Federated Learning"
Shay Vargaftik, Ran Ben Basat, Amit Portnoy, Gal Mendelson, Yaniv Ben Itzhak, Michael Mitzenmacher,
Proceedings of the 39th International Conference on Machine Learning, PMLR 162:21984-22014, 2022.

https://proceedings.mlr.press/v162/vargaftik22a.html

@yanivbi yanivbi marked this pull request as ready for review October 9, 2022 11:34
@psfoley
Copy link
Contributor

psfoley commented Oct 10, 2022

Thank you @yanivbi and @shayvar for your compression pipeline contribution! As soon as the failing import is addressed, I think this is ready to merge.

Co-Authored-By: Shay Vargaftik <16802840+shayvar@users.noreply.github.com>
@yanivbi
Copy link
Contributor Author

yanivbi commented Oct 11, 2022

@psfoley, we committed the requested conditional import, thanks.

@psfoley psfoley self-requested a review October 11, 2022 18:38
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

Successfully merging this pull request may close these issues.

3 participants