Skip to content
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

Add blacklist twitter userIDs #21

Merged
merged 5 commits into from
Feb 27, 2022
Merged

Conversation

GabrieleCalarota
Copy link
Contributor

@GabrieleCalarota GabrieleCalarota commented Feb 27, 2022

Fix #5

Excluding accounts that spreads misinformation

This pullrequest added:

  • Changes in README (adding point 4)

image

  • Adds new env variable to handle blacklisted userIDs
  • Add test folder and some test coverage of the implemented feature

Added check to exclude blacklisted authors both in write_tweets and in scrape_users
Added kwargs parameter to get author_id and users data when searching for recent tweets with client
Added test folder with:
 - conftest.py
 - test_scrape.py

 Added 2 simple test case for ensuring that blacklist_usernames are correctly loaded from environment variable and that has correct format
@sahal-mulki
Copy link
Contributor

So the goal is to manually pick out offenders?

@GabrieleCalarota
Copy link
Contributor Author

GabrieleCalarota commented Feb 27, 2022

So the goal is to manually pick out offenders?

At the moment there's no other way to do it, but as I see in the roadmap

Add misinformation prevention algorithm

It will probably be automated in some way in the following updates

In the meantime we will have a manual way to exclude accounts that we already know are not spreading the right informations

@kinshukdua kinshukdua merged commit 16202fc into kinshukdua:main Feb 27, 2022
@kinshukdua kinshukdua mentioned this pull request Feb 27, 2022
@sahal-mulki
Copy link
Contributor

Sure, cool

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add blacklist
3 participants