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

FedML-AI Docstrings Update #1279

Open
wants to merge 77 commits into
base: master
Choose a base branch
from
Open

Conversation

rajveer43
Copy link

@rajveer43 rajveer43 commented Sep 1, 2023

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

@rajveer43 rajveer43 changed the title FedML-AI Docs Update FedML-AI Docstrings Update Sep 7, 2023
@rajveer43
Copy link
Author

@alaydshah I have solved the conflicts. kindly have a look

@fedml-dimitris fedml-dimitris added documentation Improvements or additions to documentation python Pull requests that update Python code labels Oct 21, 2023
@fedml-dimitris
Copy link
Contributor

fedml-dimitris commented Oct 21, 2023

@rajveer43 can you please check if this PR works on dev and checkout from there?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding docstrings to the codebase
3 participants