FedML-AI Docstrings Update
close #1259
Add doc strings for the python methods. and code update
I will add docstrings to every file in the repo. please do not merge it now.
@fedml-alex would you please approve the workflows
python\fedml\fa\aggregator\avg_aggregator.py
Could you fix merge conflicts please?
Could you fix merge conflicts please?
Sure I am trying to solve it ASAP.
Could you fix merge conflicts please?
Sure I am trying to solve it ASAP.
Actually, while you're at it, could you instead split the PR's per module? Would give us more velocity to review that way
I was thinking of it. at the start of this week.
Could you fix merge conflicts please?
Sure I am trying to solve it ASAP.
Actually, while you're at it, could you instead split the PR's per module? Would give us more velocity to review that way
![]()
@alaydshah the conflicts are not visible in the local text editor. what should I do?
@alaydshah I have solved the conflicts. kindly have a look
@rajveer43 can you please check if this PR works on dev and checkout from there?