-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Exclude retweeted_user_name/quoted_user_name from mentioned_user_names ? #20
Comments
It seems mistake-inducing to me that those would be in This said a case could be made to have a column with a different name containing all involved participant as it is sometimes useful to build networks. But then I feel we can also build this list from the data we already have so I would lean in favor of not adding columns to stay lean and clear, no? |
Yes that's exactly how I see it as well |
So, for the reference, things happen as is right now because the API returned |
Yep indeed |
Do we consider that replies and mentions are the same thing ? |
Good question indeed, I didn't think about this, but we also have the replied_to_user_name as well to consider. If we decide to remove quoted and retweeted from mentioned, it would make sense to also remove replied_to maybe indeed! |
To be discussed
(identified within the fakers corpus while questionning why the proportion of tweets with mentioned_user_name was over 70%)
cc @bmaz @Yomguithereal
The text was updated successfully, but these errors were encountered: