Skip to content

Commit

Permalink
Remove unused arg and docstring (securefederatedai#1090)
Browse files Browse the repository at this point in the history
Signed-off-by: Shah, Karan <[email protected]>
  • Loading branch information
MasterSkepticista committed Oct 22, 2024
1 parent 2f9bfaa commit d52acba
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 30 deletions.
15 changes: 2 additions & 13 deletions openfl/interface/aggregator.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,13 +52,7 @@ def aggregator(context):
type=ClickPath(exists=True),
)
def start_(plan, authorized_cols):
"""
Start the aggregator service.
Args:
plan (str): Path to the federated learning plan.
authorized_cols (str): Path to the authorized collaborator list.
"""
"""Start the aggregator service."""

if is_directory_traversal(plan):
echo("Federated learning plan path is out of the openfl workspace scope.")
Expand Down Expand Up @@ -154,12 +148,7 @@ def find_certificate_name(file_name):
)
@option("-s", "--silent", help="Do not prompt", is_flag=True)
def _certify(fqdn, silent):
"""Sign/certify the aggregator certificate key pair.
Args:
fqdn (str): The fully qualified domain name of aggregator node.
silent (bool): Flag to enable silent mode.
"""
"""Sign/certify the aggregator certificate key pair."""
certify(fqdn, silent)


Expand Down
19 changes: 2 additions & 17 deletions openfl/interface/collaborator.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,7 @@ def collaborator(context):
help="The certified common name of the collaborator",
)
def start_(plan, collaborator_name, data_config):
"""Start a collaborator service.
Args:
plan (str): Federated learning plan.
collaborator_name (str): The certified common name of the collaborator.
data_config (str): The data set/shard configuration file.
"""
"""Start a collaborator service."""

if plan and is_directory_traversal(plan):
echo("Federated learning plan path is out of the openfl workspace scope.")
Expand Down Expand Up @@ -337,16 +331,7 @@ def register_collaborator(file_name):
help="Import the archive containing the collaborator's" " certificate (signed by the CA)",
)
def certify_(collaborator_name, silent, request_pkg, import_):
"""Certify the collaborator.
Args:
collaborator_name (str): The certified common name of the collaborator.
silent (bool): Do not prompt.
request_pkg (str): The archive containing the certificate signing
request (*.zip) for a collaborator.
import_ (str): Import the archive containing the collaborator's
certificate (signed by the CA).
"""
"""Certify the collaborator."""
certify(collaborator_name, silent, request_pkg, import_)


Expand Down

0 comments on commit d52acba

Please sign in to comment.