Fan Lai

+1. We should add `MovieLens1M` and `Yelp` for the emerging recommendation task. Please help when you are free (@dywsjtu ). Thanks! Also, we should plan for cross-silo datasets. Do you...

Hello. We are pretty happy to see that we are on the same page. :) And we are indeed planning these! Thanks for raising it up here. (1) Decentralized aggregators:...

Agree to do so. But please do not change the augmentation_factor. Instead, 1. remove [these hard codes](https://github.com/SymbioticLab/FedScale/blob/master/fedscale/core/arg_parser.py#L201); 2. Multiply the compute_speed by the factor from FLOPs. Note that the current...

We are very happy to see your entire team trying out FedScale! This means we are on the right track, and will encourage us to work hard to improve FedScale...

Thank you for diligently helping us to improve FedScale! These similar concerns remind us of the thoughtful Reviewer#3@ NeurIPS'22. While Reviewer#3 gave us a strong reject -- the rest of...

> Thank you for the fix! Thanks a lot for your feedback! This PR is still WIP, since we notice the weird accuracy issue has not been well addressed (although...

@AmberLJC Can you please help to review and test the last commit? Training gets stuck as the model (version) is missing on executors. Thanks!

Pushed the tensorflow training (#124 Support TF Training). However, there are some loose ends left: - Support model testing. This requires us to add tf modules in the current [testing...

Thanks for raising it up. Yes, the current dataloader is less efficient in my test, and I agree we'd better support naive tf dataloaders. However, it may take a while...

Hello. Thanks for trying FedScale. We want to note that **FedScale now is indeed a pull-based system and is able to support real deployment**. For example, [clients/executors periodically ping](https://github.com/SymbioticLab/FedScale/blob/master/fedscale/core/executor.py#L335) the...