processforge 0.3.19__tar.gz → 0.3.20__tar.gz

{processforge-0.3.19/src/processforge.egg-info → processforge-0.3.20}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: processforge
-Version: 0.3.19
+Version: 0.3.20
 Summary: A Python-based process simulation framework for chemical engineering applications.
 Author-email: Processforge Team <team@processforge.dev>
 License-Expression: BSD-3-Clause

{processforge-0.3.19 → processforge-0.3.20}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "processforge"
-version = "0.3.19"
+version = "0.3.20"
 description = "A Python-based process simulation framework for chemical engineering applications."
 readme = "README.md"
 license = "BSD-3-Clause"

{processforge-0.3.19 → processforge-0.3.20}/src/processforge/cli/run.py

@@ -102,6 +102,7 @@ def run(
             raise SystemExit(1)
 
         run_info = build_run_info(config, x0=x0, var_names=var_names)
+        snap_x, snap_vn, snap_backend, snap_ok = x0, var_names, "dynamic", True
     else:
         # Pass None so EOFlowsheet resolves backend from config (with scipy default).
         fs = EOFlowsheet(config, backend=None)
@@ -117,6 +118,9 @@ def run(
                 logger.warning("Steady-state simulation did NOT converge. Results may be unreliable.")
 
         run_info = build_run_info(config, x0=fs.x0, var_names=fs.var_names)
+        snap_x, snap_vn = fs.x_converged, fs.var_names
+        snap_backend = getattr(fs, "backend", "scipy")
+        snap_ok = bool(getattr(fs, "converged", False))
 
     # Propagate run context to any containerized providers so their S3 uploads
     # are keyed by the same run_id/flowsheet_hash as this archive run.
@@ -134,6 +138,21 @@ def run(
     }
     archive.save_run(manifest, stream_results=stream_results)
 
+    # Persist a StateManager snapshot so `pf plan` (and `pf apply`) can diff
+    # against this run as a baseline. Without it, `pf plan` always reports
+    # "No prior state found" and never surfaces flowsheet edits.
+    if (is_dynamic or snap_ok) and snap_x is not None:
+        try:
+            from .common import build_run_metadata, save_snapshot
+
+            meta = build_run_metadata(config, 1e-6, 50, snap_backend)
+            save_snapshot(
+                archive, config, snap_x, snap_vn,
+                metadata=meta, parent_snapshot_id=None, label="run state",
+            )
+        except Exception as e:
+            logger.warning(f"Failed to save state snapshot: {type(e).__name__}: {e}")
+
     # Always persist a Zarr copy of the standardized outputs (fields + artifacts)
     # inside the archive. Works for every provider (OpenMC, CoolProp, FESTIM,
     # Cantera) and for both local and remote docker runs — remote artifacts

{processforge-0.3.19 → processforge-0.3.20}/src/processforge/state.py

@@ -302,27 +302,47 @@ class StateManager:
                 if ov != nv:
                     changes.append(f"{key}: {ov!r} → {nv!r}")
 
-            # Parameter changes (scalar unit params, not nested "parameters" dict)
-            all_keys = set(old_cfg) | set(new_cfg)
-            skip = {"type", "in", "out", "out_vap", "out_liq", "provider", "parameters"}
-            for k in all_keys - skip:
-                ov, nv = old_cfg.get(k), new_cfg.get(k)
-                if ov != nv:
-                    changes.append(f"{k}: {ov} → {nv}")
-
-            # Nested "parameters" dict
-            old_params = old_cfg.get("parameters", {})
-            new_params = new_cfg.get("parameters", {})
-            for pk in set(old_params) | set(new_params):
-                ov, nv = old_params.get(pk), new_params.get(pk)
-                if ov != nv:
-                    changes.append(f"parameters.{pk}: {ov} → {nv}")
+            # Parameter changes — recurse into nested dicts so leaf values
+            # (e.g. solver_config.batches) are reported individually rather
+            # than dumping the whole dict.
+            changes.extend(
+                StateManager._diff_params(old_cfg, new_cfg)
+            )
 
             if changes:
                 modified[name] = {"type": new_cfg.get("type", "?"), "changes": changes}
 
         return {"added": added, "removed": removed, "modified": modified}
 
+    # ------------------------------------------------------------------
+    # Structural-diff helpers
+    # ------------------------------------------------------------------
+    @staticmethod
+    def _diff_params(
+        old_cfg: dict, new_cfg: dict, prefix: str = "", skip: set | None = None
+    ) -> list[str]:
+        """Recursively collect changed scalar/leaf params between two dicts.
+
+        Nested dicts are descended (``solver_config.batches: 25 → 999``) so the
+        structural diff reports leaf-level changes instead of whole-dict blobs.
+        """
+        if skip is None:
+            skip = {"type", "in", "out", "out_vap", "out_liq", "provider"}
+        all_keys = set(old_cfg) | set(new_cfg)
+        if not prefix:
+            all_keys -= skip
+        changes: list[str] = []
+        for k in all_keys:
+            ov, nv = old_cfg.get(k), new_cfg.get(k)
+            label = f"{prefix}{k}"
+            if isinstance(ov, dict) and isinstance(nv, dict):
+                changes.extend(
+                    StateManager._diff_params(ov, nv, prefix=f"{label}.", skip=set())
+                )
+            elif ov != nv:
+                changes.append(f"{label}: {ov} → {nv}")
+        return changes
+
     # ------------------------------------------------------------------
     # State → stream dict conversion (for dynamic t=0 initialisation)
     # ------------------------------------------------------------------

{processforge-0.3.19 → processforge-0.3.20/src/processforge.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: processforge
-Version: 0.3.19
+Version: 0.3.20
 Summary: A Python-based process simulation framework for chemical engineering applications.
 Author-email: Processforge Team <team@processforge.dev>
 License-Expression: BSD-3-Clause