Bugfixes to stabilize latest application version

src/api/ExampleAPI/exampleapi.py

@@ -22,7 +22,7 @@ def __init__(self, **kwargs):
 
     def on_change(self, id: str):
         if id not in self.ids:
-            log.error(f"{self.__class__.__name__}: Invalid setting ID: {id}")
+            log.error("%s: Invalid setting ID: %s", self.__class__.__name__, id)
         # NOTE I'd recommend a more sophisticated way of determining which widget corresponds to which setting
         # for the sake of simplicity, I'll use the ID and the type of the widget
         if self.ids[id] is MDCheckbox:

src/api/api_factory.py

@@ -13,15 +13,15 @@ class ApiFactory:
     def get_api(api_name: str):
         try:
             api_module = importlib.import_module(f"api.{api_name}.{api_name.lower()}")
-            log.debug(f"{__class__.__name__}: Imported API module: {api_module}")
+            log.debug("%s: Imported API module: %s", __class__.__name__, api_module)
             api_class = getattr(api_module, api_name) # TODO Unused api_class
             settings_class = getattr(api_module, f"{api_name}Settings") # TODO Unused settings_class
             widget_class = getattr(api_module, f"{api_name}Widget")
             load_widget(os.path.join(os.path.dirname(api_module.__file__), f"{api_name.lower()}.kv"))
             return widget_class()
         except (ModuleNotFoundError, AttributeError) as e:
-            log.error(f"{__class__.__name__}: Error loading API {api_name}: {e}")
-            log.debug(f"{__class__.__name__}: {traceback.format_exc()}")
+            log.error("%s: Error loading API {api_name}: {e}", __class__.__name__, api_name=api_name, e=e)
+            log.debug("%s: %s", __class__.__name__, traceback.format_exc())
             return None
 
 # Dynamic loading of APIs based on directory structure

src/main.py

@@ -23,6 +23,8 @@
 from settings.app_settings import GlobalSettings 
 from api.api_factory import load_apis
 
+# FIXME This folder shall always be created in the location of the executable, but currently will always be created in the current working directory
+# NOTE Need to differentiate launch environment (Release vs. Debug) - where one gets executed via python, the other via deployed executable
 TMP_FOLDER = 'tmp'
 
 class TrailingPressedIconButton(
@@ -81,7 +83,7 @@ def tap_expansion_chevron(
             # panel.ids.content.height += child.height if not panel.is_open else -child.height
             # panel.parent.height += child.height if not panel.is_open else -child.height
             # panel.parent.parent.height += child.height if not panel.is_open else -child.height
-         os.makedirs(TMP_FOLDER, exist_ok=True)
+        os.makedirs(TMP_FOLDER, exist_ok=True)
 
 if __name__ == '__main__':
     if hasattr(sys, '_MEIPASS'):

src/modules/dialog/loaddialog.py

@@ -19,15 +19,15 @@ def on_browser_select(self, selection):
         if len(self.filechooser.selection) == 0:
             return
         selection = selection[0]
-        log.debug(f"Selection: {selection}")
+        log.debug("Selection: %s", selection)
         self.label.text = selection
 
     def on_fileload(self):
         if len(self.filechooser.selection) == 0:
             log.info("No file selected")
             return
         selection = self.filechooser.selection[0]
-        log.info(f"Selected file: {selection}")
+        log.info("Selected file: %s", selection)
         self.dismiss()
         self.callback(selection)
         self.label.text = ""

src/modules/dialog/savedialog.py

@@ -18,7 +18,7 @@ def __init__(self, callback, **kwargs):
     def on_browser_select(self, selection):
         if len(selection) == 0:
             return
-        log.debug(f"File selected: {selection[0]}")
+        log.debug("File selected: %s", selection[0])
         self.filename_input.text = os.path.basename(selection[0])
 
     def on_filesave(self, path, filename):