Allow flexible cluster ids

[zoccoler]

This should fix #204

[codecov]

Codecov Report

Merging #205 (00cc22a) into main (6e39600) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #205   +/-   ##
=======================================
  Coverage   77.03%   77.03%           
=======================================
  Files          14       14           
  Lines        1337     1337           
=======================================
  Hits         1030     1030           
  Misses        307      307           

Impacted Files	Coverage Δ
napari_clusters_plotter/_plotter.py	50.21% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[lazigu]

Marcelo @zoccoler, I think this should be marked as ready for review and merged. The problem with algorithmic clustering is related to plotting the 2D histogram in general, and not hiding not selected clusters, which is fixed by this PR.

[lazigu]

Accidentally marked this as ready for review.

It turns out, hiding non-selected clusters is not necessary for algorithmic clustering results, because that is already done by default, e.g. hdbscan detects noise points (non-selected grey points), which are not visualized:

Checking the "hide non-selected clusters" would hide in this case the first (orange) cluster:

I think for correctness we should rename the checkbox or add a tooltip, and leave it enabled only for manual clsutering results.

[lazigu]

I will merge this because it is needed to fix the issue of visualizing algorithmic clustering results in the 2D histogram plot, and deal with the issue described in my comment above in another PR.