siliconcompiler 0.35.4__py3-none-any.whl → 0.36.1__py3-none-any.whl

siliconcompiler/utils/settings.py

@@ -1,6 +1,7 @@
 import json
 import os
 import logging
+import threading
 
 import os.path
 
@@ -24,9 +25,14 @@ class SettingsManager:
 
         Args:
             filepath (str): The path to the JSON file where settings are stored.
+                If None, settings are kept in memory only.
+            logger (logging.Logger): Logger for logging errors and information.
+            timeout (float): Timeout in seconds for acquiring the file lock.
         """
         self.__filepath = filepath
-        self.__lock = InterProcessLock(self.__filepath + ".lock")
+        if self.__filepath is not None:
+            self.__lock = InterProcessLock(self.__filepath + ".lock")
+        self.__settings_lock = threading.Lock()
         self.__timeout = timeout
         self.__logger = logger.getChild("settings")
         self.__settings = {}
@@ -37,58 +43,63 @@ class SettingsManager:
         Internal method to load settings from disk.
         It handles missing files and malformed JSON gracefully.
         """
-        if not os.path.exists(self.__filepath):
+        if self.__filepath is None or not os.path.exists(self.__filepath):
             self.__settings = {}
             return
 
-        try:
-            if self.__lock.acquire(timeout=self.__timeout):
-                try:
-                    with sc_open(self.__filepath, encoding='utf-8') as f:
-                        data = json.load(f)
-                finally:
-                    self.__lock.release()
-            else:
-                self.__logger.error(f"Timeout acquiring lock for {self.__filepath}. "
+        with self.__settings_lock:
+            try:
+                if self.__lock.acquire(timeout=self.__timeout):
+                    try:
+                        with sc_open(self.__filepath, encoding='utf-8') as f:
+                            data = json.load(f)
+                    finally:
+                        self.__lock.release()
+                else:
+                    self.__logger.error(f"Timeout acquiring lock for {self.__filepath}. "
+                                        "Starting with empty settings.")
+                    data = {}
+
+                # Ensure the loaded data is actually a dictionary
+                if isinstance(data, dict):
+                    self.__settings = data
+                else:
+                    # If valid JSON but not a dict (e.g. a list), reset to empty
+                    self.__logger.warning(f"File {self.__filepath} did not contain a JSON object. "
+                                          "Resetting.")
+                    self.__settings = {}
+
+            except json.JSONDecodeError:
+                self.__logger.error(f"File {self.__filepath} is malformed. "
                                     "Starting with empty settings.")
-                data = {}
-
-            # Ensure the loaded data is actually a dictionary
-            if isinstance(data, dict):
-                self.__settings = data
-            else:
-                # If valid JSON but not a dict (e.g. a list), reset to empty
-                self.__logger.warning(f"File {self.__filepath} did not contain a JSON object. "
-                                      "Resetting.")
                 self.__settings = {}
-
-        except json.JSONDecodeError:
-            self.__logger.error(f"File {self.__filepath} is malformed. "
-                                "Starting with empty settings.")
-            self.__settings = {}
-        except Exception as e:
-            # Catch-all for permission errors, etc., to ensure __init__ doesn't crash
-            self.__logger.error(f"Unexpected error loading settings: {e}")
-            self.__settings = {}
+            except Exception as e:
+                # Catch-all for permission errors, etc., to ensure __init__ doesn't crash
+                self.__logger.error(f"Unexpected error loading settings: {e}")
+                self.__settings = {}
 
     def save(self):
         """
         Save the current settings to the disk in JSON format.
         """
+        if self.__filepath is None:
+            return
+
         try:
             # Ensure directory exists
             directory = os.path.dirname(self.__filepath)
             if directory and not os.path.exists(directory):
                 os.makedirs(directory)
 
-            with self.__lock:
-                with open(self.__filepath, 'w', encoding='utf-8') as f:
-                    json.dump(self.__settings, f, indent=4)
+            with self.__settings_lock:
+                with self.__lock:
+                    with open(self.__filepath, 'w', encoding='utf-8') as f:
+                        json.dump(self.__settings, f, indent=4)
         except Exception as e:
             self.__logger.error(f"Failed to save settings to {self.__filepath}: {e}")
             raise e
 
-    def set(self, category: str, key: str, value):
+    def set(self, category: str, key: str, value, keep: bool = False):
         """
         Set a specific setting within a category.
 
@@ -96,11 +107,15 @@ class SettingsManager:
             category (str): The group name (e.g., 'showtools', 'options').
             key (str): The specific setting name.
             value: The value to store (must be JSON serializable).
+            keep (bool): If True, do not overwrite existing value.
         """
-        if category not in self.__settings:
-            self.__settings[category] = {}
+        with self.__settings_lock:
+            if category not in self.__settings:
+                self.__settings[category] = {}
 
-        self.__settings[category][key] = value
+            if keep and key in self.__settings[category]:
+                return
+            self.__settings[category][key] = value
 
     def get(self, category: str, key: str, default=None):
         """
@@ -114,28 +129,34 @@ class SettingsManager:
         Returns:
             The stored value or the default.
         """
-        if category not in self.__settings:
-            return default
+        with self.__settings_lock:
+            if category not in self.__settings:
+                return default
 
-        return self.__settings[category].get(key, default)
+            return self.__settings[category].get(key, default)
 
     def get_category(self, category: str):
         """
         Retrieve all settings for a specific category.
         Returns an empty dict if category does not exist.
         """
-        return self.__settings.get(category, {})
+        with self.__settings_lock:
+            if category not in self.__settings:
+                self.__settings[category] = {}
+
+            return self.__settings.get(category).copy()
 
     def delete(self, category: str, key: Optional[str] = None):
         """
         Remove a setting.
         """
-        if category in self.__settings:
-            if key:
-                if key in self.__settings[category]:
-                    del self.__settings[category][key]
-                    # Clean up empty categories
-                    if not self.__settings[category]:
-                        del self.__settings[category]
-            else:
-                del self.__settings[category]
+        with self.__settings_lock:
+            if category in self.__settings:
+                if key:
+                    if key in self.__settings[category]:
+                        del self.__settings[category][key]
+                        # Clean up empty categories
+                        if not self.__settings[category]:
+                            del self.__settings[category]
+                else:
+                    del self.__settings[category]

{siliconcompiler-0.35.4.dist-info → siliconcompiler-0.36.1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: siliconcompiler
-Version: 0.35.4
+Version: 0.36.1
 Summary: A compiler framework that automates translation from source code to silicon.
 Author: Zero ASIC
 License: Apache License 2.0
@@ -31,14 +31,14 @@ Requires-Dist: docker<8.0.0,>=7.1.0
 Requires-Dist: fastjsonschema<2.22.0,>=2.20.0
 Requires-Dist: graphviz<0.22,>=0.20
 Requires-Dist: orjson<4,>=3.11.0
-Requires-Dist: packaging<25,>=21.3
+Requires-Dist: packaging<26,>=21.3
 Requires-Dist: requests<2.33.0,>=2.32.0
 Requires-Dist: Pillow<11.4.0,>=11.3.0
 Requires-Dist: PyYAML<7.0.0,>=6.0.0
 Requires-Dist: GitPython<3.2,>=3.1.44
 Requires-Dist: PyGithub<2.9.0,>=2.8.0
 Requires-Dist: urllib3>=1.26.0
-Requires-Dist: lambdapdk>=0.2.4
+Requires-Dist: lambdapdk>=0.2.6
 Requires-Dist: fasteners>=0.20
 Requires-Dist: pandas>=1.1.5
 Requires-Dist: psutil>=5.8.0
@@ -61,7 +61,7 @@ Requires-Dist: responses==0.25.8; extra == "test"
 Requires-Dist: PyVirtualDisplay==3.0; extra == "test"
 Provides-Extra: lint
 Requires-Dist: flake8==7.3.0; extra == "lint"
-Requires-Dist: tclint==0.6.2; extra == "lint"
+Requires-Dist: tclint==0.7.0; extra == "lint"
 Requires-Dist: codespell==2.4.1; extra == "lint"
 Provides-Extra: docs
 Requires-Dist: Sphinx<8.3,>=7.4; extra == "docs"