jpcompartir / LandscapeRGolem

playing_with_golem_learning
Other
0 stars 0 forks source link

Select grouping variable for umap plot + others #19

Closed jpcompartir closed 8 months ago

jpcompartir commented 1 year ago

Should this be a global reactive value passed through R, with default value set at 'cluster'? If changing in conversation landscape (or umap_plot module) should this change everywhere? Dropdown menu, button, sweet alert. pop out box?

jpcompartir commented 1 year ago

This is implemented in incoming PR, but should function do some work to prevent user selecting continuous variables?

i.e. let user select factors/characters only. And perhaps an limit on levels/uniques? If using bertopic output, topics could be very high, which presents other problems.

jpcompartir commented 1 year ago

https://github.com/jpcompartir/LandscaperGolem/pull/24

haven't implemented filtering the names shown yet

jpcompartir commented 8 months ago

This is implemented here with the helper function which only shows columns below a cardinality threshold https://github.com/jpcompartir/LandscapeRGolem/pull/44