pymmcore-plus
diff --git a/‎examples/config_groups_editor.py‎
Lines changed: 17 additions & 0 deletions b/‎examples/config_groups_editor.py‎
Lines changed: 17 additions & 0 deletions
diff --git a/‎src/pymmcore_widgets/__init__.py‎
Lines changed: 2 additions & 0 deletions b/‎src/pymmcore_widgets/__init__.py‎
Lines changed: 2 additions & 0 deletions
diff --git a/‎src/pymmcore_widgets/_help/__init__.py‎ b/‎src/pymmcore_widgets/_help/__init__.py‎
diff --git a/‎src/pymmcore_widgets/_help/_config_groups_help.py‎
Lines changed: 42 additions & 0 deletions b/‎src/pymmcore_widgets/_help/_config_groups_help.py‎
Lines changed: 42 additions & 0 deletions
diff --git a/‎src/pymmcore_widgets/_help/config_groups_help.html‎
Lines changed: 90 additions & 0 deletions b/‎src/pymmcore_widgets/_help/config_groups_help.html‎
Lines changed: 90 additions & 0 deletions
diff --git a/‎src/pymmcore_widgets/_icons.py‎
Lines changed: 31 additions & 1 deletion b/‎src/pymmcore_widgets/_icons.py‎
Lines changed: 31 additions & 1 deletion
diff --git a/‎src/pymmcore_widgets/_log.py‎
Lines changed: 1 addition & 1 deletion b/‎src/pymmcore_widgets/_log.py‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/pymmcore_widgets/_models/__init__.py‎
Lines changed: 2 additions & 0 deletions b/‎src/pymmcore_widgets/_models/__init__.py‎
Lines changed: 2 additions & 0 deletions
diff --git a/‎src/pymmcore_widgets/_models/_base_tree_model.py‎
Lines changed: 42 additions & 11 deletions b/‎src/pymmcore_widgets/_models/_base_tree_model.py‎
Lines changed: 42 additions & 11 deletions
@@ -0,0 +1,17 @@
+import sys
+
+from pymmcore_plus import CMMCorePlus
+from qtpy.QtWidgets import QApplication
+
+from pymmcore_widgets import ConfigGroupsEditor
+
+app = QApplication([])
+core = CMMCorePlus()
+core.loadSystemConfiguration()
+
+with_cfg = sys.argv[1] in ("1", "true") if len(sys.argv) > 1 else False
+cfg = ConfigGroupsEditor.create_from_core(core, load_configs=with_cfg)
+cfg.resize(900, 550)
+cfg.show()
+
+app.exec()
@@ -17,6 +17,7 @@
     "ChannelGroupWidget",
     "ChannelTable",
     "ChannelWidget",
+    "ConfigGroupsEditor",
     "ConfigGroupsTree",
     "ConfigWizard",
     "ConfigurationWidget",
@@ -52,6 +53,7 @@
 from ._install_widget import InstallWidget
 from ._log import CoreLogWidget
 from .config_presets import (
+    ConfigGroupsEditor,
     ConfigGroupsTree,
     GroupPresetTableWidget,
     ObjectivesPixelConfigurationWidget,
 
@@ -0,0 +1,42 @@
+from pathlib import Path
+
+from qtpy.QtWidgets import (
+    QDialog,
+    QDialogButtonBox,
+    QFrame,
+    QTextBrowser,
+    QVBoxLayout,
+    QWidget,
+)
+
+HELP_DOC = Path(__file__).parent / "config_groups_help.html"
+
+
+class ConfigGroupsHelpDialog(QDialog):
+    """Help dialog for the Config Groups Editor."""
+
+    def __init__(self, parent: QWidget | None = None):
+        super().__init__(parent)
+        self.setWindowTitle("Config Groups and Presets")
+        self.setModal(True)
+
+        # Add help content here
+        help_text = QTextBrowser(self)
+        help_text.setReadOnly(True)
+        help_text.setAcceptRichText(True)
+        help_text.setHtml(HELP_DOC.read_text())
+        help_text.setFrameShape(QFrame.Shape.NoFrame)
+        # enable links
+        help_text.setOpenExternalLinks(True)
+
+        # make the background match the dialog
+        pal = self.palette()
+        pal.setColor(pal.ColorRole.Base, pal.color(pal.ColorRole.Window))
+        help_text.setPalette(pal)
+
+        btn_box = QDialogButtonBox(QDialogButtonBox.StandardButton.Ok)
+        btn_box.accepted.connect(self.accept)
+
+        layout = QVBoxLayout(self)
+        layout.addWidget(help_text)
+        layout.addWidget(btn_box)
@@ -0,0 +1,90 @@
+<!DOCTYPE html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <title>Micro-Manager Configuration Groups and Presets - Quick Guide</title>
+  </head>
+  <body>
+    <h1>Configuration Groups and Presets</h1>
+
+    <p>
+      Micro-Manager lets you bundle hardware settings into
+      <a
+        href="https://micro-manager.org/Micro-Manager_User%27s_Guide#controlling-devices"
+        ><strong>Groups of Configuration Presets</strong></a
+      >.
+    </p>
+
+    <p>
+      Each <strong>Group</strong> contains named <strong>Presets</strong> that
+      define a set of <strong>Device Property values</strong> to be applied to
+      the hardware.
+    </p>
+
+    <h2>Typical Workflow</h2>
+
+    <ol>
+      <li>
+        <strong>Add a configuration group</strong> (e.g. "Channel") using the
+        <em>Add Group</em> button at the top.
+      </li>
+      <li>
+        <strong>Add a preset to the group</strong> (e.g. "DAPI"), by selecting
+        the group and clicking the <em>Add Preset</em> button.
+      </li>
+      <li>
+        <strong>Select device properties to include in the preset</strong> (e.g.
+        "Filter Wheel Position", "Light Source Power") using the device property
+        selector at the right.
+      </li>
+      <li>
+        <strong>Set the device properties to desired values</strong> for the
+        currently selected preset in the Presets Table at the bottom.
+      </li>
+      <li>
+        <strong>Repeat steps 2-4 to add more presets to the group</strong> (e.g.
+        "FITC", "Cy3", "Cy5"). You may <em>Duplicate</em> an existing preset to
+        copy its device property values if another preset is similar.
+      </li>
+    </ol>
+
+    <h2>Key Building Blocks</h2>
+
+    <h3>1. Devices</h3>
+    <p>
+      Physical hardware (camera, filter wheel, stage, light source). Each device
+      exposes one or more <strong>properties</strong> that can be changed or
+      read.
+    </p>
+
+    <h3>2. Device Properties</h3>
+    <p>
+      A single adjustable parameter on a device (e.g.
+      <code>Exposure</code>, <code>Position</code>).
+    </p>
+
+    <h3>3. Configuration Presets</h3>
+    <p>
+      A preset is a snapshot of <em>specific</em> values for a set of device
+      <strong>properties</strong>. Activating a preset sets each device property
+      to its stored value.
+    </p>
+    <p>
+      For example, you might have a <em>FITC</em> preset that sets the
+      appropriate filter wheel positions and light sources for imaging the FITC
+      channel.
+    </p>
+
+    <h3>4. Configuration Groups</h3>
+    <p>
+      A configuration group is a <strong>collection of presets</strong>. It is
+      customary (<em>but not mandatory</em>) for each preset in a group to share
+      the same set of device properties.
+    </p>
+    <p>
+      For example, you might have a <em>Channel</em> group that includes presets
+      for each optical configuration preset (<em>DAPI</em>, <em>FITC</em>,
+      <em>Cy3</em>, <em>Cy5</em>).
+    </p>
+  </body>
+</html>
@@ -2,7 +2,7 @@
 
 from enum import Enum
 
-from pymmcore_plus import CMMCorePlus, DeviceType
+from pymmcore_plus import CMMCorePlus, DeviceType, PropertyType
 from superqt import QIconifyIcon
 
 
@@ -15,6 +15,7 @@ class StandardIcon(str, Enum):
     TREE = "ph:tree-view"
     FOLDER_ADD = "fluent:folder-add-24-regular"
     DOCUMENT_ADD = "fluent:document-add-24-regular"
+    PROPERTY_ADD = "fluent:form-new-24-regular"
     DELETE = "fluent:delete-24-regular"
     COPY = "fluent:save-copy-24-regular"
     TRANSPOSE = "carbon:transpose"
@@ -28,6 +29,13 @@ class StandardIcon(str, Enum):
     UNDO = "mdi:undo"
     REDO = "mdi:redo"
 
+    PROP_STRING = "mdi:alphabetical-variant"
+    PROP_INTEGER = "mdi:numeric"
+    PROP_FLOAT = "mdi:decimal"
+    PROP_ENUM = "mdi:format-list-bulleted"
+    PROP_BOOLEAN = "mdi:toggle-switch-outline"
+    PROP_UNDEF = "mdi:help-circle-outline"
+
     DEVICE_ANY = "mdi:devices"
     DEVICE_AUTOFOCUS = "mdi:focus-auto"
     DEVICE_CAMERA = "mdi:camera"
@@ -67,6 +75,28 @@ def for_device_type(cls, device_type: DeviceType | str) -> StandardIcon:
 
         return _DEVICE_TYPE_MAP.get(device_type, StandardIcon.DEVICE_UNKNOWN)
 
+    @classmethod
+    def for_property_type(
+        cls,
+        prop_type: PropertyType,
+        allowed: tuple[str, ...] = (),
+    ) -> StandardIcon:
+        """Return an icon for a specific property type."""
+        if prop_type is PropertyType.Integer and set(allowed) == {"0", "1"}:
+            return StandardIcon.PROP_BOOLEAN
+        if allowed:
+            return StandardIcon.PROP_ENUM
+        return _PROPERTY_TYPE_MAP.get(prop_type, StandardIcon.PROP_UNDEF)
+
+
+_PROPERTY_TYPE_MAP: dict[PropertyType, StandardIcon] = {
+    PropertyType.String: StandardIcon.PROP_STRING,
+    PropertyType.Integer: StandardIcon.PROP_INTEGER,
+    PropertyType.Float: StandardIcon.PROP_FLOAT,
+    PropertyType.Enum: StandardIcon.PROP_ENUM,
+    PropertyType.Boolean: StandardIcon.PROP_BOOLEAN,
+    PropertyType.Undef: StandardIcon.PROP_UNDEF,
+}
 
 _DEVICE_TYPE_MAP: dict[DeviceType, StandardIcon] = {
     DeviceType.Any: StandardIcon.DEVICE_ANY,
 
@@ -122,7 +122,7 @@ def __init__(
         mmcore: CMMCorePlus | None = None,
     ) -> None:
         super().__init__(parent)
-        self._mmcore = mmcore or CMMCorePlus().instance()
+        self._mmcore = mmcore or CMMCorePlus.instance()
         self.setWindowTitle("Log Console")
 
         # --- Log path ---
 
@@ -16,6 +16,7 @@
     PixelSizePreset,
 )
 from ._q_config_model import QConfigGroupsModel
+from ._q_device_prop_model import QDevicePropertyModel
 
 __all__ = [
     "ConfigGroup",
@@ -26,6 +27,7 @@
     "PixelSizeConfigs",
     "PixelSizePreset",
     "QConfigGroupsModel",
+    "QDevicePropertyModel",
     "get_available_devices",
     "get_config_groups",
     "get_config_presets",
 
@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import itertools
 from typing import overload
 
 from qtpy.QtCore import QAbstractItemModel, QModelIndex, QObject, Qt
@@ -9,11 +10,14 @@
 
 NULL_INDEX = QModelIndex()
 
+_node_id_counter = itertools.count()
+
 
 class _Node:
     """Generic tree node that wraps a ConfigGroup, ConfigPreset, or Setting."""
 
     __slots__ = (
+        "_id",
         "check_state",
         "children",
         "name",
@@ -59,6 +63,7 @@ def __init__(
         | None = None,
         parent: _Node | None = None,
     ) -> None:
+        self._id: int = next(_node_id_counter)
         self.name = name
         self.payload = payload
         self.parent = parent
@@ -104,7 +109,12 @@ def is_device(self) -> bool:
 
 
 class _BaseTreeModel(QAbstractItemModel):
-    """Thin abstract tree model.
+    """Thin abstract tree model using integer IDs for createIndex.
+
+    Uses the integer-ID pattern: each _Node gets a unique int ID, stored in a
+    registry dict.  createIndex() receives the int (not the Python object), so
+    stale QModelIndex / QPersistentModelIndex objects resolve to a safe dict
+    lookup instead of a dangling void* pointer.
 
     Sub-classes should implement at least the following methods:
 
@@ -117,16 +127,36 @@ class _BaseTreeModel(QAbstractItemModel):
     def __init__(self, parent: QObject | None = None) -> None:
         super().__init__(parent)
         self._root = _Node("<root>", None)
+        self._node_registry: dict[int, _Node] = {}
+
+    def _create_node(
+        self,
+        payload: ConfigGroup | ConfigPreset | DevicePropertySetting | Device,
+        parent: _Node | None = None,
+        recursive: bool = True,
+    ) -> _Node:
+        """Create a _Node, register it and all descendants, and return it."""
+        node = _Node.create(payload, parent, recursive=recursive)
+        self._register_tree(node)
+        return node
+
+    def _register_tree(self, node: _Node) -> None:
+        """Register *node* and all its descendants in the node registry."""
+        self._node_registry[node._id] = node
+        for child in node.children:
+            self._register_tree(child)
+
+    def _unregister_tree(self, node: _Node) -> None:
+        """Remove *node* and all its descendants from the node registry."""
+        self._node_registry.pop(node._id, None)
+        for child in node.children:
+            self._unregister_tree(child)
 
     def _node_from_index(self, index: QModelIndex | None) -> _Node:
-        if (
-            index
-            and index.isValid()
-            and isinstance((node := index.internalPointer()), _Node)
-        ):
-            # return the node if index is valid
-            return node
-        # otherwise return the root node
+        if index and index.isValid():
+            node = self._node_registry.get(index.internalId())
+            if node is not None:
+                return node
         return self._root
 
     # # ---------- Qt plumbing ----------------------------------------------
@@ -143,7 +173,8 @@ def index(
         """Return the index of the item specified by row, column and parent index."""
         parent_node = self._node_from_index(parent)
         if 0 <= row < len(parent_node.children):
-            return self.createIndex(row, column, parent_node.children[row])
+            child = parent_node.children[row]
+            return self.createIndex(row, column, child._id)
         return QModelIndex()  # pragma: no cover
 
     @overload
@@ -167,4 +198,4 @@ def parent(self, child: QModelIndex | None = None) -> QModelIndex | QObject | No
 
         # A common convention used in models that expose tree data structures is that
         # only items in the first column have children.
-        return self.createIndex(parent_node.row_in_parent(), 0, parent_node)
+        return self.createIndex(parent_node.row_in_parent(), 0, parent_node._id)