Feature/SK-918 | Add --version to fedn CLI (#649)

* added --version command for fedn package

* fixed single quotes with double quotes

* fixed double quotes for version

* intendation error fixed

* fixed the global variable

* removed raise in the round handler

fedn/cli/main.py

@@ -1,14 +1,22 @@
+import importlib.metadata
+
 import click
 
 CONTEXT_SETTINGS = dict(
     # Support -h as a shortcut for --help
     help_option_names=["-h", "--help"],
 )
 
+# Dynamically get the version of the package
+try:
+    version = importlib.metadata.version("fedn")
+except importlib.metadata.PackageNotFoundError:
+    version = "unknown"
+
 
 @click.group(context_settings=CONTEXT_SETTINGS)
+@click.version_option(version)
 @click.pass_context
 def main(ctx):
-    """:param ctx:
-    """
+    """:param ctx:"""
     ctx.obj = dict()

fedn/common/config.py

@@ -2,9 +2,6 @@
 
 import yaml
 
-global STATESTORE_CONFIG
-global MODELSTORAGE_CONFIG
-
 SECRET_KEY = os.environ.get("FEDN_JWT_SECRET_KEY", False)
 FEDN_JWT_CUSTOM_CLAIM_KEY = os.environ.get("FEDN_JWT_CUSTOM_CLAIM_KEY", False)
 FEDN_JWT_CUSTOM_CLAIM_VALUE = os.environ.get("FEDN_JWT_CUSTOM_CLAIM_VALUE", False)

fedn/network/combiner/roundhandler.py

@@ -309,7 +309,6 @@ def _assign_round_clients(self, n, type="trainers"):
             clients = self.server.get_active_trainers()
         else:
             logger.error("(ERROR): {} is not a supported type of client".format(type))
-            raise
 
         # If the number of requested trainers exceeds the number of available, use all available.
         if n > len(clients):