celldetective 1.5.0b5__py3-none-any.whl → 1.5.0b7__py3-none-any.whl

celldetective/_version.py

@@ -1 +1 @@
-__version__ = "1.5.0b5"
+__version__ = "1.5.0b7"

celldetective/gui/viewers/base_viewer.py

@@ -879,6 +879,14 @@ class StackVisualizer(CelldetectiveWidget):
         # Event handler for closing the widget
         if self.loader_thread:
             self.loader_thread.stop()
+            self.loader_thread = None
         if hasattr(self, "frame_cache") and isinstance(self.frame_cache, OrderedDict):
             self.frame_cache.clear()
         self.canvas.close()
+
+    def __del__(self):
+        try:
+            if hasattr(self, "loader_thread") and self.loader_thread:
+                self.loader_thread.stop()
+        except:
+            pass

celldetective/processes/unified_process.py

@@ -122,15 +122,29 @@ class UnifiedBatchProcess(Process):
         signal_model = None
 
         if self.run_signals:
-            from celldetective.utils.event_detection import _prep_event_detection_model
-
             try:
+                from celldetective.utils.event_detection import (
+                    _prep_event_detection_model,
+                )
+
                 logger.info("Loading the event detection model...")
                 self.queue.put({"status": "Loading event detection model..."})
                 model_name = self.signal_args["model_name"]
                 signal_model = _prep_event_detection_model(
                     model_name, use_gpu=self.signal_args.get("gpu", True)
                 )
+            except (ImportError, ModuleNotFoundError) as e:
+                if "tensorflow" in str(e):
+                    logger.warning(
+                        "Tensorflow not found/installed. Signal analysis will be skipped."
+                    )
+                    self.run_signals = False
+                else:
+                    logger.error(
+                        f"Failed to initialize event detection model: {e}",
+                        exc_info=True,
+                    )
+                    self.run_signals = False
             except Exception as e:
                 logger.error(
                     f"Failed to initialize event detection model: {e}", exc_info=True

celldetective/utils/cellpose_utils/__init__.py

@@ -104,14 +104,27 @@ def _prep_cellpose_model(
       `diam_labels` are attributes of the model.
     """
 
-    import torch
+    try:
+        import torch
+    except ImportError as e:
+        raise RuntimeError(
+            "Torch is not installed. Please install it to use this feature.\n"
+            "You can install the full package with: pip install celldetective[all]\n"
+        ) from e
 
     if not use_gpu:
         device = torch.device("cpu")
     else:
         device = torch.device("cuda")
 
-    from cellpose.models import CellposeModel
+    try:
+        from cellpose.models import CellposeModel
+    except ImportError as e:
+        raise RuntimeError(
+            "Cellpose is not installed. Please install it to use this feature.\n"
+            "You can install the full package with: pip install celldetective[all]\n"
+            "Or specifically: pip install celldetective[cellpose]"
+        ) from e
 
     try:
         model = CellposeModel(

celldetective/utils/downloaders.py

@@ -77,18 +77,39 @@ def download_url_to_file(url, dst, progress=True):
                     Default: True
 
     """
-    file_size = None
     import ssl
+    import time
+    from urllib.error import HTTPError, URLError
 
+    file_size = None
     ssl._create_default_https_context = ssl._create_unverified_context
-    u = urlopen(url)
-    meta = u.info()
-    if hasattr(meta, "getheaders"):
-        content_length = meta.getheaders("Content-Length")
-    else:
-        content_length = meta.get_all("Content-Length")
-    if content_length is not None and len(content_length) > 0:
-        file_size = int(content_length[0])
+
+    # Retry configuration
+    max_retries = 5
+    retry_delay = 10  # Initial delay in seconds
+
+    for attempt in range(max_retries):
+        try:
+            u = urlopen(url)
+            meta = u.info()
+            if hasattr(meta, "getheaders"):
+                content_length = meta.getheaders("Content-Length")
+            else:
+                content_length = meta.get_all("Content-Length")
+            if content_length is not None and len(content_length) > 0:
+                file_size = int(content_length[0])
+            break  # Success
+        except (HTTPError, URLError) as e:
+            if attempt < max_retries - 1:
+                logger.warning(
+                    f"Download check failed: {e}. Retrying in {retry_delay}s..."
+                )
+                time.sleep(retry_delay)
+                retry_delay *= 2  # Exponential backoff
+            else:
+                logger.error(f"Download check failed after {max_retries} attempts: {e}")
+                raise e
+
     # We deliberately save it in a temp file and move it after
     dst = os.path.expanduser(dst)
     dst_dir = os.path.dirname(dst)
@@ -143,14 +164,27 @@ def download_url_to_file(url, dst, progress=True):
                 unit_divisor=1024,
             ) as pbar:
                 while True:
-                    buffer = u.read(8192)  # 8192
-                    if len(buffer) == 0:
-                        break
-                    f.write(buffer)
-                    pbar.update(len(buffer))
+                    try:
+                        buffer = u.read(8192)  # 8192
+                        if len(buffer) == 0:
+                            break
+                        f.write(buffer)
+                        pbar.update(len(buffer))
+                    except (HTTPError, URLError) as e:
+                        # Attempt rudimentary resume-like behavior or just fail?
+                        # Simple retry of read is hard without Range headers on a stream.
+                        # Best to just fail the whole download and rely on outer retry if we wrapped the whole thing.
+                        # For now, let's just let it raise, but really we should wrap the whole download block.
+                        raise e
 
         f.close()
         shutil.move(f.name, dst)
+    except Exception as e:
+        f.close()
+        remove_file_if_exists(f.name)
+        # If we failed during download reading (after open), we should probably retry the whole function from start
+        # but that requires significant refactoring. Given the error was 504 on open, the retry block above handles it.
+        raise e
     finally:
         f.close()
         remove_file_if_exists(f.name)

celldetective/utils/stardist_utils/__init__.py

@@ -43,7 +43,13 @@ def _prep_stardist_model(
     - GPU support depends on the availability of compatible hardware and software setup.
     """
 
-    from stardist.models import StarDist2D
+    try:
+        from stardist.models import StarDist2D
+    except ImportError as e:
+        raise RuntimeError(
+            "StarDist is not installed. Please install it to use this feature.\n"
+            "You can install the full package with: pip install celldetective[all]"
+        ) from e
 
     model = StarDist2D(None, name=model_name, basedir=path)
     model.config.use_gpu = use_gpu

{celldetective-1.5.0b5.dist-info → celldetective-1.5.0b7.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: celldetective
-Version: 1.5.0b5
+Version: 1.5.0b7
 Summary: description
 Home-page: http://github.com/remyeltorro/celldetective
 Author: Rémy Torro
@@ -15,11 +15,8 @@ Requires-Dist: sphinx_rtd_theme
 Requires-Dist: sphinx
 Requires-Dist: jinja2
 Requires-Dist: ipykernel
-Requires-Dist: stardist
-Requires-Dist: cellpose<3
 Requires-Dist: scikit-learn
 Requires-Dist: btrack
-Requires-Dist: tensorflow~=2.15.0
 Requires-Dist: napari<0.6.0
 Requires-Dist: tqdm
 Requires-Dist: mahotas
@@ -48,6 +45,15 @@ Requires-Dist: prettyprint
 Requires-Dist: pandas
 Requires-Dist: matplotlib
 Requires-Dist: prettytable
+Requires-Dist: scikit-image
+Requires-Dist: natsort
+Provides-Extra: tensorflow
+Requires-Dist: tensorflow~=2.15.0; extra == "tensorflow"
+Requires-Dist: stardist; extra == "tensorflow"
+Provides-Extra: all
+Requires-Dist: cellpose<3; extra == "all"
+Requires-Dist: stardist; extra == "all"
+Requires-Dist: tensorflow~=2.15.0; extra == "all"
 Dynamic: author
 Dynamic: author-email
 Dynamic: description
@@ -55,6 +61,7 @@ Dynamic: description-content-type
 Dynamic: home-page
 Dynamic: license
 Dynamic: license-file
+Dynamic: provides-extra
 Dynamic: requires-dist
 Dynamic: summary
 
@@ -148,10 +155,12 @@ To use the software, you must install python, *e.g.* through
 
 Celldetective requires a version of Python between 3.9 and 3.11 (included). If your Python version is older or more recent, consider using `conda` to create an environment as described below.
 
-With the proper Python version, Celldetective can be directly installed with `pip`:
+With the proper Python version, Celldetective can be directly installed with `pip`.
+**Important**: By default, `pip install celldetective` will **not** install deep-learning libraries (`tensorflow`, `cellpose`, `stardist`) to allow users to manage their own GPU environment (e.g. `torch`, `cuda`).
 
+If you want the standard full installation (recommended for most users), use:
 ``` bash
-pip install celldetective
+pip install celldetective[all]
 ```
 
 We recommend that you create an environment to use Celldetective, to protect your package versions and fix the Python version *e.g.*
@@ -160,7 +169,7 @@ with `conda`:
 ``` bash
 conda create -n celldetective python=3.11 pyqt
 conda activate celldetective
-pip install celldetective
+pip install celldetective[all]
 ```
 
 Need an update? Simply type the following in the terminal (in your

{celldetective-1.5.0b5.dist-info → celldetective-1.5.0b7.dist-info}/RECORD

@@ -1,6 +1,6 @@
 celldetective/__init__.py,sha256=LfnOyfUnYPGDc8xcsF_PfYEL7-CqAb7BMBPBIWGv84o,666
 celldetective/__main__.py,sha256=Rzzu9ArxZSgfBVjV-lyn-3oanQB2MumQR6itK5ZaRpA,2597
-celldetective/_version.py,sha256=WqWa8H3XCjzmFMLQoflfXoLyQGm-ej1fcgZfMqTF--U,24
+celldetective/_version.py,sha256=PxAZ8DovjTTFvob973OaT7yXi4GAYpuAcRntt0ZchLM,24
 celldetective/events.py,sha256=n15R53c7QZ2wT8gjb0oeNikQbuRBrVVbyNsRCqXjzXA,8166
 celldetective/exceptions.py,sha256=f3VmIYOthWTiqMEV5xQCox2rw5c5e7yog88h-CcV4oI,356
 celldetective/extra_properties.py,sha256=s_2R4_El2p-gQNZ_EpgDxgrN3UnRitN7KDKHhyLuoHc,21681
@@ -87,7 +87,7 @@ celldetective/gui/table_ops/_merge_one_hot.py,sha256=gKRgem-u_-JENkVkbjRobsH4TkS
 celldetective/gui/table_ops/_query_table.py,sha256=K-XHSZ1I4v2wwqWjyQAgyFRZJbem3CmTfHmO0vijh9g,1345
 celldetective/gui/table_ops/_rename_col.py,sha256=UAgDSpXJo_h4pLJpHaNc2w2VhbaW4D2JZTgJ3cYC4-g,1457
 celldetective/gui/viewers/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-celldetective/gui/viewers/base_viewer.py,sha256=ir3J8QyLDAijOMu0Np56lUR0fSeWhz_1xdb_bX6rdRk,31939
+celldetective/gui/viewers/base_viewer.py,sha256=wA2Zrje5nQSUmwnAFCe9hcL1MOWg8uMKwFTWhEYiMXI,32159
 celldetective/gui/viewers/channel_offset_viewer.py,sha256=cywBkxyMPyKIuwZTGA03DBSS4a-H1SAohMJYOPISLEE,16289
 celldetective/gui/viewers/contour_viewer.py,sha256=riHj03LKXLoa-Ys2o2EhCE5nULfcHMohx9LFoXbI6zU,14720
 celldetective/gui/viewers/size_viewer.py,sha256=uXITjaxg5dhQ09Q6TNUxwLxi-ZglyGFcxEH1RtGIZWw,6020
@@ -126,7 +126,7 @@ celldetective/processes/segment_cells.py,sha256=klGl5y3tHbLbQFNAQz0PV1FspmiPC-GH
 celldetective/processes/track_cells.py,sha256=ndKZpVXonnooOVnHK5AVED5i_i2bZ4FefTt1_dEZS48,15188
 celldetective/processes/train_segmentation_model.py,sha256=j1aPgR4CzF2OR-9njriNmUStVViiWaiicQA9YWRWOTo,25411
 celldetective/processes/train_signal_model.py,sha256=HInUHCXNePhfQs8-DQOreJ19LtLkQopvYSyX8tj0mOo,9498
-celldetective/processes/unified_process.py,sha256=vHAS_LbtwmlOjKho_FpwDy7WHVgyY38s3X2BtHxep7A,11671
+celldetective/processes/unified_process.py,sha256=xvNrYtZaRgpjEWzrPSYoY6dmqJbC81udDojkHuT4iTc,12251
 celldetective/regionprops/__init__.py,sha256=ohe9vC7j4lnpKnGXidVo1PVfoQfC8TqCuHTNo8rXmdg,44
 celldetective/regionprops/_regionprops.py,sha256=spq_Mb43HjHiKqot7yjLJl_jrMTNfIP8NLrPnCEm8Nc,13306
 celldetective/regionprops/props.json,sha256=sCwACmbh0n-JAw9eve9yV85REukoMBJLsRjxCwTRMm8,2424
@@ -143,7 +143,7 @@ celldetective/utils/color_mappings.py,sha256=yarqOTSrTsnOPPiPrrN_vLoPCbgWqF3wjqF
 celldetective/utils/data_cleaning.py,sha256=K-2gScxLreX7QkrM0h3dZdP0IsmvCzcxNh2-M9PALZY,22025
 celldetective/utils/data_loaders.py,sha256=6Jg99U93qYjjs2xZErc2cz37tcH5e4vEqDH8PJgoEJs,17077
 celldetective/utils/dataset_helpers.py,sha256=3ezpHO6nytw2Mx0D3maP_4535V2ohOTQn6Qpfk8gnms,6898
-celldetective/utils/downloaders.py,sha256=BIl_8XCeaKvtMVC36WITT2g5-O-n0csoMEQXVoa1B4o,7887
+celldetective/utils/downloaders.py,sha256=o5sogEeYr-LR8mp1D7uNHH1aUJN3V82VVwYZvoSNTkQ,9506
 celldetective/utils/experiment.py,sha256=bgADS70QuW4KGbzDJbVpVM-tw4qmZKMWtDT1cSxugrY,58342
 celldetective/utils/image_augmenters.py,sha256=USYd8z6dVn5z1x96IYJ4mG0smN9I_S21QMGU0wyHmjc,11654
 celldetective/utils/image_cleaning.py,sha256=KliQ3K5hdwPx4eFxJnmg3yi-ZIoimEveunPJkbbA6wA,2388
@@ -161,12 +161,12 @@ celldetective/utils/parsing.py,sha256=1zpIH9tyULCRmO5Kwzy6yA01fqm5uE_mZKYtondy-V
 celldetective/utils/resources.py,sha256=3Fz_W0NYWl_Ixc2AjEmkOv5f7ejXerCLJ2z1iWhGWUI,1153
 celldetective/utils/stats.py,sha256=4TVHRqi38Y0sed-izaMI51sMP0fd5tC5M68EYyfJjkE,3636
 celldetective/utils/types.py,sha256=lRfWSMVzTkxgoctGGp0NqD551akuxu0ygna7zVGonTg,397
-celldetective/utils/cellpose_utils/__init__.py,sha256=MeRDojrAkBSXe-wlLu5KIih5wXP9B2aPdP39JLYpoGE,5417
+celldetective/utils/cellpose_utils/__init__.py,sha256=g41EiR6p29bHkXQN-SFHymoeaoTEfuozEFSGMuTd30Q,5991
 celldetective/utils/event_detection/__init__.py,sha256=KX20PwPTevdbZ-25otDy_QTmealcDx5xNCfH2SOVIZM,323
 celldetective/utils/plots/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 celldetective/utils/plots/regression.py,sha256=oUCn29-hp7PxMqC-R0yoL60KMw5ZWpZAIoCDh2ErlcY,1764
-celldetective/utils/stardist_utils/__init__.py,sha256=e9s3DEaTKCUOGZb5k_DgShBTl4B0U-Jmg3Ioo8D5PyE,3978
-celldetective-1.5.0b5.dist-info/licenses/LICENSE,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
+celldetective/utils/stardist_utils/__init__.py,sha256=SY2kxFNXSRjXN4ncs3heDdXT3UNk8M3dELJQySysAf4,4231
+celldetective-1.5.0b7.dist-info/licenses/LICENSE,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
 tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tests/test_cellpose_fallback.py,sha256=BJZTDFF8sFR1x7rDbvZQ2RQOB1OP6wuFBRfc8zbl5zw,3513
 tests/test_events.py,sha256=eLFwwEEJfQAdwhews3-fn1HSvzozcNNFN_Qn0gOvQkE,685
@@ -175,6 +175,7 @@ tests/test_io.py,sha256=gk5FmoI7ANEczUtNXYRxc48KzkfYzemwS_eYaLq4_NI,2093
 tests/test_measure.py,sha256=FEUAs1rVHylvIvubCb0bJDNGZLVmkgXNgI3NaGQ1dA8,4542
 tests/test_neighborhood.py,sha256=gk5FmoI7ANEczUtNXYRxc48KzkfYzemwS_eYaLq4_NI,2093
 tests/test_notebooks.py,sha256=7HVmYiytsz0QIJ11iRkGGs4_hzNjofXAUs_OZou3Gm0,301
+tests/test_partial_install.py,sha256=G69-GNcJ9YNgs6K2bVTEZO0Jpb14xMRQWTm8A6VuIco,2841
 tests/test_preprocessing.py,sha256=c0rKS9d5h37uDcV7fVOTnn5GMVbEB84b8ZTCTdRmvFs,1422
 tests/test_segmentation.py,sha256=k1b_zIZdlytEdJcHjAUQEO3gTBAHtv5WvrwQN2xD4kc,3470
 tests/test_signals.py,sha256=No4cah6KxplhDcKXnU8RrA7eDla4hWw6ccf7xGnBokU,3599
@@ -185,8 +186,8 @@ tests/gui/test_enhancements.py,sha256=3x9au_rkQtMZ94DRj3OaEHKPr511RrWqBAUAcNQn1y
 tests/gui/test_measure_annotator_bugfix.py,sha256=tPfgWNKC0UkvrVssSrUcVDC1qgpzx6l2yCqvKtKYkM4,4544
 tests/gui/test_new_project.py,sha256=wRjW2vEaZb0LWT-f8G8-Ptk8CW9z8-FDPLpV5uqj6ck,8778
 tests/gui/test_project.py,sha256=KzAnodIc0Ovta0ARL5Kr5PkOR5euA6qczT_GhEZpyE4,4710
-celldetective-1.5.0b5.dist-info/METADATA,sha256=HXfAb1NZp3md807sossVoljx3m1O4VYRaJrgjaq8T0c,10947
-celldetective-1.5.0b5.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-celldetective-1.5.0b5.dist-info/entry_points.txt,sha256=2NU6_EOByvPxqBbCvjwxlVlvnQreqZ3BKRCVIKEv3dg,62
-celldetective-1.5.0b5.dist-info/top_level.txt,sha256=6rsIKKfGMKgud7HPuATcpq6EhdXwcg_yknBVWn9x4C4,20
-celldetective-1.5.0b5.dist-info/RECORD,,
+celldetective-1.5.0b7.dist-info/METADATA,sha256=qOwqT_qSpFnGY0CXpVIFdSUV4atTi3FV4M4Kn57tNAg,11523
+celldetective-1.5.0b7.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+celldetective-1.5.0b7.dist-info/entry_points.txt,sha256=2NU6_EOByvPxqBbCvjwxlVlvnQreqZ3BKRCVIKEv3dg,62
+celldetective-1.5.0b7.dist-info/top_level.txt,sha256=6rsIKKfGMKgud7HPuATcpq6EhdXwcg_yknBVWn9x4C4,20
+celldetective-1.5.0b7.dist-info/RECORD,,

tests/test_partial_install.py

@@ -0,0 +1,75 @@
+import unittest
+from unittest.mock import MagicMock, patch
+import sys
+import importlib
+
+
+class TestPartialValidation(unittest.TestCase):
+
+    def test_imports_without_extras(self):
+        """Test that main modules can be imported even if optional extras are missing."""
+        # This test assumes the environment MIGHT have them, so we must mock them as missing
+        # to ensure the code handles it.
+
+        with patch.dict(
+            sys.modules,
+            {
+                "tensorflow": None,
+                "torch": None,
+                "stardist": None,
+                "cellpose": None,
+                "cellpose.models": None,
+                "stardist.models": None,
+            },
+        ):
+            # Force reload of celldetective.segmentation to test its imports
+            try:
+                import celldetective.segmentation
+
+                importlib.reload(celldetective.segmentation)
+            except ImportError as e:
+                self.fail(
+                    f"Could not import celldetective.segmentation without extras: {e}"
+                )
+            except Exception as e:
+                self.fail(f"Unexpected error importing celldetective.segmentation: {e}")
+
+    def test_graceful_failure_stardist(self):
+        """Test that calling stardist functions raises RuntimeError if missing."""
+        with patch.dict(sys.modules, {"stardist": None, "stardist.models": None}):
+            # We need to reload the util module to pick up the missing module
+            import celldetective.utils.stardist_utils
+
+            importlib.reload(celldetective.utils.stardist_utils)
+
+            from celldetective.utils.stardist_utils import _prep_stardist_model
+
+            with self.assertRaises(RuntimeError) as cm:
+                _prep_stardist_model("fake_model", "fake_path")
+
+            self.assertIn("StarDist is not installed", str(cm.exception))
+
+    def test_graceful_failure_cellpose(self):
+        """Test that calling cellpose functions raises RuntimeError if missing."""
+        with patch.dict(
+            sys.modules, {"cellpose": None, "cellpose.models": None, "torch": None}
+        ):
+            import celldetective.utils.cellpose_utils
+
+            importlib.reload(celldetective.utils.cellpose_utils)
+
+            from celldetective.utils.cellpose_utils import _prep_cellpose_model
+
+            with self.assertRaises(RuntimeError) as cm:
+                _prep_cellpose_model("fake_model", "fake_path")
+
+            # Message check might correspond to torch or cellpose depending on which import hits first
+            # Our code checks torch first.
+            self.assertTrue(
+                "Torch is not installed" in str(cm.exception)
+                or "Cellpose is not installed" in str(cm.exception)
+            )
+
+
+if __name__ == "__main__":
+    unittest.main()