setiastrosuitepro 1.6.1.post1__py3-none-any.whl → 1.6.2__py3-none-any.whl

setiastro/saspro/crop_dialog_pro.py

@@ -271,7 +271,16 @@ class CropDialogPro(QDialog):
     def __init__(self, parent, document):
         super().__init__(parent)
         self.setWindowTitle(self.tr("Crop Tool"))
+        self.setWindowFlag(Qt.WindowType.Window, True)
+        self.setWindowModality(Qt.WindowModality.NonModal)
+        self.setModal(False)
+        self._main = parent
         self.doc = document
+
+        # Connect to active document change signal
+        if hasattr(self._main, "currentDocumentChanged"):
+            self._main.currentDocumentChanged.connect(self._on_active_doc_changed)
+
         self._rect_item: Optional[ResizableRotatableRectItem] = None
         self._pix_item: Optional[QGraphicsPixmapItem] = None
         self._drawing = False
@@ -463,6 +472,14 @@ class CropDialogPro(QDialog):
             arr = arr[..., 0]
         return np.clip(arr, 0.0, 1.0)
 
+    def _on_active_doc_changed(self, doc):
+        """Called when user clicks a different image window."""
+        if doc is None or getattr(doc, "image", None) is None:
+            return
+        self.doc = doc
+        self._rect_item = None
+        self._load_from_doc()
+
     def _load_from_doc(self):
         self._full01 = self._img01_from_doc()
         self._orig_h, self._orig_w = self._full01.shape[:2]

setiastro/saspro/curve_editor_pro.py

@@ -1016,7 +1016,16 @@ class CurvesDialogPro(QDialog):
     def __init__(self, parent, document):
         super().__init__(parent)
         self.setWindowTitle(self.tr("Curves Editor"))
+        self.setWindowFlag(Qt.WindowType.Window, True)
+        self.setWindowModality(Qt.WindowModality.NonModal)
+        self.setModal(False)
+        self._main = parent
         self.doc = document
+
+        # Connect to active document change signal
+        if hasattr(self._main, "currentDocumentChanged"):
+            self._main.currentDocumentChanged.connect(self._on_active_doc_changed)
+
         self._preview_img = None     # downsampled float01
         self._full_img = None        # full-res float01
         self._pix = None
@@ -1788,6 +1797,15 @@ class CurvesDialogPro(QDialog):
                 self._rebuild_presets_menu()
 
 
+    # ----- active document change -----
+    def _on_active_doc_changed(self, doc):
+        """Called when user clicks a different image window."""
+        if doc is None or getattr(doc, "image", None) is None:
+            return
+        self.doc = doc
+        self._load_from_doc()
+        QTimer.singleShot(0, self._fit_after_load)
+
     # ----- data -----
     def _load_from_doc(self):
         img = self.doc.image

setiastro/saspro/debayer.py

@@ -402,6 +402,9 @@ class DebayerDialog(QDialog):
     def __init__(self, parent, doc_manager, active_doc):
         super().__init__(parent)
         self.setWindowTitle(self.tr("Debayer"))
+        self.setWindowFlag(Qt.WindowType.Window, True)
+        self.setWindowModality(Qt.WindowModality.NonModal)
+        self.setModal(False)
         self.dm = doc_manager
         self.doc = active_doc
         self.worker: Optional[_DebayerWorker] = None

setiastro/saspro/doc_manager.py

@@ -2472,23 +2472,29 @@ class DocManager(QObject):
         If a Preview tab is selected on the active ImageSubWindow, return a cached
         _RoiViewDocument so tools and the Preview tab share the same instance.
         Otherwise return the real ImageDocument.
+        
+        IMPORTANT: Always check the currently active MDI subwindow first,
+        as that's what the user expects to be the "active" document.
         """
-        # Prefer cached (if set and still valid)
-        if self._active_doc is not None and self._active_doc in self._docs:
-            base_doc = self._active_doc
-        else:
-            base_doc = None
-            try:
-                if self._mdi is not None:
-                    sw = self._mdi.activeSubWindow()
-                    if sw is not None:
-                        w = sw.widget()
-                        base_doc = getattr(w, "document", None) or getattr(sw, "document", None)
-                        if base_doc is not None:
-                            self._active_doc = base_doc
-            except Exception:
-                pass
-            if base_doc is None:
+        base_doc = None
+        
+        # ALWAYS check the MDI active subwindow first - this is the source of truth
+        try:
+            if self._mdi is not None:
+                sw = self._mdi.activeSubWindow()
+                if sw is not None:
+                    w = sw.widget()
+                    base_doc = getattr(w, "document", None) or getattr(sw, "document", None)
+                    if base_doc is not None:
+                        self._active_doc = base_doc
+        except Exception:
+            pass
+        
+        # Fallback to cached value only if MDI lookup failed
+        if base_doc is None:
+            if self._active_doc is not None and self._active_doc in self._docs:
+                base_doc = self._active_doc
+            else:
                 base_doc = self._docs[-1] if self._docs else None
 
         # Non-image docs just pass through
@@ -2524,6 +2530,23 @@ class DocManager(QObject):
 
         # Prefer explicit doc if given; otherwise fall back to "active"
         view_doc = doc or self.get_active_document()
+
+        # DEBUG: Trace why LinearFit might fail
+        # print(f"[DocManager] update_active_document target: {view_doc}, type: {type(view_doc).__name__}")
+
+        # NEW: Unwrap proxy objects (_DocProxy / LiveViewDocument)
+        tname = type(view_doc).__name__
+        if "LiveViewDocument" in tname:
+            try:
+                view_doc = view_doc._current()
+            except Exception:
+                pass
+        elif "_DocProxy" in tname:
+            try:
+                view_doc = view_doc._target()
+            except Exception:
+                pass
+
         if view_doc is None:
             raise RuntimeError("No active document")
 

setiastro/saspro/fitsmodifier.py

@@ -606,6 +606,9 @@ class BatchFITSHeaderDialog(QDialog):
     def __init__(self, parent=None, preset_keyword: str = "", preset_value: str = "", preset_comment: str = ""):
         super().__init__(parent)
         self.setWindowTitle(self.tr("Batch Modify FITS Headers"))
+        self.setWindowFlag(Qt.WindowType.Window, True)
+        self.setWindowModality(Qt.WindowModality.NonModal)
+        self.setModal(False)
         self.resize(520, 220)
 
         v = QVBoxLayout(self)

setiastro/saspro/frequency_separation.py

@@ -708,10 +708,16 @@ class FrequencySeperationTab(QWidget):
 
         # channel reconcile
         if rch == 1 and ch == 3:
-            # convert RGB→mono (luma or average; we’ll use average)
+            # convert RGB→mono (use weighted luma for consistency, or mean if desired. Original was mean)
             a = a.mean(axis=2).astype(np.float32)
         elif rch == 3 and ch == 1:
-            a = np.repeat(a[..., None], 3, axis=2).astype(np.float32)
+            # Broadcast mono to 3 channels without copying
+            # (H,W,1) -> (H,W,3) via broadcasted view if consumer allows,
+            # but usually downstream (like subtraction) handles broadcasting fine.
+            # If explicit physical layout is needed, we must check usage.
+            # Here: used for subtraction (OK) and preview (OK).
+            # We return a view using broadcast_to or striding tricks.
+            a = np.broadcast_to(a, (ah, aw, 3))
 
         return a
 

setiastro/saspro/function_bundle.py

@@ -334,6 +334,8 @@ class FunctionBundleDialog(QDialog):
         super().__init__(parent)
         _pin_on_top_mac(self)
         self.setWindowTitle("Function Bundles")
+        self.setWindowFlag(Qt.WindowType.Window, True)
+        self.setWindowModality(Qt.WindowModality.NonModal)
         self.setModal(False)
         self.resize(920, 560)
         self.setAcceptDrops(True)