search

snorkel-team / snorkel

A system for quickly generating training data with weak supervision
https://snorkel.org
Apache License 2.0
5.81k stars 857 forks source link

Make helper function to print feature names for a given candidate #369

Closed bhancock8 closed 8 years ago

bhancock8 commented 8 years ago

Currently will accept candidates[:0] as input but not candidates[0]. Should be a simple fix.

bhancock8 commented 8 years ago

Added print_features(candidate) method to Featurizer. Will merge once the big ORM-learning_refactor merge has happened.

ajratner commented 8 years ago

Awesome! Want to merge this into the learning-refactor branch for now?

On Tue, Aug 2, 2016 at 10:40 AM Braden Hancock notifications@github.com wrote:

Added print_features(candidate) method to Featurizer. Will merge once the big ORM-learning_refactor merge has happened.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/HazyResearch/snorkel/issues/369#issuecomment-236982331, or mute the thread https://github.com/notifications/unsubscribe-auth/ABgw_YnNzHDZP4BYD43sLSiTBAkpNcYIks5qb4EPgaJpZM4Ja2F5 .

ajratner commented 8 years ago

And agreed this is a better option, I think accepting both single candidates and lists/sets would be confusing

bhancock8 commented 8 years ago

Yeah, I didn't think all the way through that transform() just returns a sparse matrix anyway, which isn't helpful if you're just trying to see which features are being generated on a particular candidate. Yep, I'll merge it into learning-refactor.