search

biolab / orange3-text

🍊 :page_facing_up: Text Mining add-on for Orange3
Other
128 stars 84 forks source link

Twitter: don't set Author as class variable #535

Closed ajdapretnar closed 4 years ago

ajdapretnar commented 4 years ago

Twitter widget automatically sets Author as class variable. Don't do it. Most of the time it doesn't make sense. One can always use Select Columns to do it explicitly.

irgolic commented 4 years ago

Knowing to use Select Columns to set the target variable is a UX issue. I mentioned this in https://github.com/biolab/orange3/issues/3997 in regard to CSV File Import, but a solution for all data set loading widgets should be devised.

Maybe if all data set widgets inherited the same base class, which contains File's attribute role controls? I guess this is more of an orange3 discussion, I'm just mentioning it here because it's relevant.