MoleditPy-linux 4.2.1__tar.gz → 4.2.2__tar.gz

{moleditpy_linux-4.2.1 → moleditpy_linux-4.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: MoleditPy-linux
-Version: 4.2.1
+Version: 4.2.2
 Summary: A cross-platform, simple, and intuitive molecular structure editor built in Python. It allows 2D molecular drawing and 3D structure visualization. It supports exporting structure files for input to DFT calculation software.
 Author-email: HiroYokoyama <titech.yoko.hiro@gmail.com>
 License:                     GNU GENERAL PUBLIC LICENSE

{moleditpy_linux-4.2.1 → moleditpy_linux-4.2.2}/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "setuptools.build_meta"
 [project]
 name = "MoleditPy-linux"
 
-version = "4.2.1"
+version = "4.2.2"
 
 license = {file = "LICENSE"}
 

{moleditpy_linux-4.2.1 → moleditpy_linux-4.2.2}/src/MoleditPy_linux.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: MoleditPy-linux
-Version: 4.2.1
+Version: 4.2.2
 Summary: A cross-platform, simple, and intuitive molecular structure editor built in Python. It allows 2D molecular drawing and 3D structure visualization. It supports exporting structure files for input to DFT calculation software.
 Author-email: HiroYokoyama <titech.yoko.hiro@gmail.com>
 License:                     GNU GENERAL PUBLIC LICENSE

{moleditpy_linux-4.2.1 → moleditpy_linux-4.2.2}/src/moleditpy_linux/ui/compute_logic.py

@@ -45,6 +45,9 @@ class ComputeManager:
         self.halt_ids: Set[int] = set()
         self.next_conversion_id: int = 1
         self.active_worker_ids: Set[int] = set()
+        # worker_id -> (method_key, plugin_entry) for a plugin optimizer to run
+        # as a post-step once conversion's built-in pre-optimization completes.
+        self._pending_plugin_opt: Dict[int, Tuple[str, Dict[str, Any]]] = {}
 
     def reset_active_threads(self) -> None:
         """Reset active calculation threads list."""
@@ -239,6 +242,25 @@ class ComputeManager:
         self.next_conversion_id = run_id + 1
         self.active_worker_ids.add(run_id)
 
+        # The worker only runs built-in force fields. If the selected method is a
+        # plugin optimizer, pre-optimize with MMFF (RDKit) in the worker (which
+        # auto-falls-back to UFF if MMFF setup fails), then run the plugin
+        # callback as a post-step in on_calculation_finished.
+        effective_method = (
+            optimization_method
+            or self.host.init_manager.optimization_method
+            or "MMFF_RDKIT"
+        ).upper()
+        _reg = getattr(
+            getattr(self.host, "plugin_manager", None), "optimization_methods", None
+        )
+        plugin_entry = _reg.get(effective_method) if isinstance(_reg, dict) else None
+        if plugin_entry:
+            self._pending_plugin_opt[run_id] = (effective_method, plugin_entry)
+            worker_method = "MMFF_RDKIT"
+        else:
+            worker_method = effective_method
+
         # UI Updates
         self.host.init_manager.convert_button.setText("Halt conversion")  # type: ignore[union-attr]
         self._safe_disconnect(self.host.init_manager.convert_button.clicked)  # type: ignore[union-attr]
@@ -267,8 +289,7 @@ class ComputeManager:
         options = {
             "conversion_mode": conversion_mode
             or self.host.init_manager.settings.get("3d_conversion_mode", "fallback"),
-            "optimization_method": optimization_method
-            or self.host.init_manager.optimization_method,
+            "optimization_method": worker_method,
             "optimize_intermolecular_interaction_rdkit": self.host.init_manager.settings.get(
                 "optimize_intermolecular_interaction_rdkit", True
             ),
@@ -285,6 +306,7 @@ class ComputeManager:
         if wids_to_halt:
             self.halt_ids.update(wids_to_halt)
         self.active_worker_ids.clear()
+        self._pending_plugin_opt.clear()
 
         self._restore_button_ui()
         self.host.init_manager.cleanup_button.setEnabled(True)  # type: ignore[union-attr]
@@ -529,6 +551,16 @@ class ComputeManager:
         self.host.view_3d_manager.update_atom_id_menu_text()
         self.host.view_3d_manager.update_atom_id_menu_state()
 
+        # If the selected method was a plugin optimizer, the worker only did the
+        # built-in pre-optimization; run the plugin callback now on the result.
+        pending = (
+            self._pending_plugin_opt.pop(worker_id, None)
+            if worker_id is not None
+            else None
+        )
+        if pending and mol:
+            self._run_plugin_optimization(*pending)
+
     def on_calculation_error(self, message: Union[str, Tuple[int, str]]) -> None:
         """Handle an error or halt signal from the background optimization worker."""
         # Accept either a string or (worker_id, message) tuple from the worker signal
@@ -547,8 +579,10 @@ class ComputeManager:
                     self.host.statusBar().showMessage(  # type: ignore[union-attr]
                         f"Ignored stale worker error: {msg} (ID = {worker_id})"
                     )
+                self._pending_plugin_opt.pop(worker_id, None)
                 return  # stale worker, ignore
             self.active_worker_ids.discard(worker_id)
+            self._pending_plugin_opt.pop(worker_id, None)
         else:
             msg = str(message)
 

{moleditpy_linux-4.2.1 → moleditpy_linux-4.2.2}/src/moleditpy_linux/ui/main_window_init.py

@@ -1723,6 +1723,13 @@ class MainInitManager:
         self.opt_group.addAction(action)
         self.opt3d_actions[key_upper] = action
 
+        # Plugins register after menu-init, so restore the saved default here
+        # if it names this method (menu-init fell back to MMFF_RDKIT).
+        saved_opt = (self.settings.get("optimization_method") or "").upper()
+        if key_upper == saved_opt:
+            action.setChecked(True)
+            self.optimization_method = key_upper
+
     def _init_help_menu(self, menu_bar: Any) -> None:
         """Initialize the Help menu."""
         help_menu = menu_bar.addMenu("&Help")

{moleditpy_linux-4.2.1 → moleditpy_linux-4.2.2}/src/moleditpy_linux/ui/plugin_menu_manager.py

@@ -369,14 +369,30 @@ class PluginMenuManager:
                 menu.addAction(a)
 
     def integrate_plugin_optimization_methods(self) -> None:
-        """Inject plugin optimization methods into the 3D Optimization Settings menu."""
-        if not hasattr(self._im.host.plugin_manager, "optimization_methods"):
+        """Inject plugin optimization methods into the 3D Optimization Settings menu.
+
+        Idempotent: previously-added plugin methods are purged first, so a menu
+        rebuild (install/uninstall/reload) re-adds only the currently registered
+        ones. Without this, _clean_menu removes the menu action while
+        opt3d_actions still holds it, and add_optimization_method's dedupe guard
+        would then skip re-adding a still-installed method.
+        """
+        im = self._im
+        methods = getattr(im.host.plugin_manager, "optimization_methods", None)
+        if not isinstance(methods, dict):
             return
 
-        for key, entry in self._im.host.plugin_manager.optimization_methods.items():
-            self._im.add_optimization_method(entry["label"], key)
-            if key in self._im.opt3d_actions:
-                self._im.opt3d_actions[key].setData("plugin_managed")
+        for key, action in list(im.opt3d_actions.items()):
+            if action.data() == "plugin_managed":
+                im.opt_group.removeAction(action)
+                im.optimization_menu.removeAction(action)
+                del im.opt3d_actions[key]
+                im.opt3d_method_labels.pop(key, None)
+
+        for key, entry in methods.items():
+            im.add_optimization_method(entry["label"], key)
+            if key in im.opt3d_actions:
+                im.opt3d_actions[key].setData("plugin_managed")
 
     def integrate_plugin_file_openers(self) -> None:
         """Inject plugin file-opener entries into the File > Import menu."""