figpack 0.2.21__py3-none-any.whl → 0.2.22__py3-none-any.whl

figpack/__init__.py

@@ -2,7 +2,7 @@
 figpack - A Python package for creating shareable, interactive visualizations in the browser
 """
 
-__version__ = "0.2.21"
+__version__ = "0.2.22"
 
 from .cli import view_figure
 from .core import FigpackView, FigpackExtension, ExtensionView

figpack/core/_bundle_utils.py

@@ -58,7 +58,7 @@ def prepare_figure_bundle(
         # Write the view data to the Zarr group
         zarr_group = zarr.open_group(pathlib.Path(tmpdir) / "data.zarr", mode="w")
         zarr_group = Group(zarr_group)
-        view._write_to_zarr_group(zarr_group)
+        view.write_to_zarr_group(zarr_group)
 
         # Add title and description as attributes on the top-level zarr group
         zarr_group.attrs["title"] = title

figpack/core/_upload_bundle.py

@@ -3,9 +3,7 @@ import json
 import pathlib
 import threading
 import time
-import uuid
 from concurrent.futures import ThreadPoolExecutor, as_completed
-from datetime import datetime, timedelta, timezone
 
 import requests
 

figpack/core/extension_view.py

@@ -23,10 +23,10 @@ class ExtensionView(FigpackView):
         self.extension = extension
         self.view_type = view_type
 
-    def _write_to_zarr_group(self, group: Group) -> None:
+    def write_to_zarr_group(self, group: Group) -> None:
         """
         Write the extension view metadata to a Zarr group.
-        Subclasses should call super()._write_to_zarr_group(group) first,
+        Subclasses should call super().write_to_zarr_group(group) first,
         then add their own data.
 
         Args:

figpack/core/figpack_view.py

@@ -62,11 +62,14 @@ class FigpackView:
         if upload is None:
             upload = os.environ.get("FIGPACK_UPLOAD") == "1"
         else:
-            if upload is True and ephemeral is True:
-                # ephemeral is reserved for the case where we don't specify upload
-                # and we are in a notebook in a remote environment such as
-                # colab or jupyterhub
-                raise ValueError("ephemeral cannot be set if upload is set")
+            if upload is True:
+                if ephemeral is True:
+                    # ephemeral is reserved for the case where we don't specify upload
+                    # and we are in a notebook in a remote environment such as
+                    # colab or jupyterhub
+                    raise ValueError("ephemeral cannot be set if upload is set")
+                else:
+                    ephemeral = False  # if we excplicitly set upload=True, force ephemeral=False
 
         # determine inline
         if inline is None:
@@ -155,11 +158,11 @@ class FigpackView:
 
         _save_figure(self, output_path, title=title)
 
-    def _write_to_zarr_group(self, group: Group) -> None:
+    def write_to_zarr_group(self, group: Group) -> None:
         """
         Write the view data to a Zarr group. Must be implemented by subclasses.
 
         Args:
             group: Zarr group to write data into
         """
-        raise NotImplementedError("Subclasses must implement _write_to_zarr_group")
+        raise NotImplementedError("Subclasses must implement write_to_zarr_group")