Allow use of add_*_selector methods in ScatterGraphic#883

lkeegan

move these existing methods from LineGraphic to common base clase PositionsGraphic
regenerate docs

kushalkolar

Hi thanks for your PR!

Supporting selections for scatters will also require implementing the methods to get data from the selections, for example with the rectangle selector:

fastplotlib/fastplotlib/graphics/selectors/_rectangle.py

Line 295 in dac31cf

def get_selected_data(

fastplotlib/fastplotlib/graphics/selectors/_rectangle.py

Line 423 in dac31cf

def get_selected_indices(

I'm not sure what a linear or linear region selector means for a scatter, do you have a use case example? I can maybe see the usecase for exploring regression plots.

lkeegan

Hi thanks for your PR!

Supporting selections for scatters will also require implementing the methods to get data from the selections, for example with the rectangle selector:

fastplotlib/fastplotlib/graphics/selectors/_rectangle.py

Line 295 in dac31cf

def get_selected_data(

fastplotlib/fastplotlib/graphics/selectors/_rectangle.py

Line 423 in dac31cf

def get_selected_indices(

Thanks for your feedback - and for this very nice library! I'll look into updating these methods.

I'm not sure what a linear or linear region selector means for a scatter, do you have a use case example? I can maybe see the usecase for exploring regression plots.

I guess if one axis in the scatter is time it could make sense to select a time range of data using the linear region selector.
My personal use case is very basic - I just want to be able to select a time on linked plots of time series data, e.g. a line plot of voltage traces and a scatter plot of features extracted from these traces:

plots.mp4

clewis7

Hi @lkeegan

Thanks again for making a PR! I had a few small questions/tweaks. I'm sure @kushalkolar will also want to take a look. I checked out your examples, and they look great!

Why did you remove the ymin * 1.5 and ymax * 1.5? I think we had that there to allow the selector to go a little bit above and below the graphic.

Ah sorry - I removed it because with positive ymin the limits didn't allow you to select all the data, restored in 75b7d73

Can you add code comments here?

sure: c5ec080

clewis7

@kushalkolar LGTM as long as the examples pass

kushalkolar

I'm going to wait until #837 is merged first since that changes some things with selectors.

lkeegan

@kushalkolar I've updated this PR to include the polygon selector from #837

kushalkolar

Thanks for the update! I'll try this out and merge after #904 so that I don't have to replace the ground truth screenshots multiple times, it will be merged soon! 😄

@almarklein

@almarklein I'm guessing this was a typo? 😆

I don't think so, the type of MoveInfo.mode was str. Though in _end_move_mode I did set it to None, so I was not consistent 😆.

So this change looks fine to me, especially with the type now also being mode: str | None.

might want to pad in both x and y for a scatter?

yes that makes sense

make this true, we'd want to have a ground truth screenshot for this since it's a new feature.

done

make this true, we'd want to have a ground truth screenshot for this since it's a new feature.

done

make this true, we'd want to have a ground truth screenshot for this since it's a new feature.

done

lkeegan

@kushalkolar thanks for reviewing, I've made the suggested changes and merged the latest changes from the main branch

kushalkolar

Thanks I haven't forgotten this! We're just very busy with other parts of the library at the moment.

…

On Mon, Nov 10, 2025, 09:33 Liam Keegan ***@***.***> wrote: *lkeegan* left a comment (fastplotlib/fastplotlib#883) <#883 (comment)> @kushalkolar <https://github.com/kushalkolar> thanks for reviewing, I've made the suggested changes and merged the latest changes from the main branch — Reply to this email directly, view it on GitHub <#883 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACHXXRCGFODJ34AGDBLM2YL34CO5DAVCNFSM6AAAAACB76T6VWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTKMJSGA2DOMRRGY> . You are receiving this because you were mentioned.Message ID: ***@***.***>

kushalkolar

Some comments, sorry for the late response, lots of stuff on our plate lately

Is this always valid for an arbitrary scatter? What if we have a blob, like a Gaussian cloud? I think it's only valid for scatters that represent data which are a function of the x or y dim?

I think it's valid for any scatter data - it just returns the subset of data in the selected region:

Can you update this w.r.t. the current methods

Done in 6e5b923

Allow use of add_*_selector methods in ScatterGraphic …

acab9d0

- move these existing methods from `LineGraphic` to common base clase `PositionsGraphic` - regenerate docs

lkeegan requested review from clewis7 and kushalkolar as code owners July 21, 2025 14:03

Update linear/rectangle selectors to work with ScatterGraphics, add e… …

5ae771e

…xamples

clewis7 reviewed Jul 22, 2025

View reviewed changes

lkeegan added 2 commits July 23, 2025 08:06

refactor bounds_init logic to make intent clearer, add comment, remov… …

c5ec080

…e unused variables

restore y-padding on rectangle selector limits, refactored to also wo… …

75b7d73

…rk when ymin is positive

lkeegan added 2 commits September 29, 2025 11:20

Merge branch 'main' into scatter_graphic_add_linear_selector

c28163f

Add add_polygon_selector method to ScatterGraphic and create an e… …

e2d6b9f

…xample for it

kushalkolar reviewed Oct 1, 2025

View reviewed changes

kushalkolar mentioned this pull request Oct 1, 2025

add ImageVolumeGraphic #791

Merged

14 tasks

lkeegan added 4 commits October 1, 2025 16:00

allow selection to extend by 25% padding in all directions

8776b29

enable ground truth screenshots for scatter selector examples

7898ad2

set initial polygon selection in example and reset _move_info after s… …

82870b9

…etting selection to complete selection

Merge branch 'main' into scatter_graphic_add_linear_selector

9054ae0

kushalkolar requested changes Jan 30, 2026

View reviewed changes

lkeegan and others added 5 commits January 30, 2026 09:43

Update fastplotlib/graphics/selectors/_polygon.py …

2b607b5

Co-authored-by: Kushal Kolar <kushalkolar@gmail.com>

Update fastplotlib/graphics/selectors/_rectangle.py …

5313a05

Co-authored-by: Kushal Kolar <kushalkolar@gmail.com>

Update fastplotlib/graphics/selectors/_polygon.py …

481138b

Co-authored-by: Kushal Kolar <kushalkolar@gmail.com>

run black on examples/selection_tools/linear_region_scatter.py

41c3a02

Merge branch 'main' into scatter_graphic_add_linear_selector

4db8d34

lkeegan added 3 commits January 30, 2026 10:00

update add_*_selector to match main branch (keeping previous changes … …

6e5b923

…to support negative y-axis values)

fix typos

4598ae0

Add sizes to linear region scatter example

b0720b4

lkeegan requested a review from kushalkolar January 30, 2026 10:59

+                      axis_vals_min = np.floor(axis_vals.min()).astype(int)
+                      axis_vals_max = np.floor(axis_vals.max()).astype(int)
+                      bounds_init = axis_vals_min, axis_vals_min + 0.25 * (
+                          axis_vals_max - axis_vals_min
+                      )
+                      limits = axis_vals_min, axis_vals_max

                       # min/max limits
-                      limits = (x_axis_vals[0], x_axis_vals[-1], ymin * 1.5, ymax * 1.5)
+                      limits = (xmin, xmax, ymin, ymax)

                       self._resizable = bool(resizable)
                       BaseSelector.__init__(self, name=name, parent=parent)
-                      self._move_info = MoveInfo("none", -1, -1, None, None)

+                      xmin = np.floor(x_axis_vals.min()).astype(int)
+                      xmax = np.ceil(x_axis_vals.max()).astype(int)
+                      # min/max limits include the data + 25% padding in the y-direction

+              LinearRegionSelector which selects along the x-axis and a vertical selector which moves along the y-axis.
+              """
+              # test_example = false

                               return source.data[s]
+                      if "Scatter" in source.__class__.__name__:
+                          return source.data[ixs]


		self._size_space = SizeSpace(size_space)
		super().__init__(args, *kwargs)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Allow use of add_*_selector methods in ScatterGraphic#883