Skip to content

Commit

Permalink
Document why we don't use update_from_roi
Browse files Browse the repository at this point in the history
  • Loading branch information
jfoster17 committed Jul 14, 2023
1 parent 098747f commit b12d66a
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions glue_jupyter/bqplot/common/tools.py
Original file line number Diff line number Diff line change
Expand Up @@ -204,10 +204,11 @@ def __init__(self, viewer, roi=None, finalize_callback=None, **kwargs):

def update_from_roi(self, roi):
"""
TOTO: This should update self.xlist and self.ylist from the existing ROI
While other tools allow the user to click and drag to reposition a selection,
this probably does not make sense for a polygonal selection, so we do not do
not support this.
"""
with self.viewer._output_widget or nullcontext():
pass
pass

def activate(self):
"""
Expand Down

0 comments on commit b12d66a

Please sign in to comment.