allow disabling embeddings event registration (#1585)

Co-authored-by: Saksham Sirohi <sakshamsirohi27@gmail.com>

Author: tonypzy

README.md

@@ -499,6 +499,7 @@ The function accepts the following arguments:
 - `labels`: a list of corresponding labels for the tensors provided for `features`
 - `data_getter=fn`: (optional) a function that takes as a parameter an index into the features array and returns a summary representation of the tensor. If this is set, `data_type` must also be set.
 - `data_type=str`: (optional) currently the only acceptable value here is `"html"`
+- `opts.register_embedding_events`: (optional) set to `False` to skip registering the default Python client event handler for hover previews and lasso drilldown. This leaves embeddings interaction events for external server or frontend code to handle.
 
 We currently assume that there are no more than 10 unique labels, in the future we hope to provide a colormap in opts for other cases.
 

py/visdom/__init__.py

@@ -1675,12 +1675,10 @@ def embeddings(
             endpoint="events",
         )
 
-        # Register the handlers for managing this embeddings pane
-        # TODO allow disabling this in a way that pushes onus for calculating
-        # to the server or frontend client
-        self._register_embeddings(
-            features, labels, points, data_getter, data_type, win, env, opts
-        )
+        if opts.get("register_embedding_events", True):
+            self._register_embeddings(
+                features, labels, points, data_getter, data_type, win, env, opts
+            )
         return win
 
     @pytorch_wrap