Barebone annotate

[melonora]

This PR serves to provide a barebone annotation widget. At the moment only shape annotations are supported but I am to extend this to points and perhaps labels as well. Furthermore, at a later stage also a table widget will be provided.

Currently, the user can annotate using and also reload shape annotations. The description column at the moment serves no purpose but this will come with the table widget.

@LucaMarconato maybe before putting this in we need to discuss a little bit about UI experience.

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[LucaMarconato]

Open points from the test together (these are all mac specific):

• bug with lasso (to replicate: open the annotation widgret, create a shapes layer, rename it, then use the lasso). If the layer is linked, the lasso works again.
• (tiny) after modifying the class text it feels natural to use to clear focus
• general bug with radio selection and colors
• color selection (with the color widget) doesn't update the colors in the "layer controls" widget
• works on Wouter's Mac but not on Luca's one: switching between layers doesn't update the list of classes.

Non mac specific

• luca: mergecheckout incremental io and test the saving

[LucaMarconato]

Small comment:

• please document the code inside on_mouse_move() in _view.py.

[LucaMarconato]

Bug:

• _on_inserted() is not called when a shapes layer is created before opening the annotation widget. A consequence of this is that, after setting the annotation on shape and after deselecting the shape, hovering the mouse on the shape will not show the text that has been saved. This because the callback for on_mouse_move() is set in _on_insterted(), which is never called.

[LucaMarconato]

I tested the saving (on the incremental io branch). I found some bugs:

• when saving an element, the View widget is not updated. Workaround: from the element widget add the element, and delete it; now the View widget is updated an it works.
• The colors of the annotation widget do not correspond to the colors used for plotting, easy fix.
• The "Annotation table" dropdown combobox does not update the color classes.

Saving and overwriting work!

[LucaMarconato]

Bug that I probably introduced here (I was working on performance in order to avoid copies) https://github.com/scverse/napari-spatialdata/pull/239/files in _model.py:

• when trying to plot instance_key an error is raised. The user playing around with the annotation widget may encounter this bug because the table created has only a few columns, so it's likely to try to plot instance_id.

[melonora]

I tested the saving (on the incremental io branch). I found some bugs:

• when saving an element, the View widget is not updated. Workaround: from the element widget add the element, and delete it; now the View widget is updated an it works.
• The colors of the annotation widget do not correspond to the colors used for plotting, easy fix.
• The "Annotation table" dropdown combobox does not update the color classes.

Saving and overwriting work!

not certain what you mean with the last point

[melonora]

tests pass locally, but here it would require a release of spatialdata with the incremental IO. For hackathon purposes merging now already.

[LucaMarconato]

The "Annotation table" dropdown combobox does not update the color classes.

I will explain this in a new issue (done here: #260), I have just verified and it is still open.