dealt with flake8 issues

also the order of the parameters for the --stdout / --no-logfile were
causing the daemon to not be able to start up with any commands as the
stdout kwarg was being passed to the function when it was expecting
no_logfile.

amdfan/commands.py

@@ -78,13 +78,13 @@ class FileDescriptorOpt(click.ParamType):
     "-p",
     type=click.Path(),
     default=os.path.join(PIDFILE_DIR, "amdfan.pid"),
-    help=f"Pidfile path",
+    help="Pidfile path",
     show_default=True,
 )
 @click.option(
     "--no-pidfile",
     is_flag=True,
-    help=f"Disable pidfile",
+    help="Disable pidfile",
 )
 @click.option(
     "-l",
@@ -95,8 +95,8 @@ class FileDescriptorOpt(click.ParamType):
     show_default=True,
 )
 @click.option(
-    "--stdout",
     "--no-logfile",
+    "--stdout",
     is_flag=True,
     help="Disable logging file (prints to stdout instead)",
 )

amdfan/controller.py

@@ -6,8 +6,7 @@
 import signal
 import sys
 import threading
-import time
-from typing import Any, Callable, Dict, List, Optional, Self
+from typing import Any, Callable, Dict, List, Optional
 
 import numpy as np
 import yaml
@@ -41,7 +40,7 @@ def daemonize(stdin="/dev/null", stdout="/dev/null", stderr="/dev/null") -> None
         if pid > 0:
             os._exit(0)
     except OSError as e:
-        raise Exception("Unable to background amdfan")
+        raise Exception("Unable to background amdfan: %s" % e)
 
     os.chdir("/")
     os.setsid()
@@ -52,7 +51,7 @@ def daemonize(stdin="/dev/null", stdout="/dev/null", stderr="/dev/null") -> None
         if pid > 0:
             os._exit(0)
     except OSError as e:
-        raise Exception("Unable to daemonize amdfan")
+        raise Exception("Unable to daemonize amdfan: %s" % e)
 
     redirect_fd(stdin, stdout, stderr)