Repository navigation
instance_key as categorical in PointsModel #160
Description
Activity
so in this case,
instance_keyrefers to points, not shapes. I agree it's debatable on whether we want to have it categorical. I think it might make sense in e.g. few regions but many points. I think few can be order of thousands of regions (and millions of points). wdyt? @LucaMarconatoAh you are right! I had confused points and circles (have been working with napari today and they use the same name). It should be categorical I agree.
Reacted by Giovanni Palla- locked and limited conversation to collaborators
on Feb 28, 2023
Unrelated to the above but related to
instance_key. I think that the column specified byinstance_keyshould never be categorical, so we should remove this line:spatialdata/spatialdata/_core/models.py
Line 464 in 4fb3433
Originally posted by @LucaMarconato in #158 (comment)