fix: some minor misc fixes and comments modifications

Author: ramyamounir

src/tbp/interactive/colors.py

@@ -50,7 +50,6 @@ class Palette:
     charcoal: str = "#3f3f3f"
     link_water: str = "#dfe6f5"
 
-    # ---------- Internal helper ----------
     @classmethod
     def _validate(cls, name: str) -> str:
         if not hasattr(cls, name):
@@ -62,7 +61,6 @@ def _validate(cls, name: str) -> str:
             raise KeyError(msg)
         return getattr(cls, name)
 
-    # ---------- Public API ----------
     @classmethod
     def as_hex(cls, name: str) -> Color:
         """Return the raw hex string for a color name."""

src/tbp/interactive/events.py

@@ -12,12 +12,11 @@
 
 @dataclass
 class EventSpec:
-    """Specification for an Event to be defined inside WidgetOps.
+    """Specification for an Event to be defined as a WidgetUpdater callback trigger.
 
     Attributes:
         trigger: Event trigger name (e.g., KeyPressed)
         name: Event name field in Vedo `event.name` (e.g., keypress).
-        would be send in the topic message value.
         required: Whether this event is required for the callback trigger. If
             True, the updater will not call the callback until a message for this
             topic arrives.

src/tbp/interactive/scopes.py

@@ -22,12 +22,12 @@ class ScopeViewer:
 
     Behavior summary:
     - Scope 0:
-        * If at least one widget is hidden -> show ALL widgets.
+        * If at least one widget is hidden -> show all widgets.
         * Else (all visible) -> hide ALL widgets.
     - Scope k (1..9):
         * Toggle that scope on/off.
-        * A widget is visible if ANY active scope includes it.
-        * If no active scopes -> all widgets off.
+        * During a toggle off, a widget may remain visible if any other active scope
+            includes it.
 
     The widgets themselves decide how to hide/show internally using
     their .on() / .off() visibility handlers.

src/tbp/interactive/widget_updaters.py

@@ -90,7 +90,7 @@ def accepts(self, msg: TopicMessage) -> bool:
     def expire_topic(self, topic_name: str) -> None:
         """Expire (remove) the stored message for a given topic.
 
-        After expiration, the updater will require a new message for that topic
+        After expiration, the updater may require a new message for that topic
         before becoming ready again.
 
         Args:

src/tbp/interactive/widgets.py

@@ -211,8 +211,6 @@ def updater_events(self) -> set[str]:
             if isinstance(e, EventSpec)
         }
 
-        return {t.name for u in self.widget_ops.updaters for t in u.topics}
-
     def add(self) -> None:
         """Create the widget and register the debounce callback.
 

src/tbp/plot/plots/interactive_hypothesis_space_correlation.py

@@ -966,7 +966,7 @@ def extract_state(self, widget: Slider2D) -> int:
     def set_state(self, widget: Slider2D, value: int) -> None:
         set_slider_state(widget, value)
 
-    def state_to_messages(self, state: float) -> Iterable[TopicMessage]:
+    def state_to_messages(self, state: int) -> Iterable[TopicMessage]:
         return [TopicMessage(name="top_k", value=state)]
 
 
@@ -1396,15 +1396,15 @@ def decrement_step(self, episode: int, step: int) -> tuple[int, int]:
         Returns:
             A tuple `(prev_episode, prev_step)`.
         """
-        # Base case: already at earliest step
+        # Already at earliest step
         if episode == 0 and step == 0:
             return 0, 0
 
         # If we're not at the start of the episode, just decrement the step
         if step > 0:
             return episode, step - 1
 
-        # step == 0 → need to go to previous episode
+        # If step is 0, we need to go to previous episode
         prev_episode = episode - 1
 
         # Find last step index in previous episode