SaraBabakN / MFCL-NeurIPS23

15 stars 3 forks source link

Hello, I noticed that the code only compares FedAVG,FedProx and Oracle, but I don't see FedCIL included. #5

Open Isjacky opened 1 month ago

Isjacky commented 1 month ago

Hello, I noticed that the code only compares FedAVG , FedProx and Oracle, but I don't see FedCIL included.

SaraBabakN commented 1 month ago

Hi, I used the original code released by the authors of FedCIL, which is available in their github repo.

LiangTengwen commented 1 month ago

Hi, I used the original code released by the authors of FedCIL, which is available in their github repo.

Hi,I tried to reproduce FedCIL on your code, but encountered some problems. Is the experimental result of FedCIL directly run out with the code of FedCIL github repo? Or I have to merge the FedCIL‘s code into MFCL?

SaraBabakN commented 1 month ago

Hi, yes. Initially I also faced problems using FedCIL after fixing the problems in original setting, I directly used it on my paper. Maybe you can also make sure that the code works then just plug it in here.