napari-skimage-regionprops overwrites layers.properties

[jo-mueller]

Hi @haesleinhuepf ,

it seems that napari-skimage-regionprops overwrites whatever content has previously been stored in the layer.properties attribute. Would you consider a PR that would change the behaviour of, for instance, add_table() in such a way that it checks whether content has already been written to the respective layer?

In case values have been written already, add_table() would then have to replaced by table_widget.append_content() which would add the data to layer.properties in a "smart" way.

[haesleinhuepf]

Hi Johannes @jo-mueller ,

that sounds interesting! Do I assume right, you refer to this code line?

Could you implement the desired behaviour without breaking backwards compatibility of the API?

[jo-mueller]

Could you implement the desired behaviour without breaking backwards compatibility of the API?

I'll try 👍

[haesleinhuepf]

btw, add_table() adds a table to the viewer. It doesn't add or change any content of layer.properties.
https://github.com/haesleinhuepf/napari-skimage-regionprops/blob/master/napari_skimage_regionprops/_table.py#L139