datavisyn / visyn_core

BSD 3-Clause "New" or "Revised" License
1 stars 1 forks source link

feat: Fix Scatterplot #529

Open dvmoritzschoefl opened 2 months ago

dvmoritzschoefl commented 2 months ago

Currently the scatterplot has severe problems as described in https://github.com/datavisyn/visyn_core/issues/470

Step 1 is to collect a list of requirements for the scatter plot that we want to support in accordance with @dg-datavisyn

dg-datavisyn commented 2 months ago

I would add

dvchristianbors commented 2 months ago

Some comments:

  • [ ] Color We want to color by numerical/categorical attributes. In the faceting and matrix case the color scale is global, ensuring that the categories have the same color and the numerical relationship between plots is preserved.

Can this be overridden in the settings, with the above behavior only being the default?

  • [ ] Tooltip labels Correct me @dg-datavisyn but this should just add the attribute to the hover tooltip?

In its current functionality, you can specify columns that are shown in the tooltip. Are we removing this feature again?

dg-datavisyn commented 2 months ago

No, we are not removing this. To the contrary, Moritz is currently working on major refactorings of the scatterplot, and this ticket is here to make sure all features find their way back

dg-datavisyn commented 2 months ago

Some fundamental design drawbacks make this refactoring necessary. Moritz can explain the technical details. @thinkh, @dvmartinweigl, and @oltionchampari were involved in the discussion and decision.