search

logv / snorkel

UI for interactive data analysis | https://snorkel.logv.org
https://fb.com/groups/snorkelsnorkelsnorkel
161 stars 21 forks source link

Attempt at #48: group by integer #49

Closed niteria closed 4 years ago

niteria commented 4 years ago

It seems that adding the integer columns to the list of available group by columns just works.

okayzed commented 4 years ago

super cool! thanks for bringing this feature area to my attention. it looks like a simple change, if it works locally please use it for now.

solution wise, id like to add a dataset settings page where the columns can be annotated with formatters, groupability, etc

if you need this feature on pip, i can merge it in and upload a new package for now before building out the solution above

niteria commented 4 years ago

I can manage with my tiny fork for now. It turns out that make binary3 && pip3 install dist/current/snorkel_lite-current-py3-none-any.whl was the only way I was able to get the master branch to work, so I already have the wheel file :)

The make dev way didn't let me log in and the script for adding users seemed to have failed on some API compatibility issue. Maybe the requirements.txt file needs some bounds, but I don't know the best practices here.

okayzed commented 4 years ago

if you could open an issue for those two problems that would be a big favor to me, thank you! (if you can see my workplace / intern status status, you'd see why it's a favor)